#musicbrainz-devel

/

      • ianmcorvidae
        hm
      • 2012-11-28 33317, 2012

      • ianmcorvidae
        if it has the ternary operator it would
      • 2012-11-28 33314, 2012

      • Freso
        I guess it should be reversed though. "foo.defined ? foo : bar;"
      • 2012-11-28 33325, 2012

      • ianmcorvidae
        yeah
      • 2012-11-28 33356, 2012

      • ianmcorvidae
        yeah, it supports it looks like
      • 2012-11-28 33304, 2012

      • Freso
        What happens if you `shift` and there are no more arguments?
      • 2012-11-28 33307, 2012

      • Freso
        (Back in Perl now. :))
      • 2012-11-28 33321, 2012

      • ianmcorvidae
        I think you get undef
      • 2012-11-28 33329, 2012

      • ianmcorvidae
        but it might crash on you
      • 2012-11-28 33334, 2012

      • ianmcorvidae
        you might appreciate Devel::REPL :P
      • 2012-11-28 33343, 2012

      • ianmcorvidae
        (install it, re.pl provides a perl repl)
      • 2012-11-28 33318, 2012

      • ianmcorvidae
        yeah, you get undef looks like
      • 2012-11-28 33340, 2012

      • Freso
        What's "a perl repl"?
      • 2012-11-28 33338, 2012

      • ianmcorvidae
        do you know what a repl is?
      • 2012-11-28 33353, 2012

      • ianmcorvidae
        because a 'perl repl' is a repl for perl :P
      • 2012-11-28 33356, 2012

      • Freso
        No. :)
      • 2012-11-28 33359, 2012

      • ianmcorvidae
        ah
      • 2012-11-28 33303, 2012

      • ianmcorvidae
        read-eval-print loop
      • 2012-11-28 33304, 2012

      • Freso
        If I knew, I doubt I'd've asked.
      • 2012-11-28 33314, 2012

      • Freso
        Ah.
      • 2012-11-28 33321, 2012

      • ianmcorvidae
        slightly more precise term for an interactive shell for a language, but
      • 2012-11-28 33352, 2012

      • ianmcorvidae
        amusingly, in common lisp, you can actually implement a (poor) one with (loop(print(eval(read))) :P
      • 2012-11-28 33305, 2012

      • Freso
        :)
      • 2012-11-28 33346, 2012

      • ianmcorvidae
        anyway, yeah, Devel::REPL gives you a nice way to test things like "what happens if you shift an empty list"
      • 2012-11-28 33324, 2012

      • Freso
        The TT docs don't seem to note how you can pass multiple variables to a FILTER. Or that it's only able to take one argument.
      • 2012-11-28 33354, 2012

      • ianmcorvidae|alt joined the channel
      • 2012-11-28 33306, 2012

      • ianmcorvidae|alt
        hm
      • 2012-11-28 33311, 2012

      • ianmcorvidae|alt
        stupid connection
      • 2012-11-28 33358, 2012

      • flamingspinach joined the channel
      • 2012-11-28 33304, 2012

      • ianmcorvidae
        anyway as I was saying, multiple variables you'd just pass separately
      • 2012-11-28 33311, 2012

      • ianmcorvidae
        e.g. var1 | filter + var2 | filter
      • 2012-11-28 33316, 2012

      • ianmcorvidae
        or such
      • 2012-11-28 33351, 2012

      • ianmcorvidae
        and if you want to pass multiple things to a given thing, you probably want a macro, not a filter
      • 2012-11-28 33303, 2012

      • ianmcorvidae
        if it must be done in perl, decompose it to something that can take one argument
      • 2012-11-28 33345, 2012

      • Freso
        Yeah. I was hoping I wouldn't have to fall back on regexes.
      • 2012-11-28 33355, 2012

      • Freso
        Hm. But perhaps I don't.
      • 2012-11-28 33358, 2012

      • CatCat joined the channel
      • 2012-11-28 33301, 2012

      • ianmcorvidae
        as before, it might help if you explain what you're trying to do :P
      • 2012-11-28 33319, 2012

      • Freso
        \o/
      • 2012-11-28 33327, 2012

      • Freso
        ianmcorvidae: Where'd the fun be in that?
      • 2012-11-28 33304, 2012

      • ianmcorvidae
        it'd be less obnoxious than the long stream of questions :P but other than that
      • 2012-11-28 33338, 2012

      • warp
        >_<
      • 2012-11-28 33304, 2012

      • mb-chat-logger joined the channel
      • 2012-11-28 33315, 2012

      • warp
        unrelated: swagger isn't looking very promising.
      • 2012-11-28 33322, 2012

      • ianmcorvidae
        ah :/
      • 2012-11-28 33314, 2012

      • warp
        mostly due to the interplay between 1) it's a spec with many implementations 2) the spec is not flexible/expressive enough to describe our API
      • 2012-11-28 33319, 2012

      • ianmcorvidae
        also unrelated: my head feeling like it's slightly overstuffed with cotton balls is not helping me get work done :/
      • 2012-11-28 33324, 2012

      • ianmcorvidae
        heh
      • 2012-11-28 33302, 2012

      • ianmcorvidae
        the ?inc thing isn't really that uncommon (under different names, of course), so I'm sort of surprised that's posing so much trouble
      • 2012-11-28 33303, 2012

      • warp
        so i/o docs looks more promising, there is no spec seperate from the implementation, so it's easy to mess with it and get some real work done :)
      • 2012-11-28 33334, 2012

      • ianmcorvidae
        heh, yeah
      • 2012-11-28 33330, 2012

      • hawke_1 joined the channel
      • 2012-11-28 33344, 2012

      • Prophet5_ joined the channel
      • 2012-11-28 33358, 2012

      • warp
        ianmcorvidae: cotton balls does not sound fun.
      • 2012-11-28 33342, 2012

      • Freso
        ianmcorvidae: Is https://pastee.org/ttmen because of bad/lacking configuration on my part, something missing on rika, or a genuine code bug?
      • 2012-11-28 33344, 2012

      • ianmcorvidae
        yeah, my body is showing remarkable persistence in failing to get over this illness
      • 2012-11-28 33356, 2012

      • warp
        warp has changed the topic to: tropical week | http://musicbrainz.org/#devel | agenda: reviews, MBS-2021 (DR/hawke), MBS-5035 (various, ian can talk about it), caa (warp)
      • 2012-11-28 33320, 2012

      • ianmcorvidae
        Freso: rika's static DB hasn't been updated for the schema change, so probably that
      • 2012-11-28 33338, 2012

      • Freso
        ianmcorvidae: Okay.
      • 2012-11-28 33351, 2012

      • ianmcorvidae
        guess I should fix that :)
      • 2012-11-28 33317, 2012

      • Freso
        Does anyone know of a user profile of a deleted user I can see? Or is it even possible to see user profiles of deleted users?
      • 2012-11-28 33332, 2012

      • ianmcorvidae
        I think it's possible?
      • 2012-11-28 33354, 2012

      • ianmcorvidae
        you can search for "Deleted Editor" :P
      • 2012-11-28 33308, 2012

      • Freso
        I can? I'll try. Ty.
      • 2012-11-28 33341, 2012

      • ianmcorvidae
      • 2012-11-28 33344, 2012

      • ianmcorvidae
        for example
      • 2012-11-28 33340, 2012

      • Freso
        Looks like it's working \o/
      • 2012-11-28 33354, 2012

      • Freso
        Time to run tests then, I guess.
      • 2012-11-28 33326, 2012

      • 45PABEIK8 joined the channel
      • 2012-11-28 33326, 2012

      • holmes.freenode.net
        holmes.freenode.net has changed the topic to: tropical week | http://musicbrainz.org/#devel | agenda: reviews, MBS-2021 (DR/hawke), MBS-5035 (various, ian can talk about it)
      • 2012-11-28 33309, 2012

      • ianmcorvidae
        uh, hm
      • 2012-11-28 33328, 2012

      • ianmcorvidae
        ianmcorvidae has changed the topic to: tropical week | http://musicbrainz.org/#devel | agenda: reviews, MBS-2021 (DR/hawke), MBS-5035 (various, ian can talk about it), caa (warp)
      • 2012-11-28 33334, 2012

      • warp
      • 2012-11-28 33301, 2012

      • ianmcorvidae
        heh
      • 2012-11-28 33335, 2012

      • warp
        goodnight sirs!
      • 2012-11-28 33338, 2012

      • warp
        ianmcorvidae: get well
      • 2012-11-28 33312, 2012

      • Freso
        ....
      • 2012-11-28 33324, 2012

      • ianmcorvidae
        thanks, I hope so :)
      • 2012-11-28 33330, 2012

      • Freso
        navap's fix for his non-escaping of Git data is on the beta branch, right?
      • 2012-11-28 33339, 2012

      • ianmcorvidae
        should be
      • 2012-11-28 33342, 2012

      • Freso breaks tests all around with his "'s.
      • 2012-11-28 33314, 2012

      • LordSputnik joined the channel
      • 2012-11-28 33341, 2012

      • LordSputnik
        anyone here know how to set up picard so that i can just run tagger.py from my local repo?
      • 2012-11-28 33357, 2012

      • LordSputnik
        (instead of running setup.py)
      • 2012-11-28 33355, 2012

      • LordSputnik
        ... don't worry, read INSTALL.txt :P
      • 2012-11-28 33319, 2012

      • demosdemon joined the channel
      • 2012-11-28 33354, 2012

      • ijabz joined the channel
      • 2012-11-28 33343, 2012

      • Freso just uses a customised picard-git PKGBUILD to fetch from his local repos instead of the official one...
      • 2012-11-28 33353, 2012

      • Freso
        This test is taking a while. I'll take that as a good sign.
      • 2012-11-28 33326, 2012

      • Freso
        https://pastee.org/sf88m - it looks like the errors in the tests are due to failed connections to the validator.
      • 2012-11-28 33322, 2012

      • Freso
        "# Failed test 'valid HTML5, Could not connect to DBDefs::HTML_VALIDATOR'"
      • 2012-11-28 33313, 2012

      • Freso
        And some others because it can't find an RNG schema...
      • 2012-11-28 33340, 2012

      • ianmcorvidae
        it shouldn't actually fail from those
      • 2012-11-28 33347, 2012

      • ianmcorvidae
        just yell at you :P
      • 2012-11-28 33308, 2012

      • Freso
        It certainly looks like they're marked as fails.
      • 2012-11-28 33323, 2012

      • ianmcorvidae
        yeah, it looks like it is
      • 2012-11-28 33326, 2012

      • ianmcorvidae
        it shouldn't :P
      • 2012-11-28 33340, 2012

      • ianmcorvidae
        RNG schema stuff definitely shouldn't, and hasn't before
      • 2012-11-28 33302, 2012

      • Freso
        I'm not sure the RNG stuff is related.
      • 2012-11-28 33306, 2012

      • Freso
        That may be my setup.
      • 2012-11-28 33320, 2012

      • Freso
        I'm looking through the result more systematically now.
      • 2012-11-28 33330, 2012

      • ianmcorvidae
        the RNG stuff isn't relevant in this case
      • 2012-11-28 33340, 2012

      • ianmcorvidae
        it does look like you're not connecting to the html validator correctly
      • 2012-11-28 33358, 2012

      • ianmcorvidae
        hm
      • 2012-11-28 33306, 2012

      • ianmcorvidae
        it may need to have -> rather than ::
      • 2012-11-28 33329, 2012

      • ianmcorvidae
        yeah
      • 2012-11-28 33333, 2012

      • Freso
        :p
      • 2012-11-28 33351, 2012

      • ianmcorvidae
        not my fault people apparently still haven't bothered fixing that merge :P
      • 2012-11-28 33310, 2012

      • ianmcorvidae
        in lib/MusicBrainz/Server/Test/HTML5.pm line 185, change :: to ->
      • 2012-11-28 33354, 2012

      • Freso
        Ugh.
      • 2012-11-28 33359, 2012

      • Freso tries again - third time's the charm!
      • 2012-11-28 33315, 2012

      • ianmcorvidae
        Freso: also, just deploying the very tiny fix for the thing you mentioned on MBS-4108
      • 2012-11-28 33315, 2012

      • mb-chat-logger
      • 2012-11-28 33352, 2012

      • Freso
        ianmcorvidae: \o/
      • 2012-11-28 33314, 2012

      • ianmcorvidae
        stupid little thing, I did all the sorting and everything but I needed the end of the list, not the start :P
      • 2012-11-28 33320, 2012

      • Freso
        ianmcorvidae: I mentioned it earlier in #musicbrainz (and highlighted you too), but I guess you didn't see it, so I put it in the ticket. :)
      • 2012-11-28 33327, 2012

      • ianmcorvidae
        heh
      • 2012-11-28 33329, 2012

      • Freso
        ianmcorvidae: Heh. :)
      • 2012-11-28 33334, 2012

      • ianmcorvidae
        I should go off away and see what else I've missed
      • 2012-11-28 33350, 2012

      • Freso
        ;)
      • 2012-11-28 33326, 2012

      • ianmcorvidae
        heh, possibly it's now just failing to get an extract at all
      • 2012-11-28 33329, 2012

      • ianmcorvidae
        :P
      • 2012-11-28 33353, 2012

      • Freso
        \o/
      • 2012-11-28 33337, 2012

      • ianmcorvidae
        https://beta.musicbrainz.org/artist/cd4f5ac3-5e0e… now appears to work correctly though :D
      • 2012-11-28 33354, 2012

      • ianmcorvidae
        the german one it fails to load though
      • 2012-11-28 33355, 2012

      • ianmcorvidae
        bah
      • 2012-11-28 33356, 2012

      • Freso
        ianmcorvidae: https://pastee.org/3p732 and after the "fix" in HTML5.pm, the tests just break outright.
      • 2012-11-28 33331, 2012

      • ianmcorvidae
        have you fixed your DBDefs since the last release?
      • 2012-11-28 33345, 2012

      • Freso
        Hm... I guess not.
      • 2012-11-28 33304, 2012

      • ianmcorvidae
        add "use parent 'DBDefs::Default';"
      • 2012-11-28 33325, 2012

      • ianmcorvidae
        that's really the only super important change, especially for what you're doing
      • 2012-11-28 33332, 2012

      • ianmcorvidae
        you could also delete most of the file, but :P
      • 2012-11-28 33331, 2012

      • Freso
      • 2012-11-28 33345, 2012

      • ianmcorvidae
        yeah :P
      • 2012-11-28 33319, 2012

      • ianmcorvidae
        hm
      • 2012-11-28 33324, 2012

      • ianmcorvidae
        now it's trying to get the german page via en?
      • 2012-11-28 33332, 2012

      • Freso
        Dunno.
      • 2012-11-28 33340, 2012

      • ianmcorvidae looking locally
      • 2012-11-28 33315, 2012

      • hawke joined the channel
      • 2012-11-28 33344, 2012

      • kepstin-work notes that on http://people.scs.carleton.ca/~cwalton3/temp/mbmockups/ajax/?artist=cd4f5ac3-5e0e-45a0-ae01-32c3cf117071 he is doing no language detection whatsoever, and is just using the first wikipedia link in the webservice response :/
      • 2012-11-28 33347, 2012

      • kepstin-work
        i should fix that...
      • 2012-11-28 33318, 2012

      • ianmcorvidae
        apparently it's completely failing to actually sort the list
      • 2012-11-28 33320, 2012

      • ianmcorvidae
        bah
      • 2012-11-28 33334, 2012

      • hawke_1 joined the channel
      • 2012-11-28 33350, 2012

      • Freso
        ianmcorvidae: Can't you map it to $lang => $lang.wp.org/... falling back to looking up $lang against $random.wp.org falling back to using $random.wp.org?
      • 2012-11-28 33305, 2012

      • ianmcorvidae
        that's largely what it does, yes
      • 2012-11-28 33323, 2012

      • ianmcorvidae
        when I do the sorting correctly :P
      • 2012-11-28 33325, 2012

      • Freso
        Well... okay then. :)
      • 2012-11-28 33345, 2012

      • Freso
        (That was also something like that I'd imagined to do for the Bottle.)
      • 2012-11-28 33335, 2012

      • ianmcorvidae
        yeah; what it does is looks at interface language, if it's set chooses that as the "wanted language", otherwise en