[PATCH v4 0/6] Move --no-exclude to --exclude=(true|false|flag)

2012-04-07 Thread Mark Walters
This is version 4 of the exclude= patch set. Version 3 was at [1]. The only changes relative to version 3 are in the tests. As suggested by jrollins [2] I have added some systematic count and show tests and moved all the exclude tests to the file test/excludes. Best wishes Mark [1] id:"13337165

[PATCH v4 0/6] Move --no-exclude to --exclude=(true|false|flag)

2012-04-07 Thread Mark Walters
This is version 4 of the exclude= patch set. Version 3 was at [1]. The only changes relative to version 3 are in the tests. As suggested by jrollins [2] I have added some systematic count and show tests and moved all the exclude tests to the file test/excludes. Best wishes Mark [1] id:"13337165

[PATCH v4 0/6] Move --no-exclude to --exclude=(true|false|flag)

2012-04-07 Thread Austin Clements
LGTM. If for some reason you roll another version, you should update the commit message on the test patch, but that's obviously not enough to hold this series up. Quoth Mark Walters on Apr 07 at 5:10 pm: > This is version 4 of the exclude= patch set. Version 3 was at [1]. > > The only changes r

[PATCH v4 0/6] Move --no-exclude to --exclude=(true|false|flag)

2012-04-07 Thread Jameson Graef Rollins
On Sat, Apr 07 2012, Mark Walters wrote: > This is version 4 of the exclude= patch set. Version 3 was at [1]. > > The only changes relative to version 3 are in the tests. As suggested > by jrollins [2] I have added some systematic count and show tests and > moved all the exclude tests to the file

[PATCH v4 0/6] Move --no-exclude to --exclude=(true|false|flag)

2012-04-07 Thread David Bremner
Mark Walters writes: > This is version 4 of the exclude= patch set. Version 3 was at [1]. > > The only changes relative to version 3 are in the tests. As suggested > by jrollins [2] I have added some systematic count and show tests and > moved all the exclude tests to the file test/excludes. Pus

Re: [PATCH v4 0/6] Move --no-exclude to --exclude=(true|false|flag)

2012-04-07 Thread Austin Clements
LGTM. If for some reason you roll another version, you should update the commit message on the test patch, but that's obviously not enough to hold this series up. Quoth Mark Walters on Apr 07 at 5:10 pm: > This is version 4 of the exclude= patch set. Version 3 was at [1]. > > The only changes r

Re: [PATCH v4 0/6] Move --no-exclude to --exclude=(true|false|flag)

2012-04-07 Thread Jameson Graef Rollins
On Sat, Apr 07 2012, Mark Walters wrote: > This is version 4 of the exclude= patch set. Version 3 was at [1]. > > The only changes relative to version 3 are in the tests. As suggested > by jrollins [2] I have added some systematic count and show tests and > moved all the exclude tests to the file

Re: [PATCH v4 0/6] Move --no-exclude to --exclude=(true|false|flag)

2012-04-07 Thread David Bremner
Mark Walters writes: > This is version 4 of the exclude= patch set. Version 3 was at [1]. > > The only changes relative to version 3 are in the tests. As suggested > by jrollins [2] I have added some systematic count and show tests and > moved all the exclude tests to the file test/excludes. Pus