Merge lp://qastaging/~kalebral-deactivatedaccount/drizzle-automation/fix-lcov2 into lp://qastaging/drizzle-automation

Proposed by Lee Bieber
Status: Merged
Merged at revision: not available
Proposed branch: lp://qastaging/~kalebral-deactivatedaccount/drizzle-automation/fix-lcov2
Merge into: lp://qastaging/drizzle-automation
Diff against target: None lines
To merge this branch: bzr merge lp://qastaging/~kalebral-deactivatedaccount/drizzle-automation/fix-lcov2
Reviewer Review Type Date Requested Status
Jay Pipes Approve
Review via email: mp+8479@code.qastaging.launchpad.net
To post a comment you must log in.
Revision history for this message
Lee Bieber (kalebral-deactivatedaccount) wrote :

missed one location where we need to add --overwrite when pulling code from the repository

Revision history for this message
Jay Pipes (jaypipes) wrote :

Hmm, I thought this was already merged...approved. Will be merged into trunk shortly. :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'drizzle/automation/lib/sandbox.py'
2--- drizzle/automation/lib/sandbox.py 2009-06-25 23:29:35 +0000
3+++ drizzle/automation/lib/sandbox.py 2009-07-09 17:41:56 +0000
4@@ -152,7 +152,7 @@
5 # Check to see if it's a bzr branch.
6 os.chdir(self.revisional_branch_dir)
7
8- (retcode, output)= commands.getstatusoutput("bzr pull -r%s" % self.revno)
9+ (retcode, output)= commands.getstatusoutput("bzr pull -r%s --overwrite" % self.revno)
10 if retcode != 0:
11 logging.error("Sandbox directory \"%s\" exists, but is not a BZR branch! Exiting." % (self.revisional_branch_dir))
12 sys.exit(1)

Subscribers

People subscribed via source and target branches

to all changes: