[MERGE] Fix for Bug 129299, BZR_PLUGIN_PATH should ignore trailiing slashes.
Alexander Belchenko
bialix at ukr.net
Wed Aug 15 08:51:12 BST 2007
Alexander Belchenko has voted resubmit.
Status is now: Resubmit
Comment:
+def strip_trailing_sep(path):
+ return path.rstrip(os.sep)
^-- IMO, it's better to rstrip both \ or /, i.e. path.rstrip('\\/')
because on windows you can use both of them for BZR_PLUGIN_PATH
+ os.environ['BZR_PLUGIN_PATH'] = "first\%s%ssecond\%s"%
(os.sep, os.pathsep, os.sep)
^-- why for backslashes here? (after 'first' and 'second')
For details, see:
http://bundlebuggy.aaronbentley.com/request/%3C46BB4EAE.7050706%40latte.ca%3E
More information about the bazaar
mailing list