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 utilities do not support it. > > The POSIX equivalent of -a is -R -P -p. > > Change "cp -a" to "cp -R -P -p" so that the t7001-mv test works > on systems with a cp utility that only implements the POSIX > required set of options and not the "-a" option.
I wonder if the "-R" is the part that we actually care about here. Including the others does not hurt in that case, but using only "-R" would perhaps make it more obvious to a later reader of the code exactly what we are trying to do. -Peff -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
