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

5 messages from 2010-06-10 to 2010-06-12. Participants: Thomas Rast, Jeff King, Junio C Hamano.
Thread: https://gitlist.dev/t/24075

## Thomas Rast, 2010-06-10 18:24

Subject: [PATCH] t/README: document --root option
Message-ID: <c5efa5c15498fe562af98b9b16165127bc1690c2.1276193888.git.trast@student.ethz.ch>
URL: https://gitlist.dev/e/c5efa5c15498fe562af98b9b16165127bc1690c2.1276193888.git.trast%40student.ethz.ch

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

Signed-off-by: Thomas Rast <trast@student.ethz.ch>
---
 t/README |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/t/README b/t/README
index dcd3ebb..0470c14 100644
--- a/t/README
+++ b/t/README
@@ -84,6 +84,12 @@ appropriately before running "make".
 	implied by other options like --valgrind and
 	GIT_TEST_INSTALLED.
 
+--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.
+
 You can also set the GIT_TEST_INSTALLED environment variable to
 the bindir of an existing git installation to test that installation.
 You still need to have built this git sandbox, from which various
-- 
1.7.1.553.ga798e

```

## Jeff King, 2010-06-10 21:48

Subject: Re: [PATCH] t/README: document --root option
Message-ID: <20100610214807.GA24555@coredump.intra.peff.net>
URL: https://gitlist.dev/e/20100610214807.GA24555%40coredump.intra.peff.net
In-Reply-To: <c5efa5c15498fe562af98b9b16165127bc1690c2.1276193888.git.trast@student.ethz.ch>

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

> 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 <peff@peff.net>

> +--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.

-Peff

```

## Junio C Hamano, 2010-06-11 20:48

Subject: Re: [PATCH] t/README: document --root option
Message-ID: <7vpqzxgusl.fsf@alter.siamese.dyndns.org>
URL: https://gitlist.dev/e/7vpqzxgusl.fsf%40alter.siamese.dyndns.org
In-Reply-To: <20100610214807.GA24555@coredump.intra.peff.net>

```
Jeff King <peff@peff.net> writes:

> On Thu, Jun 10, 2010 at 08:24:46PM +0200, Thomas Rast wrote:
>
>> 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 <peff@peff.net>
>
>> +--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?

```

## Jeff King, 2010-06-12 01:30

Subject: Re: [PATCH] t/README: document --root option
Message-ID: <20100612013019.GA27604@coredump.intra.peff.net>
URL: https://gitlist.dev/e/20100612013019.GA27604%40coredump.intra.peff.net
In-Reply-To: <7vpqzxgusl.fsf@alter.siamese.dyndns.org>

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

> >> +--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

```

## Thomas Rast, 2010-06-12 07:58

Subject: Re: [PATCH] t/README: document --root option
Message-ID: <201006120958.08607.trast@student.ethz.ch>
URL: https://gitlist.dev/e/201006120958.08607.trast%40student.ethz.ch
In-Reply-To: <20100612013019.GA27604@coredump.intra.peff.net>

```
Jeff King wrote:
> On Fri, Jun 11, 2010 at 01:48:26PM -0700, Junio C Hamano wrote:
> > 
> > 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

```