default plugin path

Aaron Bentley aaron at aaronbentley.com
Tue Jan 27 03:12:22 GMT 2009


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Robert Collins wrote:
> I *do not* want to load all plugins.

Yeah, I got that partway through the email.  I thought I'd deleted that.

> 
>>> which is a dead chicken.
>> If you mean it's voodoo, I disagree.  It's reasonably easy to understand.
> 
> Its easy to understand but its replicating logic that in the common case
> isn't needed.

There is no common case.

>>> What harmful effect will setting the path have?
>> It will cause bzrlib clients to load bzr plugins from the system path,
>> which will be incompatible with their local version of bzrlib.
> 
> If I put a patch forward that doesn't spuriously add the system python
> to the path

Adding the system python to the path is *right* for many cases.  It's
also *wrong* for many cases.  Therefore the client should decide.

> (there is a bug in the path setting at the moment affecting
> normal bzr as well as bzr clients), what harmful effects would setting
> the path by default at module load have?

It would be setting bzrlib to load plugins that the client author hasn't
requested.

>> Launchpad and Bundle Buggy have both been hit by bad default plugin
>> paths, even though Launchpad tried to control the plugin path.
> 
> I know that - but I'm not proposing to *load plugins*, only to set the
> path.

Which would mean that the wrong plugins would be loaded if we used the
"from foo import bar" method.

> And both can set it to whatever they like *anyway*.

Yeah, but they need to remember to do that.  These are the sort of bad
defaults that look right up until the point that stuff starts breaking
for all users.

Aaron
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkl+e5MACgkQ0F+nu1YWqI3r9QCfeqUqXvwCkYwD96I12q3SkGvj
MfEAoIkljb7nGnw/sChKQS4ABXClN3r+
=PtJy
-----END PGP SIGNATURE-----



More information about the bazaar mailing list