#musicbrainz-devel

2ms? nice.

to answer ocharles' important question: "wo ist plaintext" is indeed the german version of "where is plaintext" :)

ruaok: 2ms everywhere

but of coursse the first request is not 2ms

of course

MBChatLogger: off

is not logging

is logging

ruaok: just curious how many /ws/2 urls are fetched more than once within say 24 hours

ah, now I know why you were looking for plaintext.

:)

would you like a splunk login to write a query yourself?

that'd be swell!

I think we currently have a week's worth of data loaded.

k

ruaok: apparently it's public

(since nikki says she got a mail about it from SU)

k

Apparently they recommended MusicBrainz in their closing mail :p

reosarevok: oh good. thats why we chatted last week.

it was somewhat weird to be walking down the street in barcelona talking to amazon in the US.

ocharles: acid2 or ocharles as the login?

ocharles

done.

splunk.mb org

doesn't seem to load for me

oh wait, hosts

oh and for anyone with spotify premium. hit me up for a copy of the huesound app.

you'll need to enable a developer account to load the app.

but I've been having lots of fun with it.

Hmm, is splunk set up with the correct source type?

can't seem to extract just the request

I haven't looked at how plaintext imported the data.

but I know he successfully used it.

ok

interesting, there are definitely some things that would benefit

bizzarely on artists that no longer exist

ocharles: sounds much better, yeah, I was writing a complicated sed script to postprocess a server log and writing a script :P

ocharles: the script might still be useful, though I only got through a chunk of the artist WS endpoints

i'm off now, time to pick up new modules :D

warp: to get a translated server, you need to get all the .po files for your language (probably from https://www.transifex.net/projects/p/musicbrain... ), put them in /po, run 'make <whatever>.mo' and then 'make install'

warp: and add the language to MB_LANGUAGES

warp: ideally ultimately we keep the .po files in the git tree but for now they're on transifex

ianmcorvidae: that should be in an INSTALL or README somewhere. is it? :)

probably not, good point :)

heh, I realized I even missed a step

which is that they need to be named in specific ways

warp: added to the codereview :)

ianmcorvidae: great, thanks. I'll try it in a couple of minutes.

warp: as an aside, if you want *all* the translations, nikki's github has a 'translations' branch you could merge in (that's what the sandbox uses)

great, I'll try that.

Current version: 5.0.1-2

:)

Now if the guy would stop finding bugs in flactag as soon as I upload it, maybe that can migrate too!

Every time I upload a fix it resets the 10 day clock.

Maybe you should ask him to check it *before* you upload it :p

Well, I asked him to try out a patch on the last bug he reported and got no response for nearly a week, so figured it was easier just to fix it :)

ianmcorvidae: hrm. lots of conflicts.

warp: from what?

git merge nikki/translations

hm

into split-domains

so I guess I should get a translation manually then?

yeah, or just use checkout rather than an actual merge

it looks like the conflicts are in the .pot, which don't actually matter for your use :)

but then I won't have your split-domains work.

I mean checkout individual files, not the branch

ok, let's try again.

ianmcorvidae: I don't get a translated site yet. so there is more I need to do.

warp: what all have you done, then?

from nikki's branch make a copy of po/*nl*

put those in po/ on the split-domains branch

22:10 warp [at bullet-bill]:~/code/mb2/po (split-domains *)$ rm mb_server.nl.mo

22:10 warp [at bullet-bill]:~/code/mb2/po (split-domains *)$ make mb_server.nl.mo

22:10 warp [at bullet-bill]:~/code/mb2 (split-domains *)$ cat lib/DBDefs.pm | grep LANG

sub MB_LANGUAGES {qw(nl)}

make install

ah!

the .mo files need to end up in lib/LocaleData/<lang>/LC_MESSAGES/<domain>.mo :)

yeah, in fact, just doing make install should also make the .mo files, nikki set me straight on that :)

right. I still don't see anything translated though.

browser accept-language set?

noting that it needs to be exact, at least at the moment -- that's a later fix :)

ianmcorvidae: yes, http://i18n.mbsandbox.org/artist/301a88e0-d105-... has lots of dutch in the same browser.

hm

lib/LocaleData/nl/LC_MESSAGES/ has all nine .mo?

you demonstrated the DBDefs stuff was correct

22:14 warp [at bullet-bill]:~/code/mb2 (split-domains *)$ ls lib/LocaleData/nl/LC_MESSAGES/

attributes.mo countries.mo instrument_descriptions.mo instruments.mo languages.mo mb_server.mo relationships.mo scripts.mo statistics.mo

okay, that looks correct

22:15 warp [at bullet-bill]:~/code/mb2 (split-domains *)$ strings lib/LocaleData/nl/LC_MESSAGES/mb_server.mo | tail

{uri|Zoek op SoundUnwound}

there is actual dutch in there as well.

well hm

I'm stumped, everything looks correct there

I guess restart the server, to make sure there's not something funky with caching going on (though I should have fixed all known problems in that vein)

other than that I'm confused, unless you have enlightenment from STDERR or such :)

Use of uninitialized value in string eq at lib/MusicBrainz/Server/Controller/Artist.pm line 695.

every request says that about an uninitialized string, but no other errors.

yeah, that's probably because of memcached being stopped, something there about cookies

I did start memcached again ofcourse :)

huh

just to confirm: what's the sha1 you have checked out?

commit d15fc01fa879579cd9be308efa2ba21344f1e613

that's old

hm, I thought I did a git fetch.

ok, let's try again with newer code then :)

yeah, I think that's before the changes to actually *use* the new domains :)

commit 34c8013a514b97d9e3528c2ad761e43818c14bcd

that looks correct :)

22:27 warp [at bullet-bill]:~/code/mb2/po (split-domains)$ make mb_server.nl.mo

make: *** No rule to make target `mb_server.nl.mo'. Stop.

...

I'm very confused, with the same 34c8013... checkout it works fine for me :/

(GNU Make 3.82, here)

presumably the %.mo: %.po entry should apply

anyway what it runs is 'msgfmt -o <mo filename> <po filename>'

oh it seems the *nl* files got lost in switching/checking out branches.

oh, hah

so back at the make install step

and now we get some proper errors :)

Error while loading app.psgi: "web_set_domain" is not exported by the Locale::Util module

Can't continue after import errors at lib/MusicBrainz/Server/Translation.pm line 10

what's your libintl-perl version?

er, hm, it still seems to exist in even old versions

er, wait

that should be web_set_locale

this may be a mistake on my part then :)

alright. are you pushing something new?

yes, one sec

warp: should be there now,

sha1 7aa72d3fe788b00...

yay! but I still don't get anything translated.

Use of uninitialized value in string eq at lib/MusicBrainz/Server/Controller/Artist.pm line 695.

Argument "1,041934" isn't numeric in numeric eq (==) at local/lib/perl5/Catalyst.pm line 1804.

Argument "0,075352" isn't numeric in numeric eq (==) at local/lib/perl5/Catalyst.pm line 1804.

oh great, someone's using localized string formatting operators for things that are used programatically :/

hm

okay

I'm going to poke around at this more, because clearly I did something incorrectly

so, uh, thanks for your review, I'll get back to you when it seems to work again :/

ianmcorvidae: haha, ok :)

warp: oh, question