[07:05] good morning everyone [09:26] Good morning :) [09:43] morning RawChid :) [09:48] Hey dpm, if I do a merge proposal. Where is the field "commit message" used for? [09:49] There is also Description of change [09:59] RawChid, the 'commit message' field is for the submitter to specify a particular commit message you'd like to be used when merging your branch into trunk. If unspecified, the person doing the merge will choose a commit message himself/herself [10:00] The description of change is to explain, in words, what the change is about. This is quite valuable as an overview of the changes for the reviewer. [10:00] Ah, loud and clear [10:00] :) [10:04] I still haven't solved the authentication problem with set-pot-prio. Weird thing is when I paste the URL in my browser, I DO get a response! [10:05] I asked in #lp, but no answer yet [10:20] RawChid, try asking on #launchpad-dev or pinging the help contact on #launchpad [10:20] Yeah, I'm asking now again [10:21] Are you sure the code did worked for you? Because I found 2 errors when I checked out your code [10:23] RawChid, it did work before I introduced the errors, yes :) I saw you fixed them in your branch, thanks [10:24] Oke, good to know that the login DID work :) [10:24] You use KDE? [10:25] I use Ubuntu with GNOME [10:26] ack [10:26] http://pastebin.com/kZ9wr6BL [10:26] I wondered because of the kwallet notifications [10:27] well, that comes from your system, not mine :) [10:29] in any case, perhaps authentication in launchpadlib does not work too well in Kubuntu? [10:29] I'd recommend reading [10:29] https://help.launchpad.net/API/launchpadlib#Getting%20started [10:29] I use natty, Gnome! [10:29] Thanks for the link [10:29] and https://help.launchpad.net/API/ThirdPartyIntegration [10:30] :) [10:44] Hmm, staging is getting a code udpate. It looks like I can get the pot from production without a problem :-D [10:50] I found it. I was authenticating to staging, but the pot URL is to production [11:01] oh, that was it then :) then we should just add the 'staging' part to the URL if we're logging into staging [11:01] good work [11:02] Yeah, I now have launchpad.distributions["ubuntu"] to get the first part of the URL [11:14] RawChid, on http://bazaar.launchpad.net/~rachidbm/ubuntu-l10n-tools/pot-priority/revision/10, could you try to keep the line under 80 characters? It's just to follow standard python practice. I.e. instead of putting everything on one line, just add the missing '+' sign on the original line 87 [11:22] Oh yeah, good point. [11:23] dpm, I don't understand you comment of whitespave [11:23] whitespace*, I did not see that in the search branch [11:24] Of does the diff on launchpad ignores it? [11:24] RawChid, there were lots of unnecessary spaces at the end of statements and on blank lines. It might well be that LP's diff ignores it, but they were on the file [11:25] Oke, I'll keep that in mind [11:25] cool, thanks :) [11:26] Any suggestions on assistance to prevent this? (maybe certain IDE or somehting) [11:30] Sorry for bugging you dpm, but I have another question :P. in pot-prio you suggested --source and --template. I guess also --prio is needed then? [11:31] RawChid, I don't know if gedit can highlight superfluous whitespace. There was a really useful developer plugin, let me dig out the name. I use gvim. [11:31] btw, don't worry about asking questions ;) [11:32] let me have a look at the comments I put on the priority setter tool... [11:34] I radically changed the code of pot-prio at the moment [11:38] I've used eclipse, gvim and now geany for Python. Enabling "show line endings" helps ;) [11:38] :) [11:40] News about my --prio question? [11:42] RawChid, yeah, I've just looked at it and updated the TODO accordingly: you're correct, we need a priority option as well: http://bazaar.launchpad.net/~dpm/ubuntu-l10n-tools/trunk/view/head:/ul10n_tools/lp_set_pot_priority/__init__.py#L36 [11:44] Ok, good to know [11:52] RawChid, I also merged in https://code.launchpad.net/~rachidbm/ubuntu-l10n-tools/get-team-info - I saw that the changes were simple and made sense. So no need to submit a merge proposal, you can mark it as merged now :) [11:54] Hehe, great [11:55] I hope to make a big MP on pot-prio later this day :) [12:03] awesome [13:19] upstream translations of gconf2 overwrote LP translations 13 hours ago [13:41] dpm: hey [13:41] Is this schedule https://dev.launchpad.net/Translations/LanguagePackSchedule the one in effect now? [13:46] TLE, yeah, I sent you the e-mail confirmation as requested. The only thing is that the language pack export for natty is still not ready, so pitti can't create the packages yet. I'll wait for a couple of hours and if it's not there yet I'll ping the LP folks [13:46] in fact, let me ping them now [13:47] dpm: ok yes, I got confused for a second