#musicbrainz-devel

/

      • ruaok
        what, you imply that you have a LIFE??
      • 2013-03-15 07459, 2013

      • ruaok
        tsk tsk
      • 2013-03-15 07401, 2013

      • Freso
        Nah. I just pretend.
      • 2013-03-15 07416, 2013

      • ruaok
        I once had a life. :)
      • 2013-03-15 07426, 2013

      • ruaok
        but you'll get this done then, yes?
      • 2013-03-15 07431, 2013

      • ruaok starts rummaging for the trans-atlantic cattle prod
      • 2013-03-15 07435, 2013

      • ruaok
        I wonder if you can get a degree in cat herding.
      • 2013-03-15 07404, 2013

      • ruaok
        "Hi, I'm a cat-herding major. With a focus on pedantic ADD cats. what is your major?"
      • 2013-03-15 07408, 2013

      • Freso
        I'll get the SQL done, yes.
      • 2013-03-15 07420, 2013

      • ruaok
        awesome, thanks. :)
      • 2013-03-15 07424, 2013

      • Freso
        ruaok: There is already and SQL patch, if you want to start trying to roll it.
      • 2013-03-15 07433, 2013

      • ruaok
        I will on monday.
      • 2013-03-15 07434, 2013

      • Freso
        ruaok: It just doesn't have any new and changed indexes.
      • 2013-03-15 07441, 2013

      • ruaok
        k
      • 2013-03-15 07429, 2013

      • ianmcorvidae
        Freso: needs adding to upgrade.sh, adding to CreateTables, FKs at least :)
      • 2013-03-15 07436, 2013

      • ianmcorvidae
        (before it can be used for testing)
      • 2013-03-15 07403, 2013

      • Leftmost
        Got my SQL in review.
      • 2013-03-15 07434, 2013

      • tonyschuite
        darn... apparently i should use gmake instead of make :P
      • 2013-03-15 07443, 2013

      • Freso
        ianmcorvidae: Uhm... mind writing that on the ticket? :D
      • 2013-03-15 07406, 2013

      • ianmcorvidae
        Freso: uh, sure
      • 2013-03-15 07418, 2013

      • Freso
        ianmcorvidae: ^^
      • 2013-03-15 07444, 2013

      • warp
        ruaok: pong
      • 2013-03-15 07451, 2013

      • ruaok
        hi warp.
      • 2013-03-15 07411, 2013

      • ruaok
        I did some tidying up of your tickets.
      • 2013-03-15 07417, 2013

      • ruaok
        removed that one ticket for series.
      • 2013-03-15 07421, 2013

      • warp
        ok, thanks
      • 2013-03-15 07430, 2013

      • ruaok
        please take a look at your tickets and make sure all is well.
      • 2013-03-15 07447, 2013

      • ruaok
        I would like to announce the final set of tickets on monday.
      • 2013-03-15 07456, 2013

      • ruaok
        and start the first data conversions then as well.
      • 2013-03-15 07436, 2013

      • warp nods.
      • 2013-03-15 07401, 2013

      • ruaok
        i'm going to send out a summary email to everyone who has schema change tickets, just to make sure we're all on the same page.
      • 2013-03-15 07420, 2013

      • tonyschuite
        does anyone now how to pull an FTP dir from ssh?
      • 2013-03-15 07428, 2013

      • warp
        I'm going to start a fresh fullexport import now, so I can do some testing of MBS-1839 tomorrow.
      • 2013-03-15 07428, 2013

      • mb-chat-logger
      • 2013-03-15 07435, 2013

      • ianmcorvidae
        tonyschuite: wget knows ftp, that should do?
      • 2013-03-15 07444, 2013

      • ruaok
        warp: thanks. I appreciate that. I think everyone should do that on their branches.
      • 2013-03-15 07450, 2013

      • tonyschuite
        no wget on freenas :(
      • 2013-03-15 07404, 2013

      • tonyschuite
        i thought i could use the fetch command but it doesnt exist
      • 2013-03-15 07428, 2013

      • warp
        ruaok: my schema change tickets both need a tiny bit more work which I will also finish tomorrow. I've had a bunch of feedback on them already, so hopefully they'll be ready then.
      • 2013-03-15 07440, 2013

      • ruaok
        ok, great.
      • 2013-03-15 07439, 2013

      • MBJenkins
        * Ian McEwen: MBS-5491: check for 'vs.' and 'feat.' in PossibleCollaborations report
      • 2013-03-15 07440, 2013

      • MBJenkins
        * Ian McEwen: MBS-6012: when extracts get an empty string back from wikipedia, don't try to display them
      • 2013-03-15 07440, 2013

      • warp
        (UI changes is a bit misleading for MBS-6023, because the vast majority of the changes shouldn't affect the interface at all)
      • 2013-03-15 07441, 2013

      • mb-chat-logger
      • 2013-03-15 07441, 2013

      • MBJenkins
        * Ian McEwen: MBS-4364: update tests to ensure VA doesn't get arbitrary attributes added to it by merges
      • 2013-03-15 07442, 2013

      • MBJenkins
        * Ian McEwen: MBS-4364: also don't merge IPIs
      • 2013-03-15 07446, 2013

      • tonyschuite
        ok, got it... was a syntax thing... i can use fetch :)
      • 2013-03-15 07416, 2013

      • ianmcorvidae
        er
      • 2013-03-15 07417, 2013

      • ianmcorvidae
        huh
      • 2013-03-15 07422, 2013

      • ianmcorvidae
        that test doesn't fail locally D:
      • 2013-03-15 07446, 2013

      • ianmcorvidae
        heh, of course, we don't consistently have artist ID 1 be VA D:
      • 2013-03-15 07446, 2013

      • warp
        ianmcorvidae: locally you don't have perl v5.old on ubuntu 10.weird.
      • 2013-03-15 07459, 2013

      • ianmcorvidae
        warp: nah, it's the DB constraint, I didn't refresh my test DB
      • 2013-03-15 07405, 2013

      • warp
        ok :)
      • 2013-03-15 07422, 2013

      • MBJenkins
        Project musicbrainz-server_beta build #378: STILL FAILING in 4 min 31 sec: http://ci.musicbrainz.org/job/musicbrainz-server_…
      • 2013-03-15 07423, 2013

      • MBJenkins
        Ian McEwen: MBS-5912: actually list cover art relationships report
      • 2013-03-15 07442, 2013

      • ianmcorvidae
        warp, ocharles: I'm fixing that test and pushing to beta for ci, but in light of our conversations, shall I subsequently open a review for it?
      • 2013-03-15 07425, 2013

      • Freso joined the channel
      • 2013-03-15 07455, 2013

      • warp
        ianmcorvidae: in general I'm fine with small fixes to make beta not fail. IMO that doesn't have to go through review.
      • 2013-03-15 07451, 2013

      • ianmcorvidae
        hm, apparently bitbucket isn't closing PRs automatically
      • 2013-03-15 07405, 2013

      • warp
        it is
      • 2013-03-15 07424, 2013

      • ianmcorvidae
        having just pushed out five changes it didn't, guessing no :)
      • 2013-03-15 07444, 2013

      • warp
        (it takes a few seconds for bitbucket to realize a pull request is closed if you just push it, but so far it has always closed them for me)
      • 2013-03-15 07454, 2013

      • ianmcorvidae
        hm
      • 2013-03-15 07414, 2013

      • ianmcorvidae
        must be taking a lot longer than usual then, I waited a few minutes
      • 2013-03-15 07423, 2013

      • warp
        or something about that particular pull requests is confusing bitbucket.
      • 2013-03-15 07439, 2013

      • hawke_1
        Good god ISRCs are so DUMB
      • 2013-03-15 07455, 2013

      • tonyschuite
        Can't exec "./admin/InitDb.pl": Permission denied at /usr/local/lib/perl5/site_perl/5.10.1/Carton/CLI.pm line 320.
      • 2013-03-15 07457, 2013

      • hawke_1
        “When the supply of registrant codes for use with the United Kingdom country code of “GB” has been exhausted, new registrant codes will be allocated for use with a new “UK” country code. This is expected to happen in early 2013.”
      • 2013-03-15 07458, 2013

      • tonyschuite
        fock!
      • 2013-03-15 07408, 2013

      • warp
        hawke_1: lol
      • 2013-03-15 07426, 2013

      • ianmcorvidae
        the US has been using QM codes for a while
      • 2013-03-15 07428, 2013

      • reosarevok
        hawke_1: why is that so dumb?
      • 2013-03-15 07403, 2013

      • warp
        tonyschuite: perhaps the script isn't +x ?
      • 2013-03-15 07413, 2013

      • ianmcorvidae
        also, there's a reason they're trying to figure out a replacement for the ISRC system :P
      • 2013-03-15 07430, 2013

      • hawke_1
        reosarevok: 'we didn't plan ahead and make our code long enough, so we’ll go through some weird contortions to shoehorn something into the existing system'
      • 2013-03-15 07436, 2013

      • warp
        tonyschuite: you can try putting perl in front of it, so something like carton exec -Ilib -- perl ./admin/InitDb.pl -- --createdb etc...
      • 2013-03-15 07449, 2013

      • tonyschuite
        warp: what do you mean?
      • 2013-03-15 07440, 2013

      • hawke_1
        and national prefixes will disappear” but that’s not mentioned in that document of course
      • 2013-03-15 07402, 2013

      • nikki
        I wonder why they chose DG
      • 2013-03-15 07442, 2013

      • nikki
        (instead of a user-assigned one like QM)
      • 2013-03-15 07404, 2013

      • hawke_1
        nikki: “[CP and DG] are used by the International Telecommunications Union to identify certain technical installations (on Clipperton Island and Diego Garcia respectively) and no ambiguity can arise.”
      • 2013-03-15 07418, 2013

      • reosarevok
        nikki, because they are the voiced alternatives to /T/ and /K/!
      • 2013-03-15 07425, 2013

      • reosarevok
        Oh, bah, my explanation was cooler
      • 2013-03-15 07450, 2013

      • hawke_1
        yes, it is. :-)
      • 2013-03-15 07451, 2013

      • nikki
        hawke_1: it still seems odd to me
      • 2013-03-15 07418, 2013

      • hawke_1
        nikki: I think they just went “well, what can we use that won’t break too many peoples’ validation systems?”
      • 2013-03-15 07441, 2013

      • hawke_1
        since their validation specifies that everything in ISO 3166-1 is allowed…
      • 2013-03-15 07459, 2013

      • hawke_1
        …that means that DG is already allowed.
      • 2013-03-15 07409, 2013

      • hawke_1
        of course that doesn’t explain the inconsistency, but hey whatever
      • 2013-03-15 07417, 2013

      • nikki
        technically DG is only "exceptionally reserved" :P
      • 2013-03-15 07429, 2013

      • hawke_1
        nikki: yes, they say that too.
      • 2013-03-15 07450, 2013

      • hawke_1
        I didn’t paste the whole bulletin. :-p
      • 2013-03-15 07405, 2013

      • nikki
        and if they're worried about breaking stuff that doesn't believe QM to QZ exists, then they're a bit late
      • 2013-03-15 07439, 2013

      • hawke_1
        Yes, it’s completely stupid.
      • 2013-03-15 07457, 2013

      • hawke_1
        Now can I move on to the stupidity in their validation criteria?
      • 2013-03-15 07458, 2013

      • hawke_1
      • 2013-03-15 07407, 2013

      • warp
        don't underestimate how dodgy most software is
      • 2013-03-15 07429, 2013

      • hawke_1
        “An ISRC is always invalid if:… it is the same as the code assigned to a different recording It is a fundamental principle of the ISRC system that two different recordings must have different ISRCs. If an ISRC is found that has already been seen assigned to a different recording, then an error has occurred. The source of the error should be investigated.”
      • 2013-03-15 07446, 2013

      • hawke_1
        How do you know if a recording is different? Well, it has a different ISRC of course!
      • 2013-03-15 07407, 2013

      • Mineo joined the channel
      • 2013-03-15 07410, 2013

      • hawke_1
        (well, or through some other method of determining wtf a recording is)
      • 2013-03-15 07436, 2013

      • hawke_1
        So yeah…ISRC is pretty much fucking useless.
      • 2013-03-15 07409, 2013

      • tonyschuite
        well... quitting for now
      • 2013-03-15 07415, 2013

      • tonyschuite
        thnx for the tips!
      • 2013-03-15 07417, 2013

      • tonyschuite
        ciao
      • 2013-03-15 07418, 2013

      • MBJenkins
        Project musicbrainz-server_beta build #379: STILL FAILING in 28 min: http://ci.musicbrainz.org/job/musicbrainz-server_…
      • 2013-03-15 07419, 2013

      • MBJenkins
        Ian McEwen: MBS-4364: correct Edit::Artist::Edit test to not use the special ID 1 for testing
      • 2013-03-15 07433, 2013

      • ianmcorvidae
        feh
      • 2013-03-15 07437, 2013

      • ianmcorvidae
        no doubt we're using 1 elsewhere then
      • 2013-03-15 07455, 2013

      • ianmcorvidae
        oh, nope
      • 2013-03-15 07400, 2013

      • ianmcorvidae
        but still something I should have seen, heh
      • 2013-03-15 07409, 2013

      • reosarevok
        ianmcorvidae: shouldn't you use gids and not rowids anyway to ensure the test remains trustworthy over time?
      • 2013-03-15 07435, 2013

      • ianmcorvidae
        reosarevok: possibly? I mean, it's for VA, which really is always ID 1
      • 2013-03-15 07439, 2013

      • ianmcorvidae
        except in tests
      • 2013-03-15 07446, 2013

      • ianmcorvidae
        well, except in that one test, now fixed :P
      • 2013-03-15 07457, 2013

      • reosarevok
        ianmcorvidae: oh, ok, it is a test *for* VA
      • 2013-03-15 07429, 2013

      • reosarevok
        I thought the problem was someone had given rowid 1 for a random test and it had broken now because of enforced no-gender no-type
      • 2013-03-15 07434, 2013

      • reosarevok
        ignore me then :)
      • 2013-03-15 07443, 2013

      • ianmcorvidae
        yes, that is what happened
      • 2013-03-15 07401, 2013

      • ianmcorvidae
        because that test used rowid 1, when rowid 1 is, in every other case, always going to be VA :)
      • 2013-03-15 07451, 2013

      • warp
        NOT READY, build status is red_anime.
      • 2013-03-15 07429, 2013

      • ianmcorvidae
        partitions are hard (and cool) :D
      • 2013-03-15 07443, 2013

      • reosarevok
        ianmcorvidae, does that mean you're fighting geordi? :p
      • 2013-03-15 07407, 2013

      • ianmcorvidae
        reosarevok: no, it means I'm discussing how to improve the dated/undated rels merge patch with ocharles :)
      • 2013-03-15 07407, 2013

      • warp runs a fullimport
      • 2013-03-15 07411, 2013

      • reosarevok
        Oh
      • 2013-03-15 07415, 2013

      • reosarevok
        With partitions?
      • 2013-03-15 07421, 2013

      • ianmcorvidae
        reosarevok: to be clear, when I say partition I mean as in PARTITION BY, the SQL command :P
      • 2013-03-15 07425, 2013

      • reosarevok
        oh
      • 2013-03-15 07426, 2013

      • reosarevok
        :)
      • 2013-03-15 07430, 2013

      • ianmcorvidae
        reosarevok: which you may not know exists, I admit :)
      • 2013-03-15 07440, 2013

      • reosarevok
        Yeah, that wasn't in my introductory course :D
      • 2013-03-15 07453, 2013

      • warp
        :)
      • 2013-03-15 07424, 2013

      • ianmcorvidae
        we may have a way to git rid of some of my perl hackery and turn it into SQL, plus combine it with other SQL to make the whole situation shorter
      • 2013-03-15 07427, 2013

      • ianmcorvidae
        oh
      • 2013-03-15 07428, 2013

      • warp
        import running. tomorrow will be more testing and fixing of schema change branches. now sleep, goodnight!
      • 2013-03-15 07436, 2013

      • ianmcorvidae
        though I just thought of why I'd done it in perl in the first place, hm.
      • 2013-03-15 07408, 2013

      • reosarevok
        bitbucket rid of it!
      • 2013-03-15 07414, 2013

      • reosarevok
        svn rid of it!
      • 2013-03-15 07419, 2013

      • reosarevok
        Ok, I'll shut up now
      • 2013-03-15 07443, 2013

      • ianmcorvidae
        bitbucket isn't a VCS, it's a hosting service :P
      • 2013-03-15 07455, 2013

      • ianmcorvidae
        guessing you meant hg rid of it or darcs rid of it ;)
      • 2013-03-15 07437, 2013

      • warp
        aw, poor bzr
      • 2013-03-15 07444, 2013

      • kepstin-work bzrs rid of stuff.
      • 2013-03-15 07419, 2013

      • ianmcorvidae
        time to perforce rid of this conversation, I think
      • 2013-03-15 07440, 2013

      • djce joined the channel
      • 2013-03-15 07409, 2013

      • kepstin-work
        djce: clearcase over here and join the conversation!
      • 2013-03-15 07428, 2013

      • ijabz joined the channel
      • 2013-03-15 07419, 2013

      • misterswag joined the channel