dpm | good morning everyone | 07:05 |
---|---|---|
RawChid | Good morning :) | 09:26 |
dpm | morning RawChid :) | 09:43 |
RawChid | Hey dpm, if I do a merge proposal. Where is the field "commit message" used for? | 09:48 |
RawChid | There is also Description of change | 09:49 |
dpm | 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 | 09:59 |
dpm | 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 |
RawChid | Ah, loud and clear | 10:00 |
dpm | :) | 10:00 |
RawChid | 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:04 |
RawChid | I asked in #lp, but no answer yet | 10:05 |
dpm | RawChid, try asking on #launchpad-dev or pinging the help contact on #launchpad | 10:20 |
RawChid | Yeah, I'm asking now again | 10:20 |
RawChid | Are you sure the code did worked for you? Because I found 2 errors when I checked out your code | 10:21 |
dpm | RawChid, it did work before I introduced the errors, yes :) I saw you fixed them in your branch, thanks | 10:23 |
RawChid | Oke, good to know that the login DID work :) | 10:24 |
RawChid | You use KDE? | 10:24 |
dpm | I use Ubuntu with GNOME | 10:25 |
RawChid | ack | 10:26 |
RawChid | http://pastebin.com/kZ9wr6BL | 10:26 |
RawChid | I wondered because of the kwallet notifications | 10:26 |
dpm | well, that comes from your system, not mine :) | 10:27 |
dpm | in any case, perhaps authentication in launchpadlib does not work too well in Kubuntu? | 10:29 |
dpm | I'd recommend reading | 10:29 |
dpm | https://help.launchpad.net/API/launchpadlib#Getting%20started | 10:29 |
RawChid | I use natty, Gnome! | 10:29 |
RawChid | Thanks for the link | 10:29 |
dpm | and https://help.launchpad.net/API/ThirdPartyIntegration | 10:29 |
dpm | :) | 10:30 |
RawChid | Hmm, staging is getting a code udpate. It looks like I can get the pot from production without a problem :-D | 10:44 |
RawChid | I found it. I was authenticating to staging, but the pot URL is to production | 10:50 |
dpm | oh, that was it then :) then we should just add the 'staging' part to the URL if we're logging into staging | 11:01 |
dpm | good work | 11:01 |
RawChid | Yeah, I now have launchpad.distributions["ubuntu"] to get the first part of the URL | 11:02 |
dpm | 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:14 |
RawChid | Oh yeah, good point. | 11:22 |
RawChid | dpm, I don't understand you comment of whitespave | 11:23 |
RawChid | whitespace*, I did not see that in the search branch | 11:23 |
RawChid | Of does the diff on launchpad ignores it? | 11:24 |
dpm | 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:24 |
RawChid | Oke, I'll keep that in mind | 11:25 |
dpm | cool, thanks :) | 11:25 |
RawChid | Any suggestions on assistance to prevent this? (maybe certain IDE or somehting) | 11:26 |
RawChid | 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:30 |
dpm | 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 |
dpm | btw, don't worry about asking questions ;) | 11:31 |
dpm | let me have a look at the comments I put on the priority setter tool... | 11:32 |
RawChid | I radically changed the code of pot-prio at the moment | 11:34 |
RawChid | I've used eclipse, gvim and now geany for Python. Enabling "show line endings" helps ;) | 11:38 |
dpm | :) | 11:38 |
RawChid | News about my --prio question? | 11:40 |
dpm | 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:42 |
RawChid | Ok, good to know | 11:44 |
dpm | 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:52 |
RawChid | Hehe, great | 11:54 |
RawChid | I hope to make a big MP on pot-prio later this day :) | 11:55 |
dpm | awesome | 12:03 |
artnay | upstream translations of gconf2 overwrote LP translations 13 hours ago | 13:19 |
TLE | dpm: hey | 13:41 |
TLE | Is this schedule https://dev.launchpad.net/Translations/LanguagePackSchedule the one in effect now? | 13:41 |
dpm | 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 |
dpm | in fact, let me ping them now | 13:46 |
TLE | dpm: ok yes, I got confused for a second | 13:47 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!