[PATCH 1 of 2] run-tests: make --tmpdir option more useful

Greg Ward greg at gerg.ca
Thu Jul 30 14:35:31 CDT 2009


On Thu, Jul 23, 2009 at 12:55 PM, Martin Geisler<mg at lazybytes.net> wrote:
> After thinking about it on my way home, I think it's the --keep-tmpdir
> flag that is unneeded. Without it, we would then have exactly two modes:
>
> * run-test.py picks a random directory and cleans up automatically.
>
> * you pick the directory and you get to clean it up.
>
> It seems like a nice feature to be able to specify a fixed directory for
> the test -- using TMP will merely specify the parent directory for the
> randomly generated test directory.
>
> Greg, what do you think -- you proposed the original patches :-)

Sorry for the long delay.  I've been trying to get work done by hiding
from my email.  ;-)

Anyways... I *think* that the patch I sent (back on July 11) is
actually very close to Martin's idea.  It's just not explicit about
the "two modes" thing.  I'll have to take a second look at the patch
and refresh my memory.

And I should add something to make Augie happy -- say, if you run

  ./run-tests.py --tmpdir /mnt/ramdisk

where /mnt/ramdisk already exists, maybe the error message should be

  error: temp dir /mnt/ramdisk already exists (did you mean to set TMP?)

(with adjustment for platform-specific environment variable: doesn't
Windows call it %TEMPDIR% or something?)

Greg


More information about the Mercurial-devel mailing list