hm, I guess I should sign up for MHD london if I'm ostensibly going, shouldn't I
2012-10-10 28445, 2012
CallerNo6 joined the channel
2012-10-10 28426, 2012
bloopletech joined the channel
2012-10-10 28404, 2012
bloopletech
Hello, I am currently setting up a local copy of the musicbrainz server software and loading a dump, so I can process my music collection more quickly
2012-10-10 28434, 2012
bloopletech
I am up to the initdb stage, but with the most recently available dump, I get a duplicate key error in createindexes
2012-10-10 28404, 2012
bloopletech
Is this a known issue at all? Can I do something about this so I can successfully import the dump?
2012-10-10 28423, 2012
ianmcorvidae
it's known
2012-10-10 28428, 2012
ianmcorvidae
comment out the relevant index
2012-10-10 28441, 2012
ianmcorvidae
it'll stop being a problem with the next dump, but the current one it's still a problem :(
2012-10-10 28406, 2012
ianmcorvidae
if you've already gotten as far as the CreateIndexes stage, what you can do in order to not need to rerun the import step is:
2012-10-10 28422, 2012
ianmcorvidae
a.) comment out the broken index(es) (I think it may just be label_idx_null_comment at this point though)
2012-10-10 28444, 2012
ianmcorvidae
b.) comment out the stuff that's already been done in InitDb.pl
2012-10-10 28457, 2012
bloopletech
oh, awesome!
2012-10-10 28407, 2012
bloopletech
Sorry I went away thinking no one was on
2012-10-10 28410, 2012
ianmcorvidae
specifically, for b.): lines 236-276
2012-10-10 28416, 2012
bloopletech
doing now
2012-10-10 28417, 2012
ianmcorvidae
no worries, I was just at the store :)
2012-10-10 28447, 2012
ianmcorvidae
but yeah, those indexes aren't super necessary, they prevent some situations that we don't want to happen, but on a mirror server they won't anyway, since you aren't editing things there :)
2012-10-10 28456, 2012
bloopletech
ok sure
2012-10-10 28433, 2012
bloopletech
thanks for the help, I'm retrying now
2012-10-10 28403, 2012
kepstin-laptop joined the channel
2012-10-10 28430, 2012
luks
nikki: pong
2012-10-10 28427, 2012
bloopletech joined the channel
2012-10-10 28455, 2012
djce joined the channel
2012-10-10 28404, 2012
ijabz joined the channel
2012-10-10 28452, 2012
adhawkins joined the channel
2012-10-10 28459, 2012
nikki
luks: I was wondering if you knew whether gettext plurals should be used when the string doesn't actually contain numbers. I could have sworn I read somewhere that they shouldn't, but then couldn't find anything to support that
2012-10-10 28456, 2012
nikki
(the thing in question being "type" or "types" on the cover art tab, depending on the number of types displayed for the cover art)
2012-10-10 28446, 2012
dsteiner joined the channel
2012-10-10 28454, 2012
warp
hello!
2012-10-10 28434, 2012
ijabz joined the channel
2012-10-10 28447, 2012
luks
nikki: no idea, but I see nothing technically wrong with it
2012-10-10 28403, 2012
luks
I can't think of a language that would actually depend on the number in there
2012-10-10 28435, 2012
nikki
we did try using plural forms there and ran into problems with translations being wrongly merged >_<
2012-10-10 28459, 2012
nikki
since the singular also exists without a plural in other places
2012-10-10 28423, 2012
luks
maybe it just needs a context?
2012-10-10 28459, 2012
nikki
no idea. no matter how I look at it, it's stupid to encounter "type" on its own and "type" with the plural "types" and then think the best way to resolve it is to to throw away the plural :P
2012-10-10 28425, 2012
ianmcorvidae
gettext considers msgid "blah" + msgid_plural "blahs" to be duplicate with just plain msgid "blah"
2012-10-10 28457, 2012
ijabz joined the channel
2012-10-10 28407, 2012
luks
yeah, that's why I'd add context to it
2012-10-10 28445, 2012
ianmcorvidae
well, plural + context is usually wrong (generally it's encouraged to just put it into the string, with the plural -- since typically you're interpolating something)
2012-10-10 28455, 2012
ianmcorvidae
I could add it to all the singular ones, but that's also annoying :P
2012-10-10 28459, 2012
luks
how are the strings displayed?
2012-10-10 28409, 2012
ianmcorvidae
er, not sure what you mean
2012-10-10 28410, 2012
luks
is it really just singular/plural without any number anywhere?
2012-10-10 28415, 2012
nikki
type: front
2012-10-10 28417, 2012
nikki
types: back, spine
2012-10-10 28418, 2012
ianmcorvidae
yeah
2012-10-10 28428, 2012
luks
ah, then I'd translate them separately
2012-10-10 28429, 2012
ianmcorvidae
it's <string:> <list of types>
2012-10-10 28452, 2012
luks
in slovak we have 3 plural form, but I would use the third one in this case
2012-10-10 28400, 2012
ianmcorvidae
yeah, I just am wary of any time I do "if whatever.size = 1, l(whatever), else l(whatever-plural)"
2012-10-10 28402, 2012
luks
it would be wrong
2012-10-10 28412, 2012
luks
but that's the right thing to do here
2012-10-10 28425, 2012
luks
er, I meant "I wouldn't use"
2012-10-10 28434, 2012
nikki was gonna say :)
2012-10-10 28452, 2012
ianmcorvidae
yeah
2012-10-10 28453, 2012
luks
"1 typ, 2 typy, 5 typov"
2012-10-10 28405, 2012
luks
but "typov: a, b, c, d, e, f" is totally wrong in slovak
2012-10-10 28420, 2012
nikki is not imagining things, woo :D
2012-10-10 28448, 2012
ianmcorvidae
in short, the pluralization rules are different in this case so don't worry about the ln() stuff?
2012-10-10 28405, 2012
ianmcorvidae
(just making sure I'm understanding what you're saying)
2012-10-10 28418, 2012
luks
yes, because you don't really know the total count
2012-10-10 28432, 2012
luks
(at the language level)
2012-10-10 28408, 2012
ianmcorvidae
yeah
2012-10-10 28424, 2012
ianmcorvidae
despite the fact the *code* can pass in an exact count, the language still needs to be written as though it's non-specific
2012-10-10 28431, 2012
ianmcorvidae
makes sense
2012-10-10 28443, 2012
luks
the hard thing to do is how to deal with 0 :)
2012-10-10 28445, 2012
djce joined the channel
2012-10-10 28452, 2012
ianmcorvidae
heh
2012-10-10 28458, 2012
nikki
which I suppose explains why I don't find "types: foo" that odd, while "1 types" is just wrong
2012-10-10 28415, 2012
ianmcorvidae
well, germanic plurals is 1 vs. everything else (0 included)
2012-10-10 28425, 2012
nikki
but not french, afaik
2012-10-10 28426, 2012
luks
I'd probably always use plural there
2012-10-10 28431, 2012
luks
just to avoid the (none) case
2012-10-10 28441, 2012
ianmcorvidae
which is I guess how we'd have to do it sans-ln()
2012-10-10 28451, 2012
adhawkins joined the channel
2012-10-10 28424, 2012
djce joined the channel
2012-10-10 28445, 2012
ianmcorvidae
warp: so, are you planning to ship your schema change stuff to the branch, or has that bug now gotten postponed 6 months?
2012-10-10 28404, 2012
warp
ianmcorvidae: MBS-4627? I was waiting for an OK from ocharles
ocharles, warp: I see ocharles' comments are all code stuff -- can we get the schema change scripts themselves into the branch so that part can get tested?
2012-10-10 28437, 2012
ianmcorvidae
(like we have with several other tickets)
2012-10-10 28433, 2012
ocharles
yea, the schema is fine
2012-10-10 28412, 2012
ianmcorvidae
k, that should probably get into the schema change branch ASAP then
2012-10-10 28419, 2012
ianmcorvidae
what with the five days until we supposedly release this :)
2012-10-10 28424, 2012
ianmcorvidae
I, however, am going to bed so I don't miss the meeting at 19:00 by sleeping through it -- tell me if I should do that merge when I wake up
block opening braces for subs I feel like I've more often seen on the next line
2012-10-10 28420, 2012
ianmcorvidae
rest looks reasonable to me
2012-10-10 28435, 2012
ianmcorvidae goes to bed now, really XD
2012-10-10 28404, 2012
nikki thought the block opening stuff was as inconsistent as the rest of our whitespace
2012-10-10 28430, 2012
nikki
but it's not like I look at it as often as you guys :P
2012-10-10 28445, 2012
nikki
ocharles: when exactly should whitespace be used before ()? it's not that I don't agree (totally throws me every time I see a space I wasn't expecting), but given that you write "if ( ..." later on, it's not *always* the case :)
2012-10-10 28400, 2012
warp returns from lunch.
2012-10-10 28411, 2012
uk
s/pareth/parenth/, s/parenthesis/parentheses/g
2012-10-10 28401, 2012
nikki
heh, I was just thinking that
2012-10-10 28407, 2012
nikki also wonders if alphabetical is case-sensitive or not
2012-10-10 28452, 2012
nikki
I think I might have too much time on my hands, 'cause I've wondered the same thing (both about spacing and ordering of imports) with my own stuff :P
2012-10-10 28416, 2012
ocharles
nikki: I was meant to qualify that with 'except for flow control' where we do use a leading space
2012-10-10 28453, 2012
warp
ocharles: the leading space around parens is a habit I should unlearn.
2012-10-10 28406, 2012
ocharles
i'm afraid so
2012-10-10 28416, 2012
warp
ocharles: but I disagree with the braces, I much prefer matching braces in the same column.
2012-10-10 28424, 2012
ocharles
hm?
2012-10-10 28425, 2012
ocharles
oh
2012-10-10 28429, 2012
ocharles
so on a separate line
2012-10-10 28447, 2012
warp
yes
2012-10-10 28456, 2012
ocharles
it just leads to less code on the screen, and more I have to hold in my head
2012-10-10 28418, 2012
warp
it leads to clearly visible start/end points for blocks :)
2012-10-10 28441, 2012
ocharles
so does indentation
2012-10-10 28441, 2012
warp
ocharles: anyway, no objections to the other bits.
2012-10-10 28456, 2012
ocharles
i'll need to flesh it out a bit more and open it for general discussion
2012-10-10 28450, 2012
nikki prefers them on the same line too
2012-10-10 28442, 2012
warp
ocharles: are you going to write a syntax linter go check these? :P
2012-10-10 28447, 2012
warp
s/go/to/
2012-10-10 28427, 2012
ocharles
i wish i could
2012-10-10 28417, 2012
CatBuss joined the channel
2012-10-10 28436, 2012
uk
GNU Indent is specific to C, I think.
2012-10-10 28438, 2012
warp
Perl::Critic is the canonical solution for perl I guess?
2012-10-10 28402, 2012
ocharles
that's more for actual code linting
2012-10-10 28409, 2012
warp
Perl::Critic::Policy::CodeLayout has checks for a couple of things you've listed in the linked gist.
2012-10-10 28434, 2012
warp
well, one thing :)
2012-10-10 28408, 2012
ocharles
we have t/author/no-tabs.t for that too
2012-10-10 28429, 2012
ijabz joined the channel
2012-10-10 28425, 2012
reosarevok joined the channel
2012-10-10 28421, 2012
kurtjx joined the channel
2012-10-10 28413, 2012
njh
reosarevok: do you think it would be a good idea for me to submit the BBC artwork for the releases we have reviewed?
2012-10-10 28436, 2012
reosarevok
Can't hurt, I think
2012-10-10 28449, 2012
reosarevok
(make sure they have no existing front image, though)
