Re: don't augment PATH in TESTS_ENVIRONMENT

2010-09-04 Thread Jim Meyering
Bruno Haible wrote: >> Good catch. Thanks for doing that. >> Your patch looks right > > OK, applied. I've just pushed this correction. Sorry I didn't catch it in review. >From 3389c116ca7f63bcad9f987cdae5774b1ec2 Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Sat, 4 Sep 2010 17:43:39 +02

Re: don't augment PATH in TESTS_ENVIRONMENT

2010-09-04 Thread Bruno Haible
Hi Jim, > Good catch. Thanks for doing that. > Your patch looks right OK, applied. > The regexp isn't as tight as it could be, but it's > adequate for now Yes, I agree. Bruno

Re: don't augment PATH in TESTS_ENVIRONMENT

2010-09-04 Thread Jim Meyering
Bruno Haible wrote: > It is not possible to portably augment the same environment variable twice > in TESTS_ENVIRONMENT. > > Witness: The shell command >foo=a:$foo foo=b:$foo sh -c 'echo $foo' > prints > b:a: in ksh, OpenBSD /bin/sh > a:b: in Solaris 8 /bin/sh > b:in NetBSD /bin

don't augment PATH in TESTS_ENVIRONMENT

2010-09-04 Thread Bruno Haible
vs.sh: Don't set PATH here. * tests/test-vc-list-files-git.sh: Likewise. Reported by Bruno Haible. 2010-09-04 Bruno Haible Don't augment PATH in TESTS_ENVIRONMENT. * modules/update-copyright-tests (Makefile.am): In TESTS_ENVIRONMENT, set abs_