Launchpad itself

Merge lp:~jtv/launchpad/bug-535661 into lp:launchpad

  1. bug-535661
  2. Merge into devel
Proposed by Jeroen T. Vermeulen on 2010-03-10
Status: Merged
Approved by: Muharem Hrnjadovic on 2010-03-11
Approved revision: no longer in the source branch.
Merged at revision: not available
Proposed branch: lp:~jtv/launchpad/bug-535661
Merge into: lp:launchpad
Diff against target: 9 lines (+1/-1)
1 file modified
utilities/start-dev-soyuz.sh (+1/-1)
To merge this branch: bzr merge lp:~jtv/launchpad/bug-535661
Reviewer Review Type Date Requested Status
Muharem Hrnjadovic (community) code 2010-03-10 Approve on 2010-03-11
Review via email: mp+21045@code.launchpad.net

Commit Message

Don't -e the soyuz manual startup script.

Description of the Change

= Bug 535661 =

This is a little helper for manual Soyuz/Buildfarm testing that starts up a few daemons. It often fails because one or more of the daemons are already running.

Removing the -e option will make it plod on regardless. This is not a sophisticated script, and the fix will almost always do the right thing.

There are no tests for this script; it applies to local dev systems only and running it in tests would wreak havoc with the environment.

Jeroen

To post a comment you must log in.
Muharem Hrnjadovic (al-maisan) :
review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'utilities/start-dev-soyuz.sh'
2--- utilities/start-dev-soyuz.sh 2010-03-02 08:12:55 +0000
3+++ utilities/start-dev-soyuz.sh 2010-03-10 12:16:25 +0000
4@@ -1,4 +1,4 @@
5-#!/bin/sh -e
6+#!/bin/sh
7 # Start up Soyuz for local testing on a dev machine.
8
9 start_twistd() {