[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: bug#10021: [PATCH id] Add error-checking on GNU
From: |
Jim Meyering |
Subject: |
Re: bug#10021: [PATCH id] Add error-checking on GNU |
Date: |
Mon, 14 Nov 2011 08:22:28 +0100 |
Ludovic Courtès wrote:
> Jim Meyering <jim@meyering.net> skribis:
>
>> Oh, I also had to make the new test script executable,
>> or else "make check" would fail.
>
> It works for me even if it’s not executable when I run “make check
> TESTS=id/gnu-zero-uids”, apparently because it’s run this way:
>
> execve("/bin/sh", ["/bin/sh", "./shell-or-perl", "--test-name",
> "id/gnu-zero-uids", "--srcdir", ".", "--shell", "/bin/sh", "--perl",
> "perl", "--", "./id/gnu-zero-uids"], [/* 96 vars */]) = 0
The test itself runs either way. However there's a make check
sub-test (vc_exe_in_TESTS) that ensures all such scripts are
consistently chmod a+x. It was failing.
> It might be different when running just “make check”.
>
> (I tested it on GNU/Linux and GNU/Hurd.)
>
>> Since I've made so many changes, I'll wait to hear back
>> from you before pushing it.
>
> Looks good to me, thank you!
>
>> + test "$(uname)" = GNU \
>
> So you’re assuming Bash, right?
No. The test starts with the usual invocation of init.sh,
which ensures we're using a shell that handles $(...).
- Re: bug#10021: [PATCH id] Add error-checking on GNU, (continued)
- Re: bug#10021: [PATCH id] Add error-checking on GNU, Jim Meyering, 2011/11/15
- Re: bug#10021: [PATCH id] Add error-checking on GNU, Paul Eggert, 2011/11/15
- Re: bug#10021: [PATCH id] Add error-checking on GNU, Paul Eggert, 2011/11/16
- Re: bug#10021: [PATCH id] Add error-checking on GNU, Paul Eggert, 2011/11/20
- Re: bug#10021: [PATCH id] Add error-checking on GNU, Jim Meyering, 2011/11/13
- Re: bug#10021: [PATCH id] Add error-checking on GNU, Ludovic Courtès, 2011/11/13
- Re: bug#10021: [PATCH id] Add error-checking on GNU,
Jim Meyering <=