[01:00] <Ubugtu> New bug: #68292 in launchpad "+newaccount workflow might oops while redirecting to the newly created user page" [Undecided,Unconfirmed]  http://launchpad.net/bugs/68292
[01:43] <BaKKaR> a question please.
[01:43] <BaKKaR> why the translators don't get crdited like developers ?
[01:43] <BaKKaR> i mean don't you - anyone - think it's not fair ?
[01:48] <kiko> BaKKaR, what do you mean?
[01:50] <BaKKaR> kiko, i mean look at the programs ( about ) or (credits) you would see the developers and few translaters. Although much more translaters were involved in that project.
[01:51] <kiko> BaKKaR, that's a problem with that specific application; IIRC firefox for instance includes full translator info
[02:59] <Burgwork> anybody alive?
[03:00] <Nafallo> .:-)
[03:00] <Burgwork> no, an LP dev
[03:00] <Nafallo> that was the smiley was there for :-)
[03:07] <lifeless> morning
[03:08] <Burgwork> lifeless: morning
[03:08] <Burgwork> how goes bzr?
[03:09] <ajmitch> hey lifeless 
[03:09] <lifeless> Burgwork: sweet mate
[03:09] <lifeless> Burgwork: 0.12 is kickarse, and 0.13 will rock some socks
[03:10] <ajmitch> great to hear that
[03:10] <Burgwork> speed-wise?
[03:10] <lifeless> jamesh: can one do declarittive security at the content object level ?
[03:10] <lifeless> jamesh: if so,w heres the doc I should read to do that
[03:26] <lifeless> jamesh: I'd like to reacquaint myself with the current lp testing and best practicec, if you have some time today
[03:30] <jamesh> lifeless: there is some info in lib/canonical/launchpad/doc/security.txt, but it seems to be a bit out of date
[03:31] <jamesh> lifeless: but the basic system is to declare the permissions needed to read or write certain attributes on a class in the ZCML (either by attribute name or via a schema)
[03:32] <jamesh> lifeless: then when you try to access an attribute on a security wrapped object, it checks what permission is required and looks up a security adapter by (context, permission_name)
[03:34] <jamesh> lifeless: the security policy is in lib/canonical/launchpad/webapp/authorization.py
[03:34] <jamesh> the security adapters it looks up are in lib/canonical/launchpad/security.py
[05:00] <mpt> Gooooooooooooooooooood afternoon Launchpadders!
[05:07] <ajmitch> helloooo mpt 
[05:09] <Nafallo> morning mpt :-)
[05:12] <ajmitch> morning?
[05:15] <jamesh> still morning here
[05:25] <Nafallo> ajmitch: just became morning :-)
[05:25] <Nafallo> 5:09 when I typed that ;-)
[06:06] <spiv> jamesh: https://launchpad.net/products/tickcount/trunk has no bzr branch?
[06:23] <jamesh> spiv: does now
[08:06] <Ubugtu> New bug: #68320 in launchpad "Python trunk reporting SVN repository as a file:// URL" [Undecided,Unconfirmed]  http://launchpad.net/bugs/68320
[08:22] <mpt> jamesh or spiv, pagetests/standalone/xx-home-page-features.txt and pagetests/standalone/xx-root-page.txt are doing the same job. Can I get an rs= for merging them?
[08:25] <jamesh> mpt: sure.
[08:25] <mpt> thanks
[08:25] <mpt> jamesh, for the pagetest restructuring, can the directories have more than one level?
[08:25] <jamesh> mpt: not at present.
[08:26] <mpt> I'm thinking of bugtracking/tags/, bugtracking/searches/, bugtracking/watches/, etc
[08:26] <jamesh> mpt: it would be possible to implement, but hasn't been done so far
[08:26] <mpt> ok, I'll report a bug about it
[08:27] <jamesh> you could do bugtracking-tags/ bugtracking-searches/ bugtracking-watches/ though
[08:31] <mpt> true
[08:40] <Ubugtu> New bug: #68323 in launchpad "Should be possible to have pagetest directories two levels deep" [Undecided,Unconfirmed]  http://launchpad.net/bugs/68323
[08:43] <carlos> morning!
[08:57] <jamesh> carlos: pygettextpo fix is in rocketfuel
[09:02] <carlos> jamesh: yeah, I saw that email. Thank you very much!!
[09:02] <carlos> also the building changes you did would make for me quite easy to release it as Mark told us sometime ago already...
[09:02] <carlos> SteveA: ping
[09:03] <jamesh> carlos: so, things left to get launchpad working nicely on edgy include: use the local twistd/trial executables rather than /usr/bin/twistd2.4 (which doesn't exist anymore), a few test fixes for pygpgme and get the launchpad-dependencies package ported over
[09:03] <carlos> I did some symbolic links to twistd in my computer to workaround that :-P
[09:04] <carlos> also, I get a bunch of test errors related with bazaar integration
[09:04] <spiv> carlos: file bugs next time, slacker :P
[09:04] <jamesh> that's cheating
[09:04] <carlos> spiv: I sent an email long time ago to launchpad@
[09:04] <carlos> when I moved to Edgy
[09:04] <carlos> I know it's not the same
[09:04] <carlos> but I reported it!
[09:05] <spiv> Heh.  I don't remember that email :)
[09:05] <carlos> you see... you don't pay attention to what I say!
[09:05] <jamesh> I don't remember an email about the twistd/twistd2.4 issue
[09:05] <spiv> It's true!
[09:05] <carlos> :-P
[09:05] <spiv> Sorry, did you just say something?  I wasn't paying attention.
[09:06] <jamesh> spiv: maybe he was speaking in mexican
[09:06] <carlos> jamesh: Message-Id: <1156504629.5152.3.camel@aragorn>
[09:06] <carlos> spiv: grrr 
[09:06] <carlos> :-P
[09:07] <spiv> jamesh: haha
[09:07] <carlos> 25th August, Subject: Launchpad changes to run on Edgy
[09:08] <jamesh> carlos: so you did.  Rather than adding symlinks or changing things to call /usr/bin/twistd instead, the correct fix is probably to call the local twistd script, btw
[09:09] <carlos> yeah, I saw that email
[09:09] <jamesh> carlos: at the moment we are relying on the fact that the system installed twistd executable is compatible with the in-tree twisted
[09:09] <carlos> do we still need local twistd once migrated to Edgy?
[09:09] <jamesh> which we shouldn't need to do
[09:09] <spiv> carlos: I haven't checked, it's quite possible edgy's Twisted is new enough.
[09:10] <spiv> carlos: but I expect we'll probably need the bleeding edge of Twisted again soon, so it's simplest to leave it there.
[09:10] <jamesh> spiv: I'm sure you'll find something that'll need a newer version at some point though ...
[09:10] <carlos> ok
[09:11] <jamesh> overall, edgy seems to run pretty well
[09:11] <spiv> jamesh: yeah, e.g. there's WSGI fixes happening in SVN at the moment, and the SFTP server keeps bumping into little issues with the twisted.vfs package...
[09:11] <jamesh> it had new enough bluetooth stuff to port the maemo obex VFS method over, which is nice ...
[09:14] <carlos> jamesh: does soyuz-upload.txt work for you?
[09:14] <jamesh> carlos: I haven't tried it today, but I haven't seen any errors related to it recently
[09:14] <carlos> It fails for me in Edgy because it gets an extra 'Section: net' line in one of the outputs
[09:15] <carlos> and last time I checked... Rosetta was not adding such tags to .deb archives so it shouldn't be my fault ;-)
[09:16] <jamesh> just testing it now
[09:16] <jamesh> yep.  It fails there for me too
[09:18] <jamesh> report a bug and send an email to the list
[09:18] <carlos> Ok, then I will get the full list of tests that fail for me and mail the list (yesterday I got 165 test failures, some were my fault, but I don't think more than 20)
[09:39] <SteveA> carlos: morning
[09:39] <carlos> SteveA: good morning
[09:48] <SteveA> mpt: ping
[09:48] <mpt> SteveA, pong
[09:48] <SteveA> anything for me to update on brilliant?
[09:51] <mpt> SteveA, no, but I'll merge pagetest fixes in an hour or two, and shiny icons
[09:51] <SteveA> ok, cool
[11:31] <indu> kiko:  r u there kiko
[11:32] <SteveA> indu: try in about 3 hrs
[11:32] <indu> SteveA: ok thankyou
[12:39] <carlos> BjornT: any chance to get fixed our infrastructure to be able to test textareas without extra '\n' chars? (bug #47511)
[12:39] <Ubugtu> Malone bug 47511 in launchpad "pagetests add ghost new lines to textareas" [Medium,Confirmed]  http://launchpad.net/bugs/47511
[12:40] <carlos> is a bit annoying to add the extra '\n' char to the output of the tests...
[12:42] <BjornT> carlos: not before 1.0, i'm quite busy with malone now since brad left.
[12:42] <carlos> Ok
[12:43] <carlos> I understand what you mean, don't worry
[12:43] <BjornT> carlos: i tried pulling in the upstream fix once, but it caused quite a lot of test failures, and i didn't have time to take a closer look at it.
[12:43] <carlos> s/mean/feel/ 
[12:43] <carlos> :-P
[12:43] <carlos> I know, just asking for an update after I had to change a new test to pass
[12:44] <carlos> now, every translation field in Rosetta is a textarea so this hit us a lot
[12:47] <carlos> danilos: ping
[01:09] <mpt> SteveA, in about 20 minutes the icons will have finished pushing to 2006-10-ui-one-zero
[01:10] <mpt> Meanwhile I have conflicts in 2006-08-ui that I'll look at after dinner
[01:10] <carlos> danilos: no, I don't see them
[01:10] <danilos> carlos: now?
[01:10] <carlos> yes
[01:13] <SteveA> mpt: thanks
[01:34] <mpt> carlos, "every translation field in Rosetta is a textarea"? Why is that?
[01:35] <carlos> because the assumption that a single line msgid should have a single line translation is wrong
[01:35] <carlos> mpt: a good example of that is the problem with translation credits in GNOME
[01:36] <SteveA> maybe start with a single line for a single line, and have a JS expander for it to make it into a text area?
[01:36] <SteveA> spiv: ping
[01:36] <carlos> SteveA: what's wrong to have textareas for all messages?
[01:37] <mpt> carlos, translation credits shouldn't be entered manually anyway
[01:38] <mpt> Do you have any other examples? :-)
[01:40] <carlos> mpt: I agree, but today, we are not in a position to solve that and prevent people to fix those messages now is giving us bad press
[01:41] <carlos> mpt: well, I don't know all details for all applications but the thing is, gettext allows that, why shouldn't we allow it?
[01:42] <mpt> because it will take up more space
[01:43] <jordi> can't you just special-case that string?
[01:43] <carlos> jordi: will we do that for any other exceptions we find?
[01:44] <carlos> mpt: do you really think that's a good 'excuse' ?  I think that's not so user friendly...
[01:44] <mpt> carlos, you think what's not so user friendly?
[01:45] <mpt> I think Rosetta requires too much scrolling
[01:45] <carlos> well, not being able to add more than one line when needed is not being user friendly IMHO
[01:45] <carlos> I think is better move to 5 messages per page than leave 10 and limit what the user can do
[01:45] <carlos> at least, I prefer that
[01:46] <mpt> My hypothesis is that that never happens, except in the case of translator credits
[01:46] <spiv> Launchpad meeting in 14 minutes.
[01:46] <spiv> Take a break now if you need one.
[01:47] <carlos> well, let me show you another problem (which also has workarounds, sure, but I still think is much more simple using textareas) looking for the bug report...
[01:48] <jamesh> spiv: are you being Steve this week?
[01:49] <jordi> carlos: well you know the situation a lot better than me, but I can't think of other real life examples I have encountered while translating
[01:50] <jordi> ie, it'd be special-casing translator_credits, translator-credits, the kde way and little more
[01:50] <jordi> and if the app uses a non-"standard" string, they either lose or we file bugs
[01:50] <carlos> jordi, mpt: https://launchpad.net/products/rosetta/+bug/38984
[01:50] <Ubugtu> Malone bug 38984 in rosetta "Boxes for translated strings is "hiding" content, wich makes wrongly translated strings." [Medium,In progress]  
[01:51] <carlos> that's caused by our lack of copy buttons, I know. I already have that implemented and this situation should not happen anymore, but...
[01:51] <carlos> What should we do when I get a .po file imported with more than one line?
[01:51] <spiv> jamesh: I am.
[01:51] <carlos> that will not be fixable in Rosetta at all
[01:52] <spiv> jamesh: Not that Steve has been Steve for quite a while now ;)
[01:52] <carlos> in fact, people will not see that in the UI
[01:52] <carlos> you could say: an easy workaround, if that's a multiline translation, show a textarea
[01:53] <carlos> ok, I do that, but when someone changes that to a single line translation, do we mute to a single line entry?
[01:53] <carlos> isn't that even more confusing?
[01:53] <carlos> what happens if that's not a wrong translation? how would they revert?
[01:54] <carlos> I'm not saying that I love having textareas for all entries, I'm just saying that I think is the less hurting solution IMHO
[01:55] <mpt> carlos, the only example given in that bug is of translator credits!
[01:56] <carlos> mpt: please, read again the main comment
[01:57] <mpt> I have
[01:57] <carlos> that's not translation credits at all
[01:58] <danilos> mpt, carlos: if we're looking for more examples, looking through help messages in things like GNU findutils, coreutils will probably reveal many more results
[01:58] <danilos> (i.e. where there are fixed-width alignment issues)
[01:59] <carlos> danilos: so you are aware of msgid that are single line and require multi line translations?
[02:00] <danilos> carlos: not concretely, but I can look for some after the meeting
[02:00] <carlos> danilos: yes, please
[02:00] <danilos> it all depends on the language as well
[02:00] <spiv> MEETING TIME
[02:00] <spiv> Who's here?
[02:00] <flacoste> me
[02:00] <matsubara> me
[02:00] <jamesh> me
[02:00] <cprov> me
[02:01] <BjornT> me
[02:01] <salgado> me
[02:01] <malcc> me
[02:01] <carlos> me
[02:01] <jordi> me
[02:01] <mpt> me
[02:02] <danilos> me
[02:02] <salgado> SteveA may be a few minutes late and sends his apologies
[02:02] <spiv> salgado: thanks
[02:02] <spiv> kiko-zzz: ?
[02:02] <spiv> _thumper_ and stub have registered their apologies.
[02:02] <kiko-zzz> me
[02:03] <salgado> that's kiko
[02:03] <spiv> Agenda
[02:03] <spiv> [02:03] <spiv>   * Roll call
[02:03] <spiv>   * Agenda
[02:03] <spiv>   * Next meeting
[02:03] <spiv>   * Activity reports
[02:03] <spiv>   * Actions from last meeting
[02:03] <spiv>   * Oops report (Matsubara)
[02:03] <spiv>   * Bug report report (mpt)
[02:03] <spiv>   * Production and staging (Stuart)
[02:03] <spiv>   * Launchpad 1.0 status reports
[02:03] <spiv>   * Sysadmin requests
[02:03] <spiv>   ----
[02:03] <spiv>   * devpad apt repository (spiv)
[02:03] <spiv>   * Use bzr+ssh:// (bzr smart server) on devpad (spiv)
[02:03] <spiv>   * Keeping cleanups/whitespace changes separate from other other work (jamesh)
[02:03] <spiv>   * (other items)
[02:03] <spiv>   ----
[02:03] <spiv>   * Keep, Bag, Change
[02:03] <spiv>   * Three sentences
[02:03] <spiv> If you have anything else to add, send me a /msg
[02:03] <spiv>   * Next meeting
[02:03] <spiv> Same time next week?
[02:04] <kiko> sure, why not.
[02:04] <spiv>   * Activity reports
[02:04] <kiko> not up to date
[02:04] <salgado> up to date
[02:04] <BjornT> up to date
[02:04] <carlos> up to date
[02:04] <cprov> up to date
[02:04] <matsubara> not up to date
[02:04] <spiv> Who's $adjective_good and who's $adjective_bad?
[02:04] <flacoste> up to date
[02:04] <mpt> up to date
[02:05] <malcc> up to date
[02:05] <spiv> I'm not up to date :(
[02:05] <danilos> still behind
[02:05] <jamesh> behind
[02:05] <danilos> (need $adjective_worst for me, I guess)
[02:05] <jordi> not up to date, I need to do a pile :|
[02:05] <spiv> jordi: start sending current ones from today, even if you haven't caught up old ones yet
[02:06] <jordi> spiv: as I send weekly, it's quite easy to catch up entirely
[02:06] <spiv> jordi: ok.  So long as catching up doesn't block sending current ones.
[02:06] <jordi> yeah
[02:07] <spiv>   * Actions from last meeting
[02:07] <spiv> * spiv to make testcase for malcc's issue (eq, ne, and hash for database objects), and write up the results
[02:07] <spiv> I can report some progress...
[02:07] <spiv> I have a test case and a simple fix for the __eq__ problem in hand.  I'll mail the list with details shortly.
[02:08] <SteveA> hi
[02:08] <spiv> ACTION: spiv to finish the damn __eq__ for database objects action item
[02:08] <spiv>   * Oops report (Matsubara)
[02:08] <matsubara> Today's oops report is about bugs 68292, 68203, 61915, 61910
[02:08] <Ubugtu> Malone bug 68292 in launchpad "+newaccount workflow might oops while redirecting to the newly created user page" [Undecided,Unconfirmed]  http://launchpad.net/bugs/68292
[02:08] <Ubugtu> Malone bug 68203 in malone "Bug description with only whitespaces should return a nice error message instead of crashing" [Medium,Confirmed]  http://launchpad.net/bugs/68203
[02:08] <Ubugtu> Bug 61915 on http://launchpad.net/bugs/61915 is private
[02:09] <matsubara> Bug 68292 is a strange one. Somehow the logged in user (returned by Launchbag) was None and after the account creation workflow, the view tries to redirect to that user's home page and crashes. Salgado, could you look at that one?
[02:09] <matsubara> BjornT, can you take bug 68203?
[02:09] <matsubara> Bugs 61915 and 61910 are related. I proposed a fix (for 61915) here: https://lists.ubuntu.com/mailman/private/launchpad/2006-September/011616.html. Jamesh, could you handle that one?
[02:09] <Ubugtu> Bug 61910 on http://launchpad.net/bugs/61910 is private
[02:09] <BjornT> matsubara: sure
[02:09] <matsubara> thanks BjornT 
[02:10] <salgado> matsubara, yeah, I'll take that one. (a fix for that may already be available to my direct-person-creation branch. I need to check that)
[02:10] <matsubara> great, thanks salgado 
[02:10] <jamesh> matsubara: okay.  (was just reading the bug content)
[02:11] <matsubara> jamesh: all right. thanks
[02:11] <matsubara> spiv: I'm done here. thank you.
[02:11] <spiv> Ta.
[02:11] <spiv>   * Bug report report (mpt)
[02:11] <mpt> There are 15 known Critical bugs in Launchpad without released fixes. The oldest ten are:
[02:11] <mpt>  * Bug #929 (Long words (such as URLs) overflow columns), Critical, In Progress, jamesh
[02:11] <mpt> jamesh, are you still waiting for review?
[02:11] <Ubugtu> Malone bug 929 in launchpad "Long words (such as URLs) overflow columns" [Critical,In progress]  http://launchpad.net/bugs/929
[02:11] <jamesh> mpt: yes.
[02:11] <mpt>  * Bug #2322 (Truncated plural forms), Critical, Fix Committed, carlos
[02:11] <mpt> carlos, since that's Critical, have you requested a cherrypick?
[02:11] <Ubugtu> Malone bug 2322 in rosetta "Truncated plural forms" [Critical,Fix committed]  http://launchpad.net/bugs/2322
[02:12] <mpt> jamesh, how long have you been waiting?
[02:12] <carlos> mpt: no, thanks for remind me
[02:12] <BjornT> mpt, jamesh: i plan to review the fix for bug 929 today
[02:12] <mpt> ok, thanks BjornT 
[02:12] <jamesh> mpt: 2 days -- not long
[02:12] <mpt> all right
[02:12] <mpt>  * Bug #2497 (/people/*/+translations times out for prolific translators), Critical, In Progress, kiko
[02:12] <mpt>  * Bug #30602 (Timeout errors in +translate), Critical, In Progress, kiko
[02:12] <mpt> kiko, what's new?
[02:12] <Ubugtu> Malone bug 2497 in rosetta "/people/*/+translations times out for prolific translators" [Critical,In progress]  http://launchpad.net/bugs/2497
[02:12] <Ubugtu> Malone bug 30602 in rosetta "Timeout errors in +translate" [Critical,In progress]  http://launchpad.net/bugs/30602
[02:12] <mpt>  * Bug #4594 (Shouldn't be able to add duplicate bug watches), Critical, In Progress, BjornT
[02:12] <mpt> BjornT, last week I think you said you were waiting for review. Are you still waiting?
[02:12] <Ubugtu> Malone bug 4594 in malone "Shouldn't be able to add duplicate bug watches" [Critical,In progress]  http://launchpad.net/bugs/4594
[02:12] <kiko> mpt, the former is blocked on stub
[02:13] <kiko> because he didn't implement the merging code for POFileTranslator
[02:13] <kiko> and even if I did, it would still need to be reviewed by him
[02:13] <mpt> kiko, should you reassign it temporarily to him, then?
[02:13] <kiko> yeah.
[02:13] <mpt> ok
[02:13] <mpt>  * Bug #44214 (We need to add code to prevent POFiles being in the same path), Critical, In Progress, carlos
[02:13] <mpt>  * Bug #44808 (Some translation templates in dapper don't contain any items), Critical, Confirmed, carlos
[02:13] <mpt>  * Bug #46982 (Rosetta does not accept correct KDE plural forms when there are more than 2), Critical, Confirmed, carlos
[02:13] <mpt> carlos, are you happy with your progress, or should danilos get one of those?
[02:13] <Ubugtu> Malone bug 44214 in rosetta "We need to add code to prevent POFiles being in the same path" [Critical,In progress]  http://launchpad.net/bugs/44214
[02:13] <Ubugtu> Malone bug 44808 in rosetta "Some translation templates in dapper don't contain any items" [Critical,Confirmed]  http://launchpad.net/bugs/44808
[02:13] <Ubugtu> Malone bug 46982 in rosetta "Rosetta does not accept correct KDE plural forms when there are more than 2" [Critical,Confirmed]  http://launchpad.net/bugs/46982
[02:14] <carlos> bug #44214 is in the review queue since yesterday, when I found the solution for the failing tests (broken sample data)
[02:14] <mpt> BjornT?
[02:14] <BjornT> mpt: it's been reviewed, i just need to address the review comments and land it.
[02:14] <danilos> mpt: with the work I am doing, I can surely help with 46982, but it might be helpful to have carlos get to know the new code as well
[02:14] <mpt> BjornT, cool, thanks
[02:14] <mpt>  * Bug #48860 ("Also notified" makes difficult to unsubscribe), Critical, Fix Committed, bradb
[02:14] <mpt> BjornT, has the fix for that been rolled out?
[02:14] <Ubugtu> Malone bug 48860 in malone ""Also notified" makes difficult to unsubscribe" [Critical,Fix committed]  http://launchpad.net/bugs/48860
[02:14] <carlos> bug #46982 is blocked on Firefox implementation being reviewed and merged (danilo is pushing that branch right now to be reviewed)
[02:14] <Ubugtu> Malone bug 46982 in rosetta "Rosetta does not accept correct KDE plural forms when there are more than 2" [Critical,Confirmed]  http://launchpad.net/bugs/46982
[02:14] <BjornT> mpt: i have to check, just a second...
[02:14] <mpt> carlos, ok, reassign it if you think that's appropriate
[02:15] <carlos> ok
[02:15] <mpt>  * Bug #48948 (dapper indices files still being regenerated but shouldn't be), Critical, Confirmed, malcc
[02:15] <mpt> malcc, will you get to that this week?
[02:15] <Ubugtu> Malone bug 48948 in soyuz "dapper indices files still being regenerated but shouldn't be" [Critical,Confirmed]  http://launchpad.net/bugs/48948
[02:15] <malcc> mpt: Progress on 48948 today, Colin took a look at the dsync source for me and thinks it may already do what we want, so I need to try out some command-line options.
[02:15] <malcc> mpt: Should get a fix real soon now
[02:15] <mpt> good good good
[02:15] <mpt> thank you everyone
[02:15] <mpt> that's all spiv
[02:16] <spiv> Thanks mpt.
[02:16] <spiv>   * Production and staging (Stuart)
[02:16] <spiv> stub isn't here.
[02:16] <spiv> Any comments? :)
[02:16] <spiv> Ok,
[02:16] <spiv>   * Launchpad 1.0 status reports
[02:16] <BjornT> mpt: yes, the fix for 48860 has been rolled out.
[02:16] <salgado> Question Tracker 1.0
[02:16] <salgado> ---------------------------------
[02:16] <salgado> - SupportTrackerWorklow: 2nd round of review, should land shortly.
[02:16] <salgado> - SupportTrackerViews: landed. Missing the 'Needs Attention' report.
[02:16] <salgado> - SupportTrackerHelp: 75% completed. Missing Highlights, help on native language support and polish.
[02:16] <salgado> - LocalizedSupportRequests: started, good progress.
[02:16] <salgado> Random Things 1.0
[02:16] <salgado> -------------------------------
[02:17] <salgado> - DirectPersonRegistration: in review.
[02:17] <cprov> = Soyuz-1.0 Report =
[02:17] <cprov>  * PPA: poppy (cprov) & nascentupload (malcc) parts started, derived from Archi\veRework
[02:17] <cprov>  * Archive Rework: pending review (malcc)
[02:17] <cprov>  * NoMoreAptFtparchive: pending review (cprov)
[02:17] <cprov>  * Code quality: nascentupload first refactoring (kiko, under development) and \DiskPoll refatoring (malcc, pending review)
[02:17] <BjornT> Malone 1.0:
[02:17] <cprov>  * General Fixing: nothing new.
[02:17] <BjornT> upstream-forwarding-workflow: code has been reviewed, kiko to take a look at the ui.
[02:17] <BjornT> series-and-distrorelease-mgmt: taken over by BjornT. first branch still in review queue, another branch added to the queue, and yet another branch coming up soon, which should take care of the remaining issues.
[02:17] <BjornT> guided-filebug-form: pending code review. BjornT will make sure the implementation branch gets landed.
[02:17] <BjornT> removing-duplicate-comments: No news, still implemented. 
[02:17] <danilos> Rosetta 1.0 weekly report:
[02:17] <danilos> - opening edgy for translation: DONE
[02:17] <danilos> - firefox import/export: done, to put up for review (connection issues this week)
[02:17] <danilos> - oo import/export: in progress
[02:17] <danilos> - translation review: fixing tests, will be ready for review today
[02:17] <BjornT> malone-essential-docs: No news, still not started.
[02:17] <danilos>   (with the same UI for reviewers and translators, once that's merged,
[02:17] <BjornT> simple-bug-keywords: No news, still implemented.
[02:17] <danilos>   the review specific UI will be implemented)
[02:17] <danilos> - essential docs: no progress this week
[02:17] <danilos> - search: not started
[02:17] <danilos> - checks not to upload wrong language PO file using "too many changes" check: not started
[02:17] <danilos> - ui fixes: mpt on those
[02:17] <danilos> - outstanding issues: none
[02:17] <mpt> What UI fixes?
[02:18] <mpt> Was that just referring to bug 46?
[02:18] <Ubugtu> Malone bug 46 in rosetta ""special symbols" when people copy-paste text from original to translation" [High,Fix released]  http://launchpad.net/bugs/46
[02:18] <carlos> mpt: sorry, I forgot to update that
[02:18] <carlos> danilos: ui fixes: DONE
[02:18] <mpt> ok :-)
[02:18] <danilos> mpt: be sure to fix that in your weekly meeting notes :P
[02:19] <kiko> danilos, what are "connection issues this week"?
[02:19] <mpt> all right, but I won't fix it for the past three weeks
[02:19] <mpt> :-)
[02:19] <carlos> mpt: that was moving the 'you are not an official translator' message to the end of the page
[02:19] <danilos> kiko: apparently, my ISP has misconfigured something on their side, and HTTP POSTs and scp/bzr push/rsync are all failing for me right now
[02:20] <danilos> I noticed that for HTTP POSTs yesterday (on bugzilla.gnome.org and wiki.canonical.com), and I was thinking it was only bzr push being slow for me
[02:20] <kiko> danilos, mail a tarball to somebody -- anything but block because of this.
[02:20] <kiko> you can use a bzr feature to send changsets
[02:20] <spiv> Connection issues can be sorted out outside of the meeting...
[02:20] <spiv>   * Sysadmin requests
[02:20] <kiko> I can't remember what it is
[02:20] <danilos> spiv: right
[02:21] <kiko> spiv, I want to make a point that that sort of issue has no place blocking 1.0 work.
[02:21] <danilos> kiko: (anyway, I am on it very seriously)
[02:21] <spiv> kiko: fair enough.
[02:21] <kiko> thanks.
[02:21] <SteveA> +1.  it's good to announce such blockage and get help with it.
[02:21] <SteveA> it's always okay to use the mailing list, or talk with me or kiko directly
[02:22] <spiv> Countdown for sysadmin issues...
[02:22] <spiv> 5
[02:22] <spiv> 4
[02:22] <danilos> SteveA, kiko: took notice, sorry for not going directly to you guys
[02:22] <spiv> 3
[02:22] <spiv> 2
[02:22] <spiv> 1
[02:22] <spiv> Awesome.
[02:22] <spiv> Two happy weeks in a row!
[02:22] <spiv>   * devpad apt repository (spiv)
[02:22] <spiv> There is an apt repository for launchpad dependencies at lpdebs.canonical.com.
[02:22] <spiv> Add "deb http://lpdebs.canonical.com/dapper/ ./" or "deb http://lpdebs.canonical.com/edgy/ ./" to your sources.list.
[02:23] <spiv> At the moment it has bzr and bzrtools 0.12rc1 debs, the plan is for it to be the home of the launchpad-dependencies package.
[02:23] <spiv> It's signed by Etienne's key.
[02:23] <spiv> This leads into the item...
[02:23] <spiv>   * Use bzr+ssh:// (bzr smart server) on devpad (spiv)
[02:23] <spiv> I sent a message to the list about using bzr 0.12rc1 and bzr+ssh:// to push branches to devpad.
[02:23] <spiv> flacoste has already tried it and sent some good feedback.
[02:23] <jamesh> it also has bzr-pqm debs
[02:23] <spiv> Please, everyone upgrade and test the new version of bzr, and this new feature.
[02:23] <spiv> Any comments?
[02:24] <danilos> hum, I think I only got bzr upgraded, not bzr-pqm
[02:24] <danilos> ah, right, I haven't install bzr-pqm using apt
[02:24] <danilos> (you might want to mention that in an announcement)
[02:25] <SteveA> spiv: we should get bzr 0.12 onto devpad and the pqm machine etc. soon
[02:25] <spiv> That would be good.
[02:25] <spiv> flacoste has reported that PQM doesn't cope with bzr+ssh:// urls, I'm not sure if that's just a config issue, a bzr version issue, or something deeper.
[02:25] <SteveA> spiv: is bzr ready for us to do that?  just ask the admins to install from lpdebs?
[02:26] <spiv> (I'll forward the PQM issue to lifeless)
[02:26] <SteveA> can we have a bug on it?
[02:26] <spiv> I believe so.
[02:26] <SteveA> there's a product for this kind of development infrastructure
[02:26] <spiv> Good point.
[02:26] <flacoste> spiv: lifeless was cc on my PQM feedback
[02:26] <SteveA> let's have a bug
[02:26] <SteveA> so we can track it
[02:26] <spiv> flacoste: ah, great, I didn't notice.  I'll just put it in a bug, then.
[02:26] <kiko> salgado, do you know how to write people merge code?
[02:26] <jamesh> flacoste: you can set public_branch or public_repository in your ~/.bazaar/locations.conf file to fix that
[02:27] <flacoste> jamesh: great, i'll do that
[02:27] <spiv>   * Keeping cleanups/whitespace changes separate from other other work (jamesh)
[02:27] <salgado> kiko, I've done it a few times, but I don't remember exactly how it's done
[02:27] <jamesh> When working on a branch, please keep actual development separate from code cleanup.  This makes it easier for reviewers to see the actual changes.
[02:27] <kiko> salgado, if you could help me here I'd super-appreciate it
[02:27] <jamesh> This also makes your branch less likely to create spurious conflicts with other people working on the same areas of code
[02:28] <SteveA> it is also important if that revision needs to be cherrypicked
[02:28] <jamesh> For the cases where cleanup is needed, it is better to do as a separate branch
[02:28] <jamesh> Code/formatting cleanup-only branches are also easier to review since the changes are more mechanical.
[02:28] <SteveA> if you do a clean-up, then do it, get it reviewed, and land it on mainline asap.  that way, you reduce conflicts with people who might have had the same idea
[02:28] <jamesh> does that sound sensible to everyone?
[02:29] <SteveA> jamesh: +1
[02:29] <kiko> jamesh, absolutely.
[02:29] <flacoste> it does
[02:29] <spiv> I like it.
[02:29] <BjornT> is there an easy and quick way of merging a specific revision of a branch?
[02:29] <SteveA> jamesh: would you write this up as launchpad coding policy on the wiki please?
[02:29] <jamesh> okay.  That's all for me.
[02:29] <jamesh> BjornT: "bzr merge -r REVNO BRANCH"
[02:29] <jamesh> SteveA: okay
[02:29] <SteveA> thanks jamesh 
[02:30] <spiv>   * propose bugwatch tag (matsubara)
[02:30] <matsubara> I'd like to propose the bugwatch tag. https://help.launchpad.net/TaggingLaunchpadBugs
[02:30] <matsubara> Please take a look and see what you think of the examples.
[02:30] <matsubara> Also, what about the sabdfl tag, will it be proposed forever?
[02:30] <SteveA> Action: jamesh to write up on the wiki about separating clean-up branches from features
[02:30] <BjornT> jamesh: i was thinking of merging to RF
[02:30] <SteveA> BjornT: you'd need to branch at that revision
[02:30] <jamesh> BjornT: ah.  I don't think PQM has regained this feature since we switched from arch
[02:30] <spiv> BjornT: put the change on a new branch and merge that.
[02:31] <SteveA> and then merge that branch
[02:31] <jamesh> BjornT: creating a new branch and merging from that would work
[02:31] <BjornT> i was looking for a quicker way than creating a new branch, though.
[02:31] <kiko> matsubara, I think the bugwatch tag is a great idea. I don't think sabdfl is important and should be dropped.
[02:32] <spiv> BjornT: discuss on the list, perhaps?  lifeless may have some insight.
[02:32] <jamesh> BjornT: creating a new branch in your repo on devpad should take about 5 seconds
[02:32] <SteveA> matsubara: so... 
[02:32] <matsubara> kiko: ok, thanks.
[02:33] <SteveA> I like the proposed bugwatch tag.  Do we have anything else that gets "watched" in the same way?
[02:33] <kiko> (MHO)
[02:33] <kiko> SteveA, remote RCS branches? :)
[02:33] <matsubara> SteveA: cve perhaps
[02:33] <BjornT> jamesh: sure creating a branch is quick, but to merge in the revision i need to create a working tree, don't i? anyway, i'll send a message to the list about this.
[02:33] <SteveA> would these be dealt with by the same people, same triage groups etc?
[02:34] <SteveA> I think not, so I'm +1 on bugwatch.
[02:34] <malcc> I'm a bit worried about naming tags for malone features
[02:34] <SteveA> I agree with kiko, -1 on a sabdfl tag.
[02:34] <malcc> My first guess as to a bugwatch tag was that it was for bugs we were watching in some sense, not that it was a feature tag about bugwatches
[02:34] <SteveA> we have the importance and milestone and release features for that
[02:34] <BjornT> +1 on a bugwatch tag
[02:34] <kiko> malcc, ah, but it's okay for soyuz features?!
[02:34] <jamesh> BjornT: don't think so.
[02:35] <SteveA> and using a sabdfl tag will reduce the effectiveness of using the proper features of launchpad's bugtracker
[02:35] <kiko> such as importance.
[02:35] <malcc> kiko: Well you won't confuse a bug about package uploads as being meta, whereas a bug about bugs is inherently more confusable
[02:36] <SteveA> malcc: I agree... except that I don't think we use "bugwatch" in the UI
[02:36] <SteveA> and I don't think it's a common term
[02:36] <SteveA> so I think there's not much room for confusion
[02:36] <spiv> I agree with Steve.
[02:36] <kiko> same here
[02:36] <spiv>   * adding help text to templates (mpt)
[02:36] <mpt> That was last week
[02:37] <mpt> I thought I removed it from the agenda, sorry
[02:37] <spiv> mpt: Steve asked me to add this.
[02:37] <SteveA> yep
[02:37] <mpt> then it should be (SteveA) :-)
[02:37] <SteveA> so, noticed that there are *very*few* templates with help text on mainline
[02:37] <SteveA> you can land helptext as rs=stevea
[02:37] <SteveA> we have a page on teh UI branch that allows us to browse through all the helptext across templates at one time
[02:38] <SteveA> so, we can improve it / catch typos etc.
[02:38] <SteveA> so, please do a few templates in your area of launchpad each day
[02:38] <SteveA> and get them into PQM
[02:39] <SteveA> if anyone is unclear what I'm talking about, please /msg me and we can talk about it
[02:39] <SteveA> that's all.  thanks andrew.
[02:39] <spiv> flacoste: grab SteveA or mpt after the meeting?  We're running low on time...
[02:39] <spiv>   * Keep, Bag, Change
[02:39] <SteveA> mpt: can you suggest a page template on mainline that is an example?
[02:40] <flacoste> will do
[02:40] <mpt> The only example I added was malone-index.pt
[02:40] <jamesh> is there a machine running ui-1.0 up for us to look at yet?
[02:40] <mpt> I haven't surveyed them since
[02:40] <SteveA> jamesh: yes, needs a cert.
[02:40] <spiv> KEEP: 0 outstanding sysadmin requests.  Someone give elmo a hug!
[02:40] <spiv> 5
[02:40] <carlos> SteveA: will you share its url and certificate with us? ;-)
[02:40] <spiv> 4
[02:41] <spiv> 3
[02:41] <spiv> 2
[02:41] <spiv> 1
[02:41] <spiv> Ok.
[02:41] <spiv>   * Three sentences
[02:41] <malcc> DONE: ArchiveRework in review, diskpool refactoring, started PPA trivial impl.
[02:41] <malcc> TODO: Finish PPA trivial impl., some bugs, start fleshing out PPA.
[02:41] <malcc> BLOCKED: No
[02:41] <spiv> Hit me!
[02:41] <jordi> carlos: here still
[02:41] <flacoste> DONE: landed tt-views, handled tt-workflow review, wrote support-tracker documentation
[02:41] <salgado> DONE: Lots of shipit and mirror-prober fixes, checked lots of mirrors and found quite a few broken ones, fixed a few regressions from person-creation-rationale
[02:41] <salgado> TODO: Land DirectPersonCreation, get back to normal life (LocalizedSupportRequests, code review and random fixes)
[02:41] <salgado> BLOCKED: No
[02:41] <flacoste> TODO: land tt-workflow, finish support-tracker documentation
[02:41] <jordi> DONE: emailing
[02:41] <flacoste> BLOCKED: no
[02:41] <BjornT> DONE: code reviews. pick up brad's pending branches. fixed a few bugs in malone-release-management.
[02:41] <mpt> DONE: 1.0 work
[02:41] <mpt> TODO: more 1.0 work
[02:41] <mpt> BLOCKED: no
[02:41] <matsubara> DONE: oops report analysis, interview for USA visa, bug triage TODO: more of the same, need to get more triage action going.
[02:41] <matsubara> BLOCKED: no
[02:41] <BjornT> TODO: code reviews. put up the last bits of malone-release-management.
[02:41] <jamesh> DONE: code reviews, bug 929 (word breaks in fmt:text-to-html), bug 67359 (fix sf bug number extraction), bug 52780 (xml-rpc crash with URLs ending in a slash), FormLayout
[02:41] <cprov> DONE: poppyPPA, started builddPPA, finish NoMoreAptFtparchive and other small b\ranches.
[02:41] <jamesh> TODO: code reviews, put FormLayout up for review, finish off url-utils, spec-branch
[02:41] <jamesh> BLOCKED: no
[02:41] <cprov> TODO: finish builddPPA
[02:41] <cprov> BLOCKED: no
[02:41] <BjornT> BLOCKED: no
[02:41] <jordi> TODO: email, clear queue
[02:41] <Ubugtu> Malone bug 929 in launchpad "Long words (such as URLs) overflow columns" [Critical,In progress]  http://launchpad.net/bugs/929
[02:41] <Ubugtu> Malone bug 67359 in malone "Bad bug id when adding a remote bug watch on SourceForge" [Undecided,Fix released]  http://launchpad.net/bugs/67359
[02:41] <Ubugtu> Malone bug 52780 in launchpad-bazaar "We need to return a user-friendly message when someone tries to register a branch with a invalid URL." [High,Fix committed]  http://launchpad.net/bugs/52780
[02:41] <jordi> BLOCKED: no
[02:41] <danilos> DONE: Firefox ready, OOo work, got USA visa, plane tickets, bus card, contract signed and sent, arranged for new ADSL
[02:41] <danilos> TODO: Finish OOo support, put ff and ooo up for review, search, install ADSL (tommorow, yay!)
[02:41] <danilos> BLOCKED: no
[02:41] <carlos> DONE: bug #2322 and done a new language pack export for Edgy with that fix included, User support (debugged some problems with imports, filed bug #67138), TranslationReview implementation resuming (lots of conflicts and test failures), Setup language pack updates for Edgy, fixed tests for bug #44214 and ask for review, DBA tasks in staging, several reports about contacts I did in Seville and new oportunity using Rosetta.
[02:41] <carlos> TODO: Merge TranslationReview, bug #46982 (KDE plural forms)
[02:41] <carlos> BLOCKED: Firefox branch being reviewed.
[02:42] <spiv> DONE: reviewing, bzr smart server
[02:42] <spiv> TODO: reviews, supermirror smart server
[02:42] <spiv> BLOCKED: no
[02:42] <Ubugtu> Malone bug 2322 in rosetta "Truncated plural forms" [Critical,Fix committed]  http://launchpad.net/bugs/2322
[02:42] <kiko> DONE: Bugwatch fixes, POFileTranslator branch update, land rosetta refactoring, discussed features with various people, Soyuz sprint
[02:42] <kiko> TODO: Launchpad report, land three branches and rejoice
[02:42] <kiko> BLOCKED: stub needs to implement the merging code, BjornT needs to review my Bugwatch patch.
[02:42] <Ubugtu> Malone bug 67138 in rosetta "poparser should use unichr instead of chr for escaped chars" [Medium,Confirmed]  http://launchpad.net/bugs/67138
[02:42] <Ubugtu> Malone bug 44214 in rosetta "We need to add code to prevent POFiles being in the same path" [Critical,In progress]  http://launchpad.net/bugs/44214
[02:42] <Ubugtu> Malone bug 46982 in rosetta "Rosetta does not accept correct KDE plural forms when there are more than 2" [Critical,Confirmed]  http://launchpad.net/bugs/46982
[02:42] <SteveA> DONE: management, ui work, code review, management
[02:42] <SteveA> TODO: more of the same
[02:42] <SteveA> BLOCKED: no
[02:42] <kiko> oh
[02:42] <kiko> TODO: review ArchiveRework
[02:43] <kiko> oh
[02:43] <SteveA> oh?
[02:43] <kiko> BLOCKED: salgado needs to review my fix for bug 30649
[02:43] <Ubugtu> Malone bug 30649 in launchpad "Shouldn't be possible to set an expiry date prior to today when editting a team membership" [Low,In progress]  http://launchpad.net/bugs/30649
[02:43] <kiko> I forgo about that.
[02:43] <kiko> +t
[02:44] <salgado> kiko, is that in my queue?
[02:44] <kiko> salgado, I emailed it to you a while back, matsubara's been reminding you of it, but you are IGNORING it
[02:44] <kiko> it's good that staff reviews are coming up
[02:44] <spiv> kiko: but is it in the PendingReviews queue?
[02:44] <kiko> I will REMEMBER
[02:44] <salgado> nobody reminded me about it
[02:44] <kiko> bla bla bla
[02:45] <kiko> 9:45
[02:45] <kiko> anything else?
[02:45] <spiv> 5
[02:45] <spiv> 4
[02:45] <spiv> 3
[02:45] <spiv> 2
[02:45] <spiv> 1
[02:45] <spiv> 0.5
[02:45] <spiv> nil
[02:45] <spiv> MEETING ENDS
[02:45] <carlos> seg fault!
[02:45] <kiko> Yes! You survive yet another Launchpad meeting. You feel rested.
[02:45] <carlos> :-D
[02:46] <spiv> Thanks everyone.
[02:46] <SteveA> thanks for running the meeting smoothly and on time spiv 
[02:46] <spiv> The trick is to paste the next agenda item whenever you get bored ;)
[02:46] <kiko> indeed
[02:47] <malcc> Hmm, when I chaired we overran. Perhaps I don't bore quickly enough
[02:47] <flacoste> SteveA, mpt: care to discuss the relationship between the new help portlet and the extra_info slot?
[02:47] <flacoste> or extra_toP?
[02:49] <mpt> flacoste, the latter two are cruft
[02:50] <mpt> afaik
[02:50] <flacoste> mpt: do you mean that all instructions should be moved from more_info to the help portlet? this doesn't sound like a good idea to me
[02:51] <flacoste> sorry, extra_info
[02:51] <mpt> No, I don't mean anything like that
[02:51] <flacoste> not all templates have instructions in extra_info, but for those that do, what should go in the help portlet?
[02:52] <mpt> If you want to remove "more_info" and put anything that uses it into "main", that would be fine
[02:52] <flacoste> extra_info is a slot in main, it appears just after the form's title
[02:53] <mpt> right
[02:53] <mpt> so there's no reason for it not to be just a <p>
[02:53] <flacoste> it has class="documentDescription" usually
[02:53] <flacoste> it usually is
[02:53] <mpt> I think the idea is for the help slot to contain more detailed help
[02:53] <mpt> that would be too wordy to show by default.
[02:54] <flacoste> ok, i think that makes sense
[02:55] <flacoste> can we link to pages on help.launchpad.net in there?
[02:55] <mpt> yes
[02:56] <mpt> Great, another entire-file merge conflict
[02:56] <flacoste> mpt: thanks that clarify the scope of the thing for me
[02:57] <flacoste> salgado: ping
[02:57] <mpt> flacoste, can you think of somewhere better I should document what the slot is for?
[02:57] <salgado> flacoste, pong
[02:57] <mpt> I suppose I could put a <tal:XXX in main-template.pt itself
[02:58] <mpt> Or <tal:instructions>
[02:58] <flacoste> mpt: and also in launchpad-form.pt
[02:58] <flacoste> or template-form actually
[02:59] <mpt> ok.
[02:59] <mpt> I'll do that.
[02:59] <flacoste> salgado: i would have time to work on localized-support-tracker if you are still overloaded
[02:59] <flacoste> LocalizedSupportRequest that is
[03:00] <flacoste> otherwise, i'll move on to writing help
[03:08] <salgado> flacoste, any help you can give me on that will be great
[03:09] <flacoste> salgado: ok, what's left to do on that branch?
[03:09] <salgado> let me see if there are any changes I need to push
[03:12] <flacoste> salgado: since it's already over 2000 lines, what do you think if I split out the preferences page changes you did and put that for review today?
[03:12] <flacoste> salgado: that way we would a preferred-languages-ui branch and a localized-support-request
[03:13] <salgado> flacoste, yeah, that sounds good
[03:13] <salgado> let me check how hard it'd be to split it
[03:14] <flacoste> salgado: from the diff on PEndingReviews, it seems not too hard to me
[03:15] <salgado> indeed it shouldn't be... I was just checking if I could just branch a given revision and get only the preferred-languages changes
[03:15] <flacoste> aha, that would be a short cut indeed
[03:16] <salgado> I think I can get that by cherry-picking two revisions, 
[03:18] <flacoste> salgado: i have to go move the car, i'll be back 5-10 mins
[03:18] <flacoste> monday and thursday are "get-a-parfking-fine day"
[03:34] <flacoste> salgado: so how did the split go?
[03:35] <mpt> "Number of remaining unsolved conflicts: 0 (of which -1224257380 are whitespace)"
[03:36] <salgado> flacoste, got 600 lines out of the diff
[03:36] <salgado> flacoste, let me explain what we're missing on localized-support-requests, and I'll take care of landing the preferred-languages branch
[03:36] <flacoste> great!
[03:38] <salgado> flacoste, there are two XXXs on browser/tickettarget.py, about adding the languages used to search for tickets on the messages displayed to users
[03:38] <salgado> I was doing that in the template, but you moved that code to generate the message to a method. :(
[03:39] <flacoste> ok, i'll take care of this
[03:39] <flacoste> that code is only in your local copy?
[03:39] <salgado> which code?
[03:39] <flacoste> the code in the template displaying the languages
[03:40] <salgado> no, I removed that code when solving conflicts. realized it'd be better to start it from scratch than to try and port it
[03:41] <flacoste> so, the branch on devpad is up to date?
[03:41] <salgado> yes, it is
[03:42] <flacoste> ok, apart those XXX, what else is left?
[03:42] <salgado> the searchByPerson() method is not tested
[03:43] <flacoste> yes, that was a tt-views addition
[03:45] <salgado> right. I added a languages argument to it but didn't have time to write tests
[03:46] <flacoste> so the feature is pretty much complete, there are only a few loose ends to tighten
[03:47] <salgado> right
[03:47] <flacoste> ok, i'm on it
[03:47] <salgado> ah, there's one other thing
[03:47] <salgado> actually, a few other things
[03:47] <flacoste> go ahead!
[03:48] <salgado> the new views, I didn't have a look at them
[03:48] <salgado> is it possible to use them to search?
[03:48] <flacoste> it is
[03:48] <flacoste> but they all inherit from the same base view in which you already take care of adding the search support i think
[03:48] <salgado> so I guess they need to be updated to include the Languages widget
[03:50] <flacoste> spiv: can we use bzr+ssh: also for branch operation?
[03:51] <salgado> flacoste, ah, I forgot about the notifications
[03:51] <flacoste> noted
[03:51] <salgado> I had to postpone the changes to the email notifications because of the refactoring you did
[03:52] <mpt> flacoste, I understand now
[03:52] <flacoste> yes, i remember talking to you about that
[03:52] <mpt> extra_info and extra_top are for forms
[03:52] <mpt> They're redundant with each other, but not with anything else, I think
[03:53] <flacoste> well, they hook in different part of the form (extra_info appears outside of <form> and the other inside)
[03:53] <flacoste> but, yeah, my main use of page templates is for forms :-)
[03:54] <flacoste> spiv: it seems that bzr+ssh: also works for branch :-)
[03:54] <salgado> I'm using it for a checkout here
[03:55] <flacoste> salgado: let me know when your preferred-languages branch is up on devpad so that I can see how the localized-support-requests diff looks like
[04:18] <salgado> flacoste, sure thing
[04:26] <Ubugtu> New bug: #68395 in launchpad "The mirror prober needs to send a "Pragma: No-Cache" header when fetching releases.ubuntu.com/.manifest" [Medium,Confirmed]  http://launchpad.net/bugs/68395
[04:36] <flacoste> salgado: you have UTF-8 character in a page test, shouldn't be this handle through an escape sequence?
[04:38] <salgado> flacoste, yes, I think it should. which pagetest is that?
[04:38] <flacoste>  xx-ticket-add-in-other-languages.txt
[04:39] <salgado> flacoste, I think it'd be better/easier to just replace  with eh
[04:39] <salgado> that character was left there by accident
[04:40] <flacoste> salgado: we can also the unicode \u{eaute} or something like that
[04:56] <_MMA_> Hello. Is this where I can get help with Launchpad?
[04:58] <malcc> _MMA_: It's quite possible that you might get some help. What's the problem?
[04:59] <_MMA_> Im trying to "Register a feature specification" and Im a little confused as to the difference between "Name" and "Title". Where they actually appear in the page when I submit it. Also can I edit this info later If I mess it up?
[05:02] <flacoste> _MMA_: the name will appear in the URL, it should be short and contain alphanumeric characters (plus hyphens)
[05:03] <flacoste> _MMA_: the title can be like a short sentence and is use in heading, listing and page titles
[05:06] <_MMA_> Ok. Im looking at it now. Can I edit if I mess up?
[05:06] <flacoste> i'm sure you can edit the title, the name i don't remember
[05:07] <flacoste> probably though
[05:07] <_MMA_> I hope I get it right. If I get it right once Ill have it down.
[05:10] <matsubara> _MMA_: yes, you can edit the spec name afterwards.
[05:12] <_MMA_> Ok. Im looking over what I've written and will submit in a moment. Thanx guys.
[05:13] <_MMA_> Can I add a URL to the summary?
[05:16] <kiko> sure
[05:16] <kiko> you can do whatever you like :)
[05:18] <_MMA_> Is there a formatting or do I just add "https://wiki.ubuntu.com/whatever"?
[05:18] <_MMA_> Im working off: https://launchpad.net/distros/ubuntu/+spec/replacement-init
[05:18] <flacoste> _MMA_: just entering the URL should auto-link it
[05:18] <_MMA_> And there is the "Read More" link in the summary.
[05:19] <_MMA_> Ahh... Ok.
[05:20] <_MMA_> Or can I do it WIKI style like [https://wiki.ubuntu.com/whatever MORE INFO] ?
[05:20] <flacoste> _MMA_: actually, the Read More is added when rendering the page to link to the specification URL field
[05:20] <_MMA_> Ok. :)
[05:20] <flacoste> no, wiki style markup is not supported yet
[05:22] <_MMA_> Gotcha.
[05:36] <SteveA> we should display "name" as "launchpad id" in forms
[05:37] <SteveA> or something that makes it clearer
[05:37] <SteveA> we can continue calling it "name" in the software of course
[05:37] <SteveA> I think we have a bug on this somewhere... mpt's talked about this before
[05:38] <matsubara> SteveA: bug 40616
[05:38] <Ubugtu> Malone bug 40616 in blueprint "Form a little too confusing" [Medium,Fix released]  http://launchpad.net/bugs/40616
[05:39] <_MMA_> I might not be one to chime in here but a "preview" function would be nice.
[05:39] <_MMA_> At least as a user.
[05:45] <SteveA> flacoste: ping sometime
[05:45] <flacoste> SteveA: pong
[05:57] <_MMA_> Im got an error with the "Drafter" entry. Im assigning  it to myself. This is me: https://features.launchpad.net/people/coryisatm Should I be inputting "coryisatm" like in the URL or "C.Kontros (MMA)"?
[06:00] <malcc> _MMA_: That field needs the email address registered in launchpad for the user. But, it's easier if you hit the "Choose..." link and use the search interface
[06:02] <_MMA_> Ok. So Ill just put in my registered email to assign it to me.
[06:03] <malcc> _MMA_: That should work, but the searching-for-a-person popup is very nice, and your launchpad-fu will become stronger if you give it a try and become comfortable with it :)
[06:04] <_MMA_> I did it worked. Searching: "coryisatm" brings up my registered email and adds it. Thanx.
[06:15] <kiko-fud> malcc, so somebody disabled b-f-ns again?
[06:15] <malcc> kiko-fud: To the owner, yes
[06:15] <kiko-fud> malcc, wtf?
[06:16] <malcc> kiko-fud: I remember cprov discussing it with some people a day or two ago. cprov, do you remember who did this and what the issue was?
[06:17] <elmo> 13:21 < cjwatson> it was mailing Debian maintainers who happened to have active Launchpad account
[06:17] <kiko-fud> hmmm
[06:17] <cprov> malcc: kiko-fud: yes, someone from debian complaining about b-f-n receiving b-f-n
[06:18] <cprov> kiko-fud: an old pkg inserted by gina with SPR.creator = SPR.maintainer and inherited til edgy
[06:18] <cprov> kiko: retried via buildd-mass-retry.
[06:19] <cprov> kiko: uhm, also some P-a-s failures we haven't identified yet.
[06:19] <kiko> cprov, so only one complaint?
[06:20] <cprov> kiko: seemed to be enough ...
[06:22] <kiko> cprov, can we fix these old packages with a mass-db update?
[06:24] <cprov> kiko: maybe, we can do this for feisty.
[06:24] <salgado> flacoste, the preferred-languages branch is on pending-reviews already
[06:25] <flacoste> salgado: great!
[06:25] <kiko> cprov?
[06:29] <elmo> kiko: spamming debian maintainers with ubuntu build logs is just about the worst thing we could do right now, from an Ubuntu/Debian relationship perspective.  one is absolutely enough when there's no guarantee it's not just one but a hundred
[06:30] <kiko> elmo, I just wanted to establish how many complaints came in, I share your opinion
[06:33] <carlos> SteveA, kiko: Should we add a spec for UDS about the meetings that Danilo, Stuart and me will have about Rosetta DB schema changes ?
[06:34] <kiko> carlos, yeah, that'd be good.
[06:34] <carlos> ok
[06:39] <carlos> done
[06:40] <flacoste> BjornT: ping
[06:40] <BjornT> flacoste: pong
[06:43] <flacoste> BjornT: i got disconnected, after I said "i think we don't currently have a launchpad.Owner permission?"
[06:44] <BjornT> flacoste: right, that's true. i think it's ok to add one, though, and send an email about it to the list.
[06:46] <flacoste> BjornT: ok, I'll do this
[06:46] <flacoste> BjornT: and thanks again for doing the tt-workflow review, it was not an easy endeavour
[06:49] <BjornT> you're welcome, and i'm glad it's over :)
[07:12] <seaLne> is there any way to remove comments which are spam on malone? not sure if this has happened much before?
[07:12] <seaLne> https://launchpad.net/distros/ubuntu/+source/kubuntu-meta/+bug/48777/comments/17
[07:12] <Ubugtu> Malone bug 48777 in kubuntu-meta "No GUI way to change screen resolution in Kubuntu" [Wishlist,Rejected]  
[07:16] <kiko> malcc?
[07:16] <kiko> seaLne, it's the second time I hear of this.
[07:17] <kiko> seaLne, I'll request it be removed.
[07:18] <seaLne> kiko: thanks
[07:18] <kiko> seaLne, do you think that user exists only to post spam?
[07:19] <seaLne> 0 karma before today and then they submit spam as a bug today, seems likely to me
[07:20] <Ubugtu> New bug: #68426 in launchpad-development-infrastructure "Default pqm-submit fails when pushing with bzr+ssh:" [Undecided,Unconfirmed]  http://launchpad.net/bugs/68426
[07:22] <carlos> see you!!
[07:26] <Burgwork> any of the blueprint hackers here?
[07:26] <kiko> who are those?
[07:26] <Burgwork> heh
[07:31] <kiko> BjornT, ping?
[07:31] <kiko> Burgwork, what's up?
[07:31] <Burgwork> kiko: just a sec phone
[07:32] <kiko> suuuure.
[07:33] <BjornT> kiko: pong
[07:34] <kiko> BjornT, I added a test to ensure that the BugWatchUpdateError thing works. would you like to see it?
[07:34] <kiko> it's a bit wonky but works
[07:35] <BjornT> sure, i could take a look at it.
[07:35] <kiko> https://sodium.ubuntu.com/~andrew/paste/filelDkZ64.html
[07:36] <kiko> BjornT, kinda assumes DNS doesn't hang forever.
[07:36] <kiko> it's not easy to test otherwise
[07:37] <kiko> matsubara, are there any other bugs related to bugzilla or issuezilla?
[07:38] <Burgwork> kiko: as part of the ubuntu weekly news, I would love it if LP could produce a "specs that have changed this week"
[07:38] <Burgwork> added, status changed, url change, et.c
[07:38] <kiko> Burgwork, hmm, interesting. I actually wanted this too
[07:38] <Burgwork> I am certain those that manage the distro would love it as well
[07:39] <matsubara> kiko: I suppose you're talking about bug 67091
[07:39] <Ubugtu> Malone bug 67091 in malone "Issuezilla bug watches are failing to update" [High,In progress]  http://launchpad.net/bugs/67091
[07:39] <kiko> matsubara, well, any others? I just found one
[07:39] <BjornT> kiko: hmm, i don't like that test. relying on dns is kind of flaky, some setups always succeeds resolving domain names.
[07:39] <kiko> https://launchpad.net/products/malone/+bug/32524
[07:40] <Ubugtu> Malone bug 32524 in malone "Should support Bugzilla bugtrackers with version under 2.16" [Medium,In progress]  
[07:40] <kiko> BjornT, should I just drop it? it's hard to test otherwise.. maybe you have a better idea.
[07:40] <matsubara> kiko: bug 67719
[07:40] <Ubugtu> Malone bug 67719 in malone "Bugzilla bug syncing stuffed" [Undecided,In progress]  http://launchpad.net/bugs/67719
[07:40] <kiko> matsubara, you can add the bugwatch tag to:
[07:40] <kiko> https://launchpad.net/products/malone/+bugs?field.searchtext=watch&orderby=-importance&search=Search&field.status%3Alist=Unconfirmed&field.status%3Alist=Confirmed&field.status%3Alist=In+Progress&field.status%3Alist=Needs+Info&field.status%3Alist=Fix+Committed&field.assignee=&field.owner=&field.omit_dupes=on&field.has_patch=&field.has_no_package=
[07:41] <kiko> https://launchpad.net/products/malone/+bugs?field.searchtext=bugzilla&orderby=-importance&search=Search&field.status%3Alist=Unconfirmed&field.status%3Alist=Confirmed&field.status%3Alist=In+Progress&field.status%3Alist=Needs+Info&field.status%3Alist=Fix+Committed&field.assignee=&field.owner=&field.omit_dupes=on&field.has_patch=&field.has_no_package=
[07:41] <Burgwork> kiko: worth creating a spec for it?
[07:41] <kiko> Burgwork, sure it is!
[07:41] <Burgwork> will do
[07:43] <BjornT> kiko: yeah, i think it's better to drop it. in this case i think it's better to have no test rather than a flaky one, and i can't think of a good and easy way of testing it.
[07:44] <kiko> BjornT, I really don't like the fact that we don't even try to test the fact that the script actually works :-/
[07:44] <kiko> anyway, ok.
[07:46] <BjornT> kiko: well, if you feel like it, you could create a test web server (similary to the test keyserver), that would use TestBugzilla or something to serve its pages.
[07:47] <kiko> BjornT, one blue day. :)
[07:52] <kiko> salgado, do you confirm that the revision 4193 was cherry-picked? it's in the wrong place in LPS.
[07:53] <salgado> kiko, yes, it was cherry picked
[07:53] <kiko> thanks, I'll update.
[07:53] <salgado> I didn't notice it was on the wrong place
[07:54] <kiko> tis ok
[08:05] <kiko> ddaa, ping?
[08:05] <ddaa> kiko: pong
[08:06] <kiko>   [r=BjornT]  saner svn_oo.RevisionRangeParser.getLowRev, make svn_oo.WorkingTree.repourl() more readable and safer
[08:06] <kiko> ddaa, in english.
[08:07] <ddaa> nothing to see there, peripheral cleanup related to other work
[08:08] <ddaa> mh
[08:08] <ddaa> wait a min... I think there was something especially bogus with that one
[08:08] <kiko> you don't want it included in the report?
[08:09] <ddaa> saner svn_oo.RevisionRangeParser.getLowRev
[08:09] <ddaa> No longer retrieves the entire log of the branch, but only the first revision.
[08:09] <ddaa> No longer requires a working tree, but can work directly on a repository URL.
[08:09] <ddaa> Well, it's not something that makes our users significantly happier
[08:10] <ddaa> it's just one of those frequent peripheral cleanups I need to do when working on cscvs
[08:10] <ddaa> When I'm done with the review of cscvs/partial-copy, we will have something that makes user happier :) But it's a nasty bitch.
[08:11] <ddaa> yeah, I remember this patch is related to partial-copy, but in a rather remote way
[08:12] <ddaa> so feel free to say "ddaa has been busy cooking some tasty cscvs voodoo to bring large improvements in svn import coverage", but the specific patches landed so far are quite boring.
[08:33] <kiko> thanks ddaa 
[08:50] <Burgwork> kiko: how do unmark a spec as superceded?
[08:50] <Burgwork> https://features.launchpad.net/distros/ubuntu/+spec/ubuntu-update-server/
[08:50] <kiko> superseded
[08:50] <Burgwork> yes
[08:50] <Burgwork> see how it is both new and superseded?
[08:51] <kiko> select Not Superseded in that amazingly long list.
[08:51] <kiko> I just did it for you
[08:51] <Burgwork> perfect
[08:51] <Burgwork> where is that?
[08:51] <Burgwork> oh, in the list?
[08:51] <Burgwork> I see now
[08:52] <kiko> yeah
[08:52] <kiko> it's not beautiful
[08:52] <Burgwork> no, very much not so
[09:09] <kiko> ddaa,   [trivial]  tweak importd so sync does not fail when the working directory of the job does not already exist
[09:09] <kiko> ddaa, what would cause this to happen?
[09:09] <ddaa> mh... I do not think that's the right question
[09:10] <ddaa> since importd now uploads and downloads source _and_ target trees to escudero
[09:10] <ddaa> there is not any requirement to keep persistent data on the importd slaves
[09:10] <ddaa> mostly, the data is currently left around to make debugging easier
[09:11] <ddaa> so this is a fix to remove an assuption that is no longer true
[09:11] <ddaa> the assumption being that the slaves needs persistent local data to do an import
[09:12] <ddaa> one case where this feature would be triggered would be when a product rename would cause an importd job to be affected to a different slave
[09:14] <ddaa> the case that actually triggered the need for the fix, was the change in the naming scheme for working directories, so it is based on the series id and no longer on the target arch name.
[09:14] <ddaa> since there is no longer ANY target arch name in the database :)
[09:15] <ddaa> I mean, ProductSeries.targetarch* columns.
[09:15] <kiko> right.
[09:15] <ddaa> looking back on this whole thing, it does give me a fuzzy feeling
[09:15] <salgado> how hard is it to write a page test that POSTs an unprintable character?
[09:16] <salgado> BjornT, maybe you know ^?
[09:16] <ddaa> how all those months of work to remove arch stuff in importd eventually all fell in place
[09:16] <kiko> salgado, you can just use http() if you can't use testbrowser
[09:16] <kiko> the form posted data is encoded
[09:17] <salgado> kiko, hmmm. I don't see what you mean
[09:18] <kiko> you know the http() function we used to use?
[09:18] <salgado> yes, I do
[09:18] <kiko> so you can use it to do form posts, right?
[09:19] <salgado> right
[09:19] <kiko> so it has no limitations as to what it can post
[09:19] <kiko> you can post whatever you like
[09:19] <kiko> so use it. :)
[09:19] <BjornT> salgado: doesn't it work with testbrowser? what error do you get?
[09:19] <salgado> right, and how would I post an unprintable character?
[09:19] <kiko> salgado, %0C or something?
[09:20] <BjornT> salgado: escape it with \uxxxx notation?
[09:22] <salgado> BjornT, would '\uxxx'.encode('ascii') fail?
[09:22] <kiko> salgado, you mean decode.
[09:22] <kiko> and yes
[09:22] <kiko> if xxx > 127
[09:23] <BjornT> salgado: yes it would. you don't have to post it as ascii, though. i'm not sure if testbrowser encodes unicode automatically. if not, post it as utf-8
[09:24] <salgado> okay, I'll explain why I need that
[09:24] <salgado> sorry for not doing that before
[09:24] <salgado> bug 63579
[09:24] <Ubugtu> Malone bug 63579 in shipit "Request CDs form needs better validation for all fields" [High,Confirmed]  http://launchpad.net/bugs/63579
[09:26] <salgado> what happens is that somebody posted a control character in the form, and although the form-validation code thinks the character is valid because it can be encoded into 'ascii', the character is not printable and our DB constraint catches that 
[09:26] <salgado> basically, the issue is that the form-validation doesn't match the DB validation
[09:26] <salgado> but I think the DB validation is the one that is correct
[09:27] <salgado> does that make sense or am I on crack?
[09:27] <salgado> BjornT, kiko ^
[09:27] <kiko> salgado, makes sense, yeah.
[09:28] <kiko> salgado, you mean decode though 
[09:28] <salgado> no, I mean encode
[09:28] <kiko> ah 
[09:28] <kiko> it's utf-8
[09:28] <kiko> and being encoded into ascii
[09:28] <kiko> so yes you are right
[09:28] <kiko> what does the DB validation check? for a range of ascii?
[09:29] <salgado> does a re.match() using string.printable
[09:29] <kiko> ah.
[09:29] <BjornT> salgado: ah, so you want ascii. you could use '\x06'
[09:29] <salgado> BjornT, where did you get that code from?
[09:30] <BjornT> salgado: i did '^F' in a python prompt (using Ctrl+f)
[09:30] <kiko> control-E works too

[09:31] <BjornT> well, that doesn't give you '\x06' :)
[09:32] <LarstiQ> ^F in a prompt the what now?
[09:33] <BjornT> >>> '^F'
[09:33] <BjornT> '\x06'
[09:35] <Ubugtu> New bug: #68457 in soyuz "P-a-s broken: yaboot built on !powerpc" [Undecided,Unconfirmed]  http://launchpad.net/bugs/68457
[09:35] <kiko> I use ^v
[09:35] <kiko> which is btw the future
[09:35] <LarstiQ> it is how I was brought up
[11:47] <kiko> ddaa,   [r=BjornT]  ChangesIterator tests check interesting attributes of yielded changes
[11:47] <ddaa> cleanup in preparation for partial-copy
[11:48] <kiko> partial-copy?
[11:50] <ddaa> the cscvs branch I mentioned before
[11:50] <ddaa> will fix a bunch of current svn import failures
[11:50] <ddaa> spent the whole day doing review fixes...
[11:52] <kiko> so partial-copy is something that makes the svn imports work much better?
[11:53] <ddaa> you could say that
[11:53] <ddaa> better in terms of improved coverage
[11:53] <kiko> I see
[11:53] <kiko> what does partial-copy mean exactly?
[11:54] <ddaa> check my reply to bjorn's review of cscvs/partial-copy-part-one
[11:54] <ddaa> at the end of the mail
[11:54] <kiko> okay
[11:55] <ddaa>     A partial copy is a directory copy change with simultaneous D (delete) or R     (replace) actions in children paths. These actions must not translate to     actual deletes, but must instead change the additions produced by the     directory copy. In particular, a delete must be interpreted as a pruning of     the file hierarchy being copied.
[11:56] <ddaa> currently, when we have a partial copy case, cscvs just blows up
[11:56] <ddaa> the partial-copy code fixes the logic to handle those cases correctly
[11:56] <kiko> I see
[11:56] <kiko> is that a common situation?
[11:56] <ddaa> quite common
[11:57] <kiko> interesting
[11:57] <ddaa> mostly it looks like it got more common over time, as people figured out how to use svn
[11:57] <kiko> I see
[11:57] <ddaa> other side effects of this work include support for all the cases of "replace" changes
[11:57] <ddaa> the older code nedeed to dealt with those with special cases, and had only one case supported
[11:58] <ddaa> it will also make it possible to support renames and file resurrection
[11:58] <ddaa> though not quite right now, but soon
[11:58] <kiko> super-cool
[11:59] <ddaa> essentially, it just replaces some ad-hoc stuff hacked up by a badly overworked lifeless at the end of 2004, with something actually designed with understanding of svn
[12:01] <kiko> heh