[MERGE][0.14] Don't check plugins for tabs
John Arbash Meinel
john at arbash-meinel.com
Thu Jan 18 14:46:51 GMT 2007
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Alexander Belchenko wrote:
> John Arbash Meinel ?8H5B:
>>> 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)
>
> It's looks very straightforward.
> But what's about item 1?
>
> --
> Alexander
This was just to fix item 2. I'll do a different patch for #1. Sorry for
the confusing. The bug focuses item 1, but it brought up the issue with
item 2.
John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.3 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFFr4hbJdeBCYSNAAMRArBHAKCaxoteG0XJrufYOy6zmp7inzhYowCgxm+l
s+N7VTD05SmD3FD/AO/210I=
=2WZ5
-----END PGP SIGNATURE-----
More information about the bazaar
mailing list