Re: [PATCH] test: fix t7001 cp to use POSIX options

2014-04-12 Thread Jens Lehmann
Am 11.04.2014 21:23, schrieb Junio C Hamano: Jeff King p...@peff.net writes: On Fri, Apr 11, 2014 at 01:24:02AM -0700, Kyle J. McKay wrote: Since 11502468 and 04c1ee57 (both first appearing in v1.8.5), the t7001-mv test has used cp -a to perform a copy in several of the tests. However,

[PATCH] test: fix t7001 cp to use POSIX options

2014-04-11 Thread Kyle J. McKay
Since 11502468 and 04c1ee57 (both first appearing in v1.8.5), the t7001-mv test has used cp -a to perform a copy in several of the tests. However, the -a option is not required for a POSIX cp utility and some platforms' cp utilities do not support it. The POSIX equivalent of -a is -R -P -p.

Re: [PATCH] test: fix t7001 cp to use POSIX options

2014-04-11 Thread Jeff King
On Fri, Apr 11, 2014 at 01:24:02AM -0700, Kyle J. McKay wrote: Since 11502468 and 04c1ee57 (both first appearing in v1.8.5), the t7001-mv test has used cp -a to perform a copy in several of the tests. However, the -a option is not required for a POSIX cp utility and some platforms' cp

Re: [PATCH] test: fix t7001 cp to use POSIX options

2014-04-11 Thread Kyle J. McKay
the tests so it's probably fine. --Kyle 8 Subject: [PATCH v2] test: fix t7001 cp to use POSIX options Since 11502468 and 04c1ee57 (both first appearing in v1.8.5), the t7001-mv test has used cp -a to perform a copy in several of the tests. However, the -a option is not required for a POSIX

Re: [PATCH] test: fix t7001 cp to use POSIX options

2014-04-11 Thread Junio C Hamano
Jeff King p...@peff.net writes: On Fri, Apr 11, 2014 at 01:24:02AM -0700, Kyle J. McKay wrote: Since 11502468 and 04c1ee57 (both first appearing in v1.8.5), the t7001-mv test has used cp -a to perform a copy in several of the tests. However, the -a option is not required for a POSIX cp