[02:54] <cjwatson>     """A utility of this interface used to create _things_."""
[02:54] <cjwatson> helpful comment is helpful
[02:58] <wgrant> cjwatson: :)
[02:58] <wgrant> We try.
[02:59] <wgrant> cjwatson: Oh, you're working on the master side of livefs things?
[02:59] <wgrant> cjwatson: That's about to less messy, though I'm off this week.
[03:00] <wgrant> about to become less messy, that is
[04:17] <alexhenrie> hello, is there currently an on-call reviewer?
[04:19] <wgrant> alexhenrie: It's a weekend, so not really. But I'm around.
[04:20] <alexhenrie> well if you can help me, great, and if not, no big deal.
[04:20] <alexhenrie> I just saw that my submission was marked "resubmit"
[04:20] <alexhenrie> https://code.launchpad.net/~alexhenrie24/ubuntu/saucy/lubuntu-default-settings/fix-for-1232578
[04:20] <alexhenrie> I didn't get an email, and I don't see any explanation
[04:20] <alexhenrie> what am I supposed to do?
[04:22] <wgrant> alexhenrie: 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:23] <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:24] <alexhenrie> I'm new to Ubuntu's code submission process. where did you find that comment?
[04:25] <wgrant> At the URL you linked.
[04:26] <alexhenrie> did you have to click something to see it?
[04:26] <wgrant> No.
[04:26] <elmo> alexhenrie: https://code.launchpad.net/~alexhenrie24/ubuntu/saucy/lubuntu-default-settings/fix-for-1232578/+merge/191047
[04:27] <wgrant> Oh
[04:27] <wgrant> I must have clicked through to the MP without thinking, sorry.
[04:27] <elmo> alexhenrie: the 'ready for review link
[04:27] <wgrant> alexhenrie: Anyway, #ubuntu-devel or a Lubuntu-related channel might be more relevant.
[04:28] <alexhenrie> okay thanks
[04:28] <alexhenrie> for now I want to concentrate on figuring out how to do what Dmitry asked
[04:29] <alexhenrie> I found a tutotial on this before but I can't find it now
[04:31] <wgrant> alexhenrie: 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] <alexhenrie> thank you :)
[04:34] <alexhenrie> btw I found the page I was looking for: http://developer.ubuntu.com/packaging/html/fixing-a-bug.html
[04:42] <alexhenrie> okay I have another easy question
[04:42] <alexhenrie> was 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/263
[04:45] <wgrant> alexhenrie: No, 0.35 is correct for that package. ubuntuX isn't appended to the version of some Ubuntu-native packages.
[04:47] <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:49] <alexhenrie> okay so I need to edit my patch and resubmit
[04:49] <alexhenrie> is there anything else obviously wrong with it?
[04:52] <wgrant> alexhenrie: I'm no Lubuntu developer, but it looks sensible enough to me.
[04:57] <alexhenrie> thanks. I was just looking for errors in the commit message or whatever.
[04:57] <alexhenrie> I think I have it fixed now: https://code.launchpad.net/~alexhenrie24/lubuntu-default-settings/fix-for-1232578
[04:58] <alexhenrie> thanks for the help!
[05:02] <wgrant> np