#musicbrainz-devel

/

      • zag
        yeh we are hitting the rate limiter in our testing, and offering to mirror
      • 2013-05-08 12814, 2013

      • ijabz
        The current reosarevok_ server has release events containing just date and area, NO country
      • 2013-05-08 12837, 2013

      • ocharles
        that sounds correct
      • 2013-05-08 12846, 2013

      • ocharles
        as reosarevok_ is running the areas branch
      • 2013-05-08 12847, 2013

      • ijabz
        Right, so both you and me need to make changes to use area instead of country
      • 2013-05-08 12858, 2013

      • ocharles
        I don't need to make any changes, as ian is handling the merge
      • 2013-05-08 12815, 2013

      • ijabz
        ok, unfortunately I do
      • 2013-05-08 12848, 2013

      • ruaok starts an export and runs off to meet up with the BMAT guys
      • 2013-05-08 12823, 2013

      • ijabz
        so that point you made about date should come before country is kind of moot, as there dis no longer country
      • 2013-05-08 12836, 2013

      • ocharles
        ijabz: true
      • 2013-05-08 12842, 2013

      • ocharles
        but it was invalid within my branch
      • 2013-05-08 12801, 2013

      • reosarevok_ joined the channel
      • 2013-05-08 12818, 2013

      • reosarevok_
        Of course it failed just when I changed my name
      • 2013-05-08 12820, 2013

      • reosarevok_
        Anyway
      • 2013-05-08 12824, 2013

      • reosarevok_
      • 2013-05-08 12857, 2013

      • ijabz
        well it will have to come after the schema release as ian hasn't left me any time to do this, and u still haven't raised an issue
      • 2013-05-08 12818, 2013

      • reosarevok_ doesn't see how he adding a ticket will make it happen faster, but ok, I will do that
      • 2013-05-08 12811, 2013

      • reosarevok
        Added
      • 2013-05-08 12818, 2013

      • ijabz
        i work from tickets
      • 2013-05-08 12837, 2013

      • reosarevok
        The search server isn't in git yet, right?
      • 2013-05-08 12848, 2013

      • reosarevok finds svn too confusing to try to do this himself
      • 2013-05-08 12808, 2013

      • ijabz
        not least because it provides a place to spec out the requests, discussions etc
      • 2013-05-08 12810, 2013

      • reosarevok
        Anyway, first I should fix all these tests
      • 2013-05-08 12820, 2013

      • ijabz
        no, thank goodness
      • 2013-05-08 12829, 2013

      • ocharles
        reosarevok: heh, never heard it that way round before :)
      • 2013-05-08 12859, 2013

      • ijabz
        Okay, but if you want to do search server ping me, I can guide you through, its really pretty easy
      • 2013-05-08 12812, 2013

      • reosarevok
        ok! :) I'll finish the broken bits here first
      • 2013-05-08 12817, 2013

      • reosarevok
        But after that I might try :)
      • 2013-05-08 12837, 2013

      • reosarevok
        ocharles: well, that's what I'm used to :)
      • 2013-05-08 12846, 2013

      • reosarevok
        maybe svn is easy, but it looks kinda intimidating
      • 2013-05-08 12814, 2013

      • ocharles
        iirc
      • 2013-05-08 12832, 2013

      • ijabz
        trust me if yo can use git you can use svn, anyway the exact commands you need for search server ar win the readme
      • 2013-05-08 12850, 2013

      • reosarevok
        huh
      • 2013-05-08 12854, 2013

      • reosarevok
      • 2013-05-08 12802, 2013

      • reosarevok really wonders wtf is with the docs
      • 2013-05-08 12820, 2013

      • reosarevok finds more tests that fail because of silly syntax errors he made
      • 2013-05-08 12828, 2013

      • reosarevok
        (on the tests themselves)
      • 2013-05-08 12808, 2013

      • reosarevok
        "
      • 2013-05-08 12809, 2013

      • reosarevok
        meeeh
      • 2013-05-08 12838, 2013

      • warp
        hola!
      • 2013-05-08 12807, 2013

      • reosarevok
        hola warp :)=
      • 2013-05-08 12855, 2013

      • warp looks at his release editor checklist.
      • 2013-05-08 12804, 2013

      • reosarevok
        Oh, so I need to add ids, huh :)
      • 2013-05-08 12819, 2013

      • reosarevok
        (or fix the ids or something)
      • 2013-05-08 12823, 2013

      • warp
        add ids?
      • 2013-05-08 12836, 2013

      • zag has left the channel
      • 2013-05-08 12845, 2013

      • reosarevok
        to the html to fix the for thingies
      • 2013-05-08 12851, 2013

      • warp
        ids are not usually mandatory in html :). (though they are often useful for our userscripting friends)
      • 2013-05-08 12811, 2013

      • warp
        ah, for clicking <label>s
      • 2013-05-08 12846, 2013

      • reosarevok
        yep
      • 2013-05-08 12826, 2013

      • ocharles
        meh, /ws/2 doesn't give you enough information to actually display a relationship
      • 2013-05-08 12855, 2013

      • ocharles
        at least, I don't think there's a way to get a verbose link phrase out of it
      • 2013-05-08 12857, 2013

      • reosarevok
        I think nikki did complain about how sucky ws/2 is for rels, yeah
      • 2013-05-08 12812, 2013

      • reosarevok
        (when trying to do stuff with it a while ago)
      • 2013-05-08 12814, 2013

      • ocharles
        I wanted to use /ws/2 to search for example relationships
      • 2013-05-08 12836, 2013

      • ocharles
        I may continue to do so, just with a crappy UI - but the UI is for relationship editors anyway so I don't mind shipping something a little rough
      • 2013-05-08 12840, 2013

      • ocharles
        burrito time now though
      • 2013-05-08 12802, 2013

      • UmkaDK joined the channel
      • 2013-05-08 12801, 2013

      • ijabz
        ocharles would that be http://tickets.musicbrainz.org/browse/SEARCH-179 feel free to comment on ticket
      • 2013-05-08 12801, 2013

      • reosarevok
        niiice, now it passes :)
      • 2013-05-08 12822, 2013

      • reosarevok
        function instead" all the time
      • 2013-05-08 12812, 2013

      • warp
        reosarevok: hm, I haven't seen that. perhaps that was recently added to Catalyst.
      • 2013-05-08 12824, 2013

      • reosarevok
        I mean, they work
      • 2013-05-08 12831, 2013

      • reosarevok
        they just say that over and over
      • 2013-05-08 12824, 2013

      • warp
        sure
      • 2013-05-08 12806, 2013

      • reosarevok fixes a few more issues, re-runs the script
      • 2013-05-08 12843, 2013

      • warp emacses lib/MusicBrainz/Server/Controller/CDTOC.pm some more.
      • 2013-05-08 12839, 2013

      • reosarevok
        D:
      • 2013-05-08 12841, 2013

      • reosarevok
        he said emacs
      • 2013-05-08 12848, 2013

      • reosarevok runs away
      • 2013-05-08 12836, 2013

      • Mineo
        "over and over" reminds me of http://youtu.be/9VDvgL58h_Y
      • 2013-05-08 12839, 2013

      • reosarevok
        hah
      • 2013-05-08 12845, 2013

      • reosarevok
        It's been a while since I saw that :)
      • 2013-05-08 12840, 2013

      • djce joined the channel
      • 2013-05-08 12841, 2013

      • ocharles
      • 2013-05-08 12811, 2013

      • warp
        what does it do?
      • 2013-05-08 12811, 2013

      • ocharles
        lets you write JavaScript in Haskell, by providing a monad that can be reified and compiled to javascript
      • 2013-05-08 12826, 2013

      • ocharles
        which is a bit different from the competition, which either try and parse haskell, or literally compile haskell to js
      • 2013-05-08 12805, 2013

      • ocharles
        So now we have blaze-html, clay and sunroof to build web applications in Haskell, all sharing code and types
      • 2013-05-08 12818, 2013

      • ocharles
        and aeson for json
      • 2013-05-08 12820, 2013

      • voiceinsideyou joined the channel
      • 2013-05-08 12820, 2013

      • warp
        ok, that sounds interesting.
      • 2013-05-08 12837, 2013

      • warp
        (I need more time per day)
      • 2013-05-08 12845, 2013

      • ocharles
        if you find some and have too much, i'll take some of that
      • 2013-05-08 12810, 2013

      • ijabz
        and me
      • 2013-05-08 12839, 2013

      • warp
        :)
      • 2013-05-08 12843, 2013

      • djce joined the channel
      • 2013-05-08 12812, 2013

      • reosarevok
      • 2013-05-08 12813, 2013

      • mb-chat-logger
      • 2013-05-08 12829, 2013

      • reosarevok
        Oh, nvm :)
      • 2013-05-08 12834, 2013

      • reosarevok
        that *should* be fine
      • 2013-05-08 12838, 2013

      • reosarevok wonders why it breaks here then
      • 2013-05-08 12840, 2013

      • reosarevok
        Oh, I see now :)
      • 2013-05-08 12816, 2013

      • reosarevok
        heh, keeps failing but with Can't locate object method "load_with_tracklist_for_recording" now
      • 2013-05-08 12812, 2013

      • warp
        that sounds like my area of breaking stuff
      • 2013-05-08 12826, 2013

      • reosarevok
        I guess we want load_with_medium_for_recording now
      • 2013-05-08 12818, 2013

      • reosarevok
        heh
      • 2013-05-08 12822, 2013

      • reosarevok
        DBD::Pg::st execute failed: ERROR: relation "tracklist" does not exist
      • 2013-05-08 12825, 2013

      • reosarevok
        fiine, not ready yet
      • 2013-05-08 12850, 2013

      • warp
        reosarevok: where/how did you get that? (so I can fix it)
      • 2013-05-08 12810, 2013

      • reosarevok
        Wait, I'm pulling in case it's fixed
      • 2013-05-08 12829, 2013

      • reosarevok
        nope
      • 2013-05-08 12833, 2013

      • reosarevok
        :)
      • 2013-05-08 12858, 2013

      • reosarevok
        There are a few tests
      • 2013-05-08 12854, 2013

      • warp
        hm, t/rdfa/recording/recording-rdfa.t seems to be the only one
      • 2013-05-08 12806, 2013

      • warp
        and perhaps the pgtap things, but I don't know exactly how that stuff works.
      • 2013-05-08 12814, 2013

      • warp
        the others seem unused
      • 2013-05-08 12833, 2013

      • reosarevok
        Well, if unused then we should remove them, right? :)
      • 2013-05-08 12800, 2013

      • reosarevok
        This is the t/lib/t/MusicBrainz/Server/Data/Release.pm one I think
      • 2013-05-08 12801, 2013

      • warp
        yes.
      • 2013-05-08 12813, 2013

      • warp
        Data::Release passes here
      • 2013-05-08 12804, 2013

      • reosarevok
        huh
      • 2013-05-08 12856, 2013

      • warp
        (here being mbs-6023-master in my account on bitbucket)
      • 2013-05-08 12857, 2013

      • mb-chat-logger
      • 2013-05-08 12829, 2013

      • reosarevok
        I just merged that one in
      • 2013-05-08 12838, 2013

      • reosarevok
        And it still fails here
      • 2013-05-08 12844, 2013

      • reosarevok
        But it fails in more places so I'll start with those
      • 2013-05-08 12850, 2013

      • reosarevok
        Maybe that'll magically fix it :p
      • 2013-05-08 12821, 2013

      • warp
        hrm, I don't know how the t/rdfa/* tests run either.
      • 2013-05-08 12851, 2013

      • Ben\Sput joined the channel
      • 2013-05-08 12829, 2013

      • CallerNo6 joined the channel
      • 2013-05-08 12811, 2013

      • outsidecontext joined the channel
      • 2013-05-08 12806, 2013

      • reosarevok
        huh
      • 2013-05-08 12807, 2013

      • reosarevok
        t::MusicBrainz::Server::Edit::Label::Edit
      • 2013-05-08 12816, 2013

      • reosarevok
        You must specify isni_codes at lib/MusicBrainz/Server/Edit/Role/ValueSet.pm line 18, <FILE> line 6.
      • 2013-05-08 12837, 2013

      • warp
        reosarevok: I think I've done enough release editor testing for now. anything I can do to help with what you're doing?
      • 2013-05-08 12824, 2013

      • reosarevok
        warp: maybe. Try running t::MusicBrainz::Server::Edit::Release::AddCoverArt
      • 2013-05-08 12838, 2013

      • warp
        on which branch?
      • 2013-05-08 12805, 2013

      • reosarevok
        Er, dunno, wherever you have your changes for file types
      • 2013-05-08 12821, 2013

      • ocharles gets tired of more runtime errors
      • 2013-05-08 12858, 2013

      • warp
        reosarevok: ok
      • 2013-05-08 12842, 2013

      • warp checks out mbs-4115
      • 2013-05-08 12843, 2013

      • mb-chat-logger
      • 2013-05-08 12858, 2013

      • reosarevok
        warp: I expect that just requires adding the same line I added to caa.sql to some other file, really
      • 2013-05-08 12850, 2013

      • reosarevok
        (the INSERT INTO cover_art_archive.image_type (mime_type, suffix) VALUES ('image/jpeg', 'jpg'); ) thingy
      • 2013-05-08 12853, 2013

      • reosarevok
        but unsure where
      • 2013-05-08 12824, 2013

      • warp
        reosarevok: possibly, though there may be some fussing with "image/jpeg" not being default for one of the columns.
      • 2013-05-08 12843, 2013

      • reosarevok
        dunno. if you fix it, I'll be happy :)
      • 2013-05-08 12846, 2013

      • warp
        (I wanted it default, ocharles didn't. I think ocharles made a ticket for it, but I cannot quickly find it now)
      • 2013-05-08 12834, 2013

      • djce joined the channel
      • 2013-05-08 12855, 2013

      • ZaphodBeeblebrox
        well I'd never seen it. agai nand agai nandagain. wtf :D
      • 2013-05-08 12821, 2013

      • hawke_1 joined the channel
      • 2013-05-08 12834, 2013

      • reosarevok
        ocharles: when you have some time, could you see if t::MusicBrainz::Server::Edit::Label::Edit passes in your ISNI branch in isolation?
      • 2013-05-08 12828, 2013

      • voiceinsideyou joined the channel
      • 2013-05-08 12827, 2013

      • hawke joined the channel
      • 2013-05-08 12832, 2013

      • ocharles
        warp: A ticket for it not being default?L
      • 2013-05-08 12842, 2013

      • ocharles
        I asked for it to not be the default until I gave you an approval on the PR
      • 2013-05-08 12846, 2013

      • warp
      • 2013-05-08 12838, 2013

      • ocharles
        ah
      • 2013-05-08 12850, 2013

      • warp
        ocharles: the approval was for the version where image/jpeg is not default