[MERGE] Cleanup BZR_HOME
John Arbash Meinel
john at arbash-meinel.com
Fri Sep 29 20:50:33 BST 2006
So we finally had someone using BZR_HOME, and we found out that it
messes up the test suite.
Specifically, BZR_HOME takes precedence over HOME and APPDATA, so if you
run the test suite and have BZR_HOME set, it causes all tests to share
their configuration. And some tests set invalid configuration
intentionally, to see how bzr complains.
One thing it found is that LockDir.lock_write() *requires* a valid email
address to be set. Which we probably don't want. But that is a different
bug. (It calls config.user_email() which doesn't handle the case when
the email isn't configured correctly).
Anyway, this fixes the test suite to properly sanitize BZR_HOME from the
environment before running any tests:
=== modified file 'bzrlib/tests/__init__.py'
--- bzrlib/tests/__init__.py 2006-09-25 19:29:26 +0000
+++ bzrlib/tests/__init__.py 2006-09-29 19:45:29 +0000
@@ -703,6 +703,7 @@
def _cleanEnvironment(self):
new_env = {
+ 'BZR_HOME': None, # Don't inherit BZR_HOME to all the tests.
'HOME': os.getcwd(),
'APPDATA': os.getcwd(),
'BZR_EMAIL': None,
John
=:->
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 254 bytes
Desc: OpenPGP digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20060929/85e090a1/attachment.pgp
More information about the bazaar
mailing list