[GOLD] Propagate error status from awk in tests

Rahul Chaudhry via binutils binutils@sourceware.org
Tue Jan 10 01:25:00 GMT 2017


The tests I was having problem with didn't have "set -e".
Please ignore this patch. I'll send another one to enable
"set -e" in test scripts that have it missing.

Thanks,
Rahul


On Mon, Jan 9, 2017 at 3:14 PM, Ian Lance Taylor <iant@google.com> wrote:
> On Mon, Jan 9, 2017 at 3:05 PM, Rahul Chaudhry via binutils
> <binutils@sourceware.org> wrote:
>> Some tests in gold/testsuite/ run awk to check for errors.
>> The awk scripts use "exit 1" to indicate test failure, but
>> the shell scripts didn't check for the exit status from awk.
>> This patch adds the checks so failing tests actually fail.
>
> In general the shell scripts use `set -e` to pick up failures in
> programs that they run.  Why isn't that working here?
>
> Ian



More information about the Binutils mailing list