[Libclc-dev] [PATCH 1/1] .travis: Make sure we report failure even if only earlier checked files fail

[Jeroen Ketema via Libclc-dev]

> On 8 Oct 2017, at 21:21, Jan Vesely <jan.vesely at rutgers.edu> wrote:
> 
> On Sun, 2017-10-08 at 21:07 +0200, Jeroen Ketema wrote:
>> Any specific reason for using the value 2?
> 
> I thought it was what the check script returns, but it returns 1.
> I'll change  this to ret=1. and also replace '[' with an explicit
> 'test' to make the script more readable. Do you want me to repost?

No need.

Jeroen

> 
>> 
>> Otherwise, LGTM
> 
> thanks,
> Jan
> 
>> 
>> -- Jeroen
>> 
>>> On 8 Oct 2017, at 12:01, Jan Vesely via Libclc-dev <libclc-dev at lists.llvm.org> wrote:
>>> 
>>> for loop would only report status of the last command
>>> 
>>> Signed-off-by: Jan Vesely <jan.vesely at rutgers.edu>
>>> ---
>>> .travis.yml | 8 +++++---
>>> 1 file changed, 5 insertions(+), 3 deletions(-)
>>> 
>>> diff --git a/.travis.yml b/.travis.yml
>>> index 2f058a7..5f5de72 100644
>>> --- a/.travis.yml
>>> +++ b/.travis.yml
>>> @@ -55,6 +55,8 @@ matrix:
>>> 
>>> script:
>>>  - $PYTHON ./configure.py --with-llvm-config=$LLVM_CONFIG --with-cxx-compiler=$CXX && make -j4
>>> -  - for f in $CHECK_FILES; do
>>> -        ./check_external_calls.sh built_libs/$f;
>>> -    done
>>> +  - ret=0;
>>> +    for f in $CHECK_FILES; do
>>> +        ./check_external_calls.sh built_libs/$f || ret=2;
>>> +    done;
>>> +    [ $ret -eq 0 ]
>>> -- 
>>> 2.13.6
>>> 
>>> _______________________________________________
>>> Libclc-dev mailing list
>>> Libclc-dev at lists.llvm.org
>>> http://lists.llvm.org/cgi-bin/mailman/listinfo/libclc-dev
>> 
>> 
> 
> -- 
> Jan Vesely <jan.vesely at rutgers.edu <mailto:jan.vesely at rutgers.edu>>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/libclc-dev/attachments/20171008/045d91b8/attachment.html>