/srv/irclogs.ubuntu.com/2009/09/19/#launchpad-reviews.txt

rockstarAnyone want to take a really mechanical branch?00:22
rockstarsinzui, maybe you're still around? ^^00:22
sinzuiyes00:22
rockstarsinzui, alright, running tests now, will send it your way.00:23
sinzuiokay. Thanks00:23
rockstarsinzui, proposal sent.00:37
sinzuiI will watch for it00:37
sinzuirockstar: r=me Good to land as it is.01:27
wgrantAny reviewers around for a fix for bug #432832?02:05
mupBug #432832: update-sourcecode broken for people without access to shipit and c-i-p <Launchpad Foundations:In Progress by wgrant> <https://launchpad.net/bugs/432832>02:05
rockstarwgrant, hi02:06
rockstarwgrant, where's you patch?02:07
wgrantrockstar: Pushing. Give me a moment.02:07
wgrantSlooow push.02:14
wgrant... and I of course stuffed up the MP.02:24
wgrantrockstar: https://code.edge.launchpad.net/~wgrant/launchpad/bug-432832/+merge/1210902:24
rockstarwgrant, you know you can just send the bundle, right?02:33
wgrantrockstar: That relies on me having email set up properly.02:33
wgrantI've never actually used bzr-send before.02:33
wgrantI probably should work out how to do it.02:33
rockstarwgrant, if you plan on submitting patches as regularly as you have been, I suggest you learn to.  :)02:34
rockstarIf there are holes in our documentation, feel free to bug me and I'll fix them.02:34
rockstarwgrant, how have you tested this?02:35
wgrantrockstar: 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:39
rockstarwgrant, how can I test it, since I have both available?02:40
wgrantrockstar: 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:42
wgrantYou should not see any errors, since you have access to shipit and c-i-p.02:44
rockstarI never knew about this script.02:55
wgrantrockstar: rf-get, so by extension rf-setup, use it.02:56
wgrantHowever it was just rewritten.02:56
rockstarwgrant, 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:57
wgrantrockstar: By overriding the LP_PROJECT_ROOT envvar I'm able to have it working on three different installations.02:58
rockstarwgrant, yeah, it used to not support that.02:59
rockstarAlso, AIUI, it still doesn't support separate branches and trees.02:59
wgrantProbably not.02:59
wgrantI don't normally use it either, but that's because it was terribly slow.02:59
wgrantNow it only takes a couple of minutes.02:59
wgrantrockstar: Thanks. Can you land that for me?03:02
rockstarwgrant, yeah.  I'll have to figure out how to do that.03:02
wgrantutilities/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.au03:05
wgrantMaybe throw a ' (wgrant)' just after the '[ui=none]'.03:05
wgrantSome people do, some people don't.03:05
rockstarYea, I'm not sure I need to run that through ec2test.03:06
wgrantTrue.03:06
rockstarI don't see a direct way of doing it with bzr-pqm, so off to ec2 it went.03:07
wgrantOK, thanks.03:07
wgrantI'll leave the big rocketfuel-setup breakage (bug #432830) to somebody more qualified.03:08
mupBug #432830: rocketfuel-setup does not work <Launchpad Foundations:New> <https://launchpad.net/bugs/432830>03:08
rockstarMan, I currently have your ec2 runs going.  My wife is going to think I'm buying crap on Amazon again.03:08
rockstars/your/four/03:08
wgrantHeh.03:08
rockstarwgrant, I killed the ec2 instance and figured out how to just pqm submit it.03:18
wgrantrockstar: Great, thanks!03:52
rockstarwgrant, thanks for your patch.03:52
bachenninge: could i trouble you for a trivial review?  it's about a four line change.12:47
bacif not i'll rs it12:47
henningebac: sure, bring it on12:48
gary_posterhenninge, btw happy late bday :-)12:50
henningegary_poster: thank you ;-)12:50
bachenninge: MP sent.  waiting for it to show up12:54
bachi gary_poster.  do you know about http://www.sparkcon.com/ ?  it sounds fun but i've never been -- all the way in raleigh...12:55
gary_posterbac: lol, no, looks cool!12:56
bachenninge: https://code.edge.launchpad.net/~bac/launchpad/import_error/+merge/1211512:56
* henninge looks12:56
bacgary_poster: some friends of mine started it a few years ago.  it's really grown huge.12:56
gary_posterbac: 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
gary_posteruntil now I mean12:59
bacgary_poster: yeah, i'm thinking i may have to break my 'no raleigh' rule today and wander down there13:00
henningebac: are you keeping the commented-out import for documentation purposes? Is that common practice so that people know what it means?13:00
bachenninge: no, it was an oversight.  i'll remove it13:02
henningebac: ok, r=me then ;-)13:02
bachenninge: danke13:02
henningebitte!13:02
=== henninge is now known as henninge-afk
bachi abentley14:25
abentleybac: Happily, this is only a problem with the test, and yes, I caused it.14:57
bacabentley: great.  if you can get a fix in rs=bac it would be great!14:57
abentleybac: Will do.14:59

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