Corrected run_subprocess_command again: it now creates a list (argv) and calls run_bzr with it. Note that the tests did not pick up the error with qcommit because the breezy test suite has a *different* run_bzr to that in the actual code, which means that the tests can happily pass a string (e.g. 'commit -m "some message") whereas the real run_bzr expects a list (e.g. ['commit', '-m', 'some message'])