[MERGE] Fix for Bug 129299, BZR_PLUGIN_PATH should ignore trailiing slashes.
Robert Collins
robertc at robertcollins.net
Thu Aug 9 07:09:31 BST 2007
Robert Collins has voted resubmit.
Status is now: Resubmit
Comment:
Couple of things. If load_from_path is now accepting paths with '/',
then you shouldn't need to santise BZR_PLUGINS_PATH - the lower level
check will suffice. I like the test that the path is set from the value
of BZR_PLUGINS_PATH - thank you for adding that. Finally, your test that
load_from_path works with '/' is clearly cargo culted - I suggest you
read through it once and make sure the comments match what your test
actually does - the don't currently.
For details, see:
http://bundlebuggy.aaronbentley.com/request/%3C46B9CB85.1050608%40latte.ca%3E
More information about the bazaar
mailing list