Re: [PATCH/RFC 0/8] use "git diff" in the test suite

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Johannes Schindelin <Johannes.Schindelin@xxxxxx> writes:

> Now that --no-index is supported by git's diff, we can use much of the 
> test suite to do the job, instead of relying on GNU diff.
>
> This is a bit work-in-progress, since the handling of "-" as a filename is 
> not fleshed out yet, and we really should have the "--quiet" option. But I 
> got side-tracked by the edit-patch-series script, and did not want to hold 
> these patches off any longer.

As you said yourself [6/8], [7/8], and [8/8] are mechanical
conversions, and I do not think it is too much effort involved
on your side to "hold these patches off".

My preference is to apply [1/8] now, cook the result in 'next'
for a few days longer to see if anybody complains or finds
glitches in the diff-ni series so far, push it out to 'master'
and then apply [2/8] and [3/8] while on 'master'.  The remainder
would be forked into a topic branch and cooked in 'pu' and then
'next' as usual.

Also I do not think we would necessarily want [7/8] and [8/8].
It would help with tests on broken distributions whose 'cmp'
misbehaves when the output is redirected to /dev/null, but I do
not think it is a good enough reason to make the tests slower.

-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]