[darcs-users] darcs patch: Make the ./tests/issue1465_ortryrunning.sh a little mo...

Trent W. Buck twb at cybersource.com.au
Thu Sep 10 04:51:30 UTC 2009


Petr Rockai <me at mornfall.net> writes:

> this should fix the failure on Nicolas' buildslave (where I noticed
> DARCS_EDITOR leaks into the test environment, which is something we
> should cater for -- either by explicitly guarding in the tests like
> this patch, or by filtering the environment more vigorously in
> ShellHarness (which is probably somewhat more robust -- patches
> welcome, I suppose).
>
> -not env -u TERM VISUAL=false </dev/null &>R/log \
> +not env -u TERM DARCS_EDITOR=false VISUAL=false </dev/null &>R/log \

In such cases I typically use env -i and then explicitly propagate only
those environment variables I *know* are needed.  I expect the
equivalent of env -i would go in ShellHarness.



More information about the darcs-users mailing list