{"thread":{"id":"24075","subject":"[PATCH] t/README: document --root option","startedAt":"2010-06-10T18:24:46Z","lastAt":"2010-06-12T07:58:08Z","messageCount":5,"participants":["Thomas Rast","Jeff King","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"143465","messageId":"c5efa5c15498fe562af98b9b16165127bc1690c2.1276193888.git.trast@student.ethz.ch","threadId":"24075","inReplyTo":null,"subject":"[PATCH] t/README: document --root option","fromName":"Thomas Rast","fromEmail":"trast@student.ethz.ch","sentAt":"2010-06-10T18:24:46Z","receivedAt":"2010-06-10T18:24:46Z","isPatch":true,"sender":{"key":"tr@thomasrast.ch","avatar":"https://avatars.githubusercontent.com/u/153510?v=4"},"body":"We've had this option since f423ef5 (tests: allow user to specify\ntrash directory location, 2009-08-09).  Make it easier to look up :-)\n\nSigned-off-by: Thomas Rast <trast@student.ethz.ch>\n---\n t/README |    6 ++++++\n 1 files changed, 6 insertions(+), 0 deletions(-)\n\ndiff --git a/t/README b/t/README\nindex dcd3ebb..0470c14 100644\n--- a/t/README\n+++ b/t/README\n@@ -84,6 +84,12 @@ appropriately before running \"make\".\n \timplied by other options like --valgrind and\n \tGIT_TEST_INSTALLED.\n \n+--root=<directory>::\n+\tUse <directory> as a trash directory to store all temporary\n+\tdata during testing, instead of the t/ directory itself.\n+\tUsing this option with a RAM-based filesystem (such as tmpfs)\n+\tcan massively speed up the test suite.\n+\n You can also set the GIT_TEST_INSTALLED environment variable to\n the bindir of an existing git installation to test that installation.\n You still need to have built this git sandbox, from which various\n-- \n1.7.1.553.ga798e\n"},{"id":"143478","messageId":"20100610214807.GA24555@coredump.intra.peff.net","threadId":"24075","inReplyTo":"c5efa5c15498fe562af98b9b16165127bc1690c2.1276193888.git.trast@student.ethz.ch","subject":"Re: [PATCH] t/README: document --root option","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2010-06-10T21:48:07Z","receivedAt":"2010-06-10T21:48:07Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Thu, Jun 10, 2010 at 08:24:46PM +0200, Thomas Rast wrote:\n\n> We've had this option since f423ef5 (tests: allow user to specify\n> trash directory location, 2009-08-09).  Make it easier to look up :-)\n\nAcked-by: Jeff King <peff@peff.net>\n\n> +--root=<directory>::\n> +\tUse <directory> as a trash directory to store all temporary\n> +\tdata during testing, instead of the t/ directory itself.\n> +\tUsing this option with a RAM-based filesystem (such as tmpfs)\n> +\tcan massively speed up the test suite.\n\nMinor nit: <directory> is not the trash directory, but rather the root\nof all trash directories. I think anybody with half a clue will\nunderstand what it means, though.\n\n-Peff\n"},{"id":"143530","messageId":"7vpqzxgusl.fsf@alter.siamese.dyndns.org","threadId":"24075","inReplyTo":"20100610214807.GA24555@coredump.intra.peff.net","subject":"Re: [PATCH] t/README: document --root option","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2010-06-11T20:48:26Z","receivedAt":"2010-06-11T20:48:26Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jeff King <peff@peff.net> writes:\n\n> On Thu, Jun 10, 2010 at 08:24:46PM +0200, Thomas Rast wrote:\n>\n>> We've had this option since f423ef5 (tests: allow user to specify\n>> trash directory location, 2009-08-09).  Make it easier to look up :-)\n>\n> Acked-by: Jeff King <peff@peff.net>\n>\n>> +--root=<directory>::\n>> +\tUse <directory> as a trash directory to store all temporary\n>> +\tdata during testing, instead of the t/ directory itself.\n>> +\tUsing this option with a RAM-based filesystem (such as tmpfs)\n>> +\tcan massively speed up the test suite.\n>\n> Minor nit: <directory> is not the trash directory, but rather the root\n> of all trash directories. I think anybody with half a clue will\n> understand what it means, though.\n\nThen...\n\n\tCreate \"trash\" directories used to store all temporary data during\n\ttesting under <directory>, instead of the t/ directory.\n\nperhaps?\n"},{"id":"143543","messageId":"20100612013019.GA27604@coredump.intra.peff.net","threadId":"24075","inReplyTo":"7vpqzxgusl.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] t/README: document --root option","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2010-06-12T01:30:20Z","receivedAt":"2010-06-12T01:30:20Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Fri, Jun 11, 2010 at 01:48:26PM -0700, Junio C Hamano wrote:\n\n> >> +--root=<directory>::\n> >> +\tUse <directory> as a trash directory to store all temporary\n> >> +\tdata during testing, instead of the t/ directory itself.\n> >> +\tUsing this option with a RAM-based filesystem (such as tmpfs)\n> >> +\tcan massively speed up the test suite.\n> >\n> > Minor nit: <directory> is not the trash directory, but rather the root\n> > of all trash directories. I think anybody with half a clue will\n> > understand what it means, though.\n> \n> Then...\n> \n> \tCreate \"trash\" directories used to store all temporary data during\n> \ttesting under <directory>, instead of the t/ directory.\n\nYeah, that is much better, IMHO.\n\n-Peff\n"},{"id":"143557","messageId":"201006120958.08607.trast@student.ethz.ch","threadId":"24075","inReplyTo":"20100612013019.GA27604@coredump.intra.peff.net","subject":"Re: [PATCH] t/README: document --root option","fromName":"Thomas Rast","fromEmail":"trast@student.ethz.ch","sentAt":"2010-06-12T07:58:08Z","receivedAt":"2010-06-12T07:58:08Z","isPatch":true,"sender":{"key":"tr@thomasrast.ch","avatar":"https://avatars.githubusercontent.com/u/153510?v=4"},"body":"Jeff King wrote:\n> On Fri, Jun 11, 2010 at 01:48:26PM -0700, Junio C Hamano wrote:\n> > \n> > Then...\n> > \n> > \tCreate \"trash\" directories used to store all temporary data during\n> > \ttesting under <directory>, instead of the t/ directory.\n> \n> Yeah, that is much better, IMHO.\n\nI see you already picked that up.  Thanks!\n\n-- \nThomas Rast\ntrast@{inf,student}.ethz.ch\n"}]}