[MERGE][0.14] Don't check plugins for tabs
John Arbash Meinel
john at arbash-meinel.com
Wed Jan 17 23:37:46 GMT 2007
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Alexander Belchenko wrote:
> John Arbash Meinel ?8H5B:
>>> Etienne just posted this bug:
>>> https://launchpad.net/bzr/+bug/80330
>>>
>>> And I noticed there are 2 bugs in selftest.
>>>
>>> 1) 'tools' only exists in the source tree
>>> 2) test_source.py is checking if plugins have tabs characters in the
>>> source. While I would recommend that they not avoid them, I don't think
>>> we should have tests failing in those circumstances. Attached is a patch
>>> which skips 'utils' and 'plugins', just like the copyright and gpl
>>> license tests.
>>>
>
> something wrong with your patch -- it seems that diff contains also
> fix for problems with annotate.
>
>
Yeah, I was planning on removing that part, and I forget. Ignore it, and
just focus on the following:
> === modified file 'bzrlib/tests/test_source.py'
> --- bzrlib/tests/test_source.py 2006-11-06 21:00:45 +0000
> +++ bzrlib/tests/test_source.py 2007-01-17 22:33:32 +0000
> @@ -258,6 +258,8 @@
> incorrect = []
>
> for fname, text in self.get_source_file_contents():
> + if '/util/' in fname or '/plugins/' in fname:
> + continue
> if '\t' in text:
> incorrect.append(fname)
>
>
>
John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.3 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFFrrNKJdeBCYSNAAMRApeEAJ0U+9rJmuo/LjLU4tOsOwUlrBiGeQCgw8hu
IbJ36LO4SiWbW76lHFmXlRM=
=Kq5V
-----END PGP SIGNATURE-----
More information about the bazaar
mailing list