[GOLD] Use "set -e" in test scripts
Rahul Chaudhry via binutils
binutils@sourceware.org
Thu Jan 12 21:35:00 GMT 2017
Hi Cary,
Thanks for your review.
What would be the next steps to get this patch merged?
I am a new contributor, and do not have write access to the repository.
Rahul
On Mon, Jan 9, 2017 at 5:49 PM, Cary Coutant <ccoutant@gmail.com> wrote:
>> Some tests in gold/testsuite/ run awk to check for errors.
>> The awk scripts use "exit 1" to indicate test failure, but
>> a few of those tests don't use "set -e", so the failures
>> get ignored. This patch adds "set -e" to those tests.
>>
>> OK?
>>
>> * testsuite/icf_safe_so_test.sh: Use "set -e".
>> * testsuite/icf_safe_test.sh: Likewise.
>> * testsuite/icf_test.sh: Likewise.
>
> Yes, this is OK. Thanks!
>
> -cary
More information about the Binutils
mailing list