Merge lp:~statik/pqm/no-dashses-workingdir into lp:pqm

[Elliot Murphy]

[Elliot Murphy (statik)]

This is a simple patch that changes the PQM working directory where tests are run so that it doesn't have leading dashes in one of the path components. This works around a problem we were having with CouchdDB, where tests were failing because of the path with leading dashes.

Yes, we should find and fix this in CouchDB. We will, but this patch was a good way to unblock a lot of engineers, and we are running it in production so I figured might as well go ahead and propose it for merging. It simplifies the code a tiny bit, so it's not a bad cleanup to have.

[Tim Penhey (thumper)]

What would the config contain in the conditional that you removed?

This patch seems fine to me, but I'm unsure as to what the config has.

=== modified file 'pqm/script.py'
--- pqm/script.py	2009-03-26 00:19:45 +0000
+++ pqm/script.py	2009-03-26 19:50:07 +0000
@@ -518,13 +518,8 @@
         branch_name = self._branch_name(branch)
         if not os.access(dirpath, os.W_OK):
             os.mkdir(dirpath)
-        # set revision to the last element of the branch...
-        if config:
-            elements=re.split("/", config)
-            fullpath=os.path.join(dirpath, "%s---%s" % (elements[0], branch_name))
-        else:
-            fullpath=os.path.join(dirpath, branch_name)
 
+        fullpath=os.path.join(dirpath, branch_name)
         if os.access(fullpath, os.W_OK):
             self.script.logger.error("Working dir already exists: " + fullpath)
             raise PQMException(sender, "Working dir already exists: " + fullpath)