Re: [PATCH] t/README: document --root option

Jeff King [off-list ref] writes:

On Thu, Jun 10, 2010 at 08:24:46PM +0200, Thomas Rast wrote:

quoted

We've had this option since f423ef5 (tests: allow user to specify
trash directory location, 2009-08-09).  Make it easier to look up :-)

Acked-by: Jeff King <redacted>

quoted

+--root=<directory>::
+	Use <directory> as a trash directory to store all temporary
+	data during testing, instead of the t/ directory itself.
+	Using this option with a RAM-based filesystem (such as tmpfs)
+	can massively speed up the test suite.

Minor nit: <directory> is not the trash directory, but rather the root
of all trash directories. I think anybody with half a clue will
understand what it means, though.

Then...

	Create "trash" directories used to store all temporary data during
	testing under <directory>, instead of the t/ directory.

perhaps?

On Fri, Jun 11, 2010 at 01:48:26PM -0700, Junio C Hamano wrote:

quoted

quoted

+--root=<directory>::
+	Use <directory> as a trash directory to store all temporary
+	data during testing, instead of the t/ directory itself.
+	Using this option with a RAM-based filesystem (such as tmpfs)
+	can massively speed up the test suite.

Minor nit: <directory> is not the trash directory, but rather the root
of all trash directories. I think anybody with half a clue will
understand what it means, though.

Then...

	Create "trash" directories used to store all temporary data during
	testing under <directory>, instead of the t/ directory.

Yeah, that is much better, IMHO.

-Peff

Jeff King wrote:

On Fri, Jun 11, 2010 at 01:48:26PM -0700, Junio C Hamano wrote:

quoted

Then...

	Create "trash" directories used to store all temporary data during
	testing under <directory>, instead of the t/ directory.

Yeah, that is much better, IMHO.

I see you already picked that up.  Thanks!

-- 
Thomas Rast
trast@{inf,student}.ethz.ch