[00:22] Anyone want to take a really mechanical branch? [00:22] sinzui, maybe you're still around? ^^ [00:22] yes [00:23] sinzui, alright, running tests now, will send it your way. [00:23] okay. Thanks [00:37] sinzui, proposal sent. [00:37] I will watch for it [01:27] rockstar: r=me Good to land as it is. [02:05] Any reviewers around for a fix for bug #432832? [02:05] Bug #432832: update-sourcecode broken for people without access to shipit and c-i-p [02:06] wgrant, hi [02:07] wgrant, where's you patch? [02:07] rockstar: Pushing. Give me a moment. [02:14] Slooow push. [02:24] ... and I of course stuffed up the MP. [02:24] rockstar: https://code.edge.launchpad.net/~wgrant/launchpad/bug-432832/+merge/12109 [02:33] wgrant, you know you can just send the bundle, right? [02:33] rockstar: That relies on me having email set up properly. [02:33] I've never actually used bzr-send before. [02:33] I probably should work out how to do it. [02:34] wgrant, if you plan on submitting patches as regularly as you have been, I suggest you learn to. :) [02:34] If there are holes in our documentation, feel free to bug me and I'll fix them. [02:35] wgrant, how have you tested this? [02:39] rockstar: lamalex installed LP with a very similar patch, and I ran rocketfuel-get with it on both existing and new installations. I haven't tested how it goes with shipit and c-i-p available, of course. [02:40] wgrant, how can I test it, since I have both available? [02:42] rockstar: Run 'utilities/update-sourcecode ../../lp-sourcedeps' to verify it works on an existing one. Then maybe uncommit a revision or two from a branch, remove another branch, run it again, and verify that they're all up to date. [02:44] You should not see any errors, since you have access to shipit and c-i-p. [02:55] I never knew about this script. [02:56] rockstar: rf-get, so by extension rf-setup, use it. [02:56] However it was just rewritten. [02:57] wgrant, ah, okay. I eventually wrote my own script because my folder layout was different, and rf-setup/rf-get only supported one installation location. [02:58] rockstar: By overriding the LP_PROJECT_ROOT envvar I'm able to have it working on three different installations. [02:59] wgrant, yeah, it used to not support that. [02:59] Also, AIUI, it still doesn't support separate branches and trees. [02:59] Probably not. [02:59] I don't normally use it either, but that's because it was terribly slow. [02:59] Now it only takes a couple of minutes. [03:02] rockstar: Thanks. Can you land that for me? [03:02] wgrant, yeah. I'll have to figure out how to do that. [03:05] utilities/ec2test.py --headless -s "[r=rockstar][ui=none] fix update-sourcecode in the case that optional branches are inaccessible" lp:~wgrant/launchpad/bug-432832 --email=me@williamgrant.id.au [03:05] Maybe throw a ' (wgrant)' just after the '[ui=none]'. [03:05] Some people do, some people don't. [03:06] Yea, I'm not sure I need to run that through ec2test. [03:06] True. [03:07] I don't see a direct way of doing it with bzr-pqm, so off to ec2 it went. [03:07] OK, thanks. [03:08] I'll leave the big rocketfuel-setup breakage (bug #432830) to somebody more qualified. [03:08] Bug #432830: rocketfuel-setup does not work [03:08] Man, I currently have your ec2 runs going. My wife is going to think I'm buying crap on Amazon again. [03:08] s/your/four/ [03:08] Heh. [03:18] wgrant, I killed the ec2 instance and figured out how to just pqm submit it. [03:52] rockstar: Great, thanks! [03:52] wgrant, thanks for your patch. [12:47] henninge: could i trouble you for a trivial review? it's about a four line change. [12:47] if not i'll rs it [12:48] bac: sure, bring it on [12:50] henninge, btw happy late bday :-) [12:50] gary_poster: thank you ;-) [12:54] henninge: MP sent. waiting for it to show up [12:55] hi gary_poster. do you know about http://www.sparkcon.com/ ? it sounds fun but i've never been -- all the way in raleigh... [12:56] bac: lol, no, looks cool! [12:56] henninge: https://code.edge.launchpad.net/~bac/launchpad/import_error/+merge/12115 [12:56] * henninge looks [12:56] gary_poster: some friends of mine started it a few years ago. it's really grown huge. [12:59] bac: wow, cool! I wish there were a better "keep track of the actually cool events in the triangle" feed. The ones I've seen don't include the stuff I want to see. ;-) until the most exciting thing I knew about this weekend was a greek festival. ;-) [12:59] until now I mean [13:00] gary_poster: yeah, i'm thinking i may have to break my 'no raleigh' rule today and wander down there [13:00] bac: are you keeping the commented-out import for documentation purposes? Is that common practice so that people know what it means? [13:02] henninge: no, it was an oversight. i'll remove it [13:02] bac: ok, r=me then ;-) [13:02] henninge: danke [13:02] bitte! === henninge is now known as henninge-afk [14:25] hi abentley [14:57] bac: Happily, this is only a problem with the test, and yes, I caused it. [14:57] abentley: great. if you can get a fix in rs=bac it would be great! [14:59] bac: Will do.