cjwatson    """A utility of this interface used to create _things_."""02:54
cjwatsonhelpful comment is helpful02:54
wgrantcjwatson: :)02:58
wgrantWe try.02:58
wgrantcjwatson: Oh, you're working on the master side of livefs things?02:59
wgrantcjwatson: That's about to less messy, though I'm off this week.02:59
wgrantabout to become less messy, that is03:00
alexhenriehello, is there currently an on-call reviewer?04:17
wgrantalexhenrie: It's a weekend, so not really. But I'm around.04:19
alexhenriewell if you can help me, great, and if not, no big deal.04:20
alexhenrieI just saw that my submission was marked "resubmit"04:20
alexhenrieI didn't get an email, and I don't see any explanation04:20
alexhenriewhat am I supposed to do?04:20
wgrantalexhenrie: Hm, I think you might be a bit confused. Reviewers here review branches of the code that runs Launchpad.net, not every branch on Launchpad. But I see a comment in that merge proposal explaining the Resubmit vote.04:22
wgrant"Looks good to me, but can you please rebase this against lp:lubuntu-default-settings so that Lubuntu guys could take a look at this?"04:23
alexhenrieI'm new to Ubuntu's code submission process. where did you find that comment?04:24
wgrantAt the URL you linked.04:25
alexhenriedid you have to click something to see it?04:26
elmoalexhenrie: https://code.launchpad.net/~alexhenrie24/ubuntu/saucy/lubuntu-default-settings/fix-for-1232578/+merge/19104704:26
wgrantI must have clicked through to the MP without thinking, sorry.04:27
elmoalexhenrie: the 'ready for review link04:27
wgrantalexhenrie: Anyway, #ubuntu-devel or a Lubuntu-related channel might be more relevant.04:27
alexhenrieokay thanks04:28
alexhenriefor now I want to concentrate on figuring out how to do what Dmitry asked04:28
alexhenrieI found a tutotial on this before but I can't find it now04:29
wgrantalexhenrie: bzr branch lp:lubuntu-default-settings; hack hack hack; bzr commit; bzr push lp:~alexhenrie24/lubuntu-default-settings/fix-for-1232578; then propose a merge using the web UI.04:31
alexhenriethank you :)04:31
alexhenriebtw I found the page I was looking for: http://developer.ubuntu.com/packaging/html/fixing-a-bug.html04:34
alexhenrieokay I have another easy question04:42
alexhenriewas I supposed to have changed the revision from 0.35 to 0.35ubuntu1? http://bazaar.launchpad.net/~alexhenrie24/lubuntu-default-settings/fix-for-1232578/revision/26304:42
wgrantalexhenrie: No, 0.35 is correct for that package. ubuntuX isn't appended to the version of some Ubuntu-native packages.04:45
wgrant(the ubuntuX suffix exists to disambiguate Ubuntu and Debian package versions, but it's unlikely that lubuntu-default-settings will ever be in Debian)04:47
alexhenrieokay so I need to edit my patch and resubmit04:49
alexhenrieis there anything else obviously wrong with it?04:49
wgrantalexhenrie: I'm no Lubuntu developer, but it looks sensible enough to me.04:52
alexhenriethanks. I was just looking for errors in the commit message or whatever.04:57
alexhenrieI think I have it fixed now: https://code.launchpad.net/~alexhenrie24/lubuntu-default-settings/fix-for-123257804:57
alexhenriethanks for the help!04:58

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!