shelf_ui import bzrtools even when --no-plugins is required

[Vincent Ladeuil]

...and also make the test suite fails.

Try the following for ~fun :
bzr selftest --no-plugins bzrtools --list

Which by definition should list 0 tests...

It's due to shelf_ui importing bzrlib.plugins.bzrtools.

I think it's a bad idea to have bzr depends on an external plugin (whatever the reason).

So far, plugins can well... plug into bzr when they want to offer additional features.

bzr can import additional libraries for its own needs, but none will plug into bzr back.

I'll try to propose a patch to both bzr core and bzrtools asap,

In the mean time commenting offending import line is an effective work-around

[Matthew Fuller (fullermd)]

Bug 296125 relates to one place this is used.

[Vincent Ladeuil (vila)]

The bzrtools related part of the fix: add colordiff support if bzr seems to provides shelf_ui.

fullermd: *you* don't want that patch if my other fix is applied to bzr :)

[Aaron Bentley (abentley)]

This approach makes it *harder* to fix bug 296125, because if any plugin can supply shelf_ui.diff_writer, there's no way to know whether it produces colourized output.

[John A Meinel (jameinel)]

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

Aaron Bentley wrote:
> This approach makes it *harder* to fix bug 296125, because if any plugin
> can supply shelf_ui.diff_writer, there's no way to know whether it
> produces colourized output.
>

Why not change the DiffWriter interface to include:
"allow_colored=True/False".

Or alternatively, have the disable code just unilaterally disable the
non-native diff_writer. For the common case that a plugin is providing
colordiff, then it will have disabled coloring. For other cases, we
don't have to provide another option to disable whatever wackyness that
plugin provides.

John
=:->

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

iEYEARECAAYFAkkaKYsACgkQJdeBCYSNAAPWLACgwKB91RlhfkkEpHGGM2xqVD8I
UFkAniSUumjroB0fmmP60CGcAjF8BOlY
=WJAK
-----END PGP SIGNATURE-----