Hi Ilya,

On Tue, Apr 16, 2019 at 11:17:49PM +0500, ???? ??????? wrote:
> +env:
> +  global:
> +    - USE_THREAD=1
> +    - USE_OPENSSL=1
> +    - USE_PCRE=1
> +    - USE_ZLIB=1 
> +    - USE_GETADDRINFO=1

It's unclear to me how these ones are supposed to be used by the Makefile
considering that they are not passed on the command line; thus the target's
default values will still apply.

> +
> +matrix:
> +  include:
> +#  - os: linux
> +#    compiler: gcc
> +#    env: TARGET=linux2628
> +  - os: linux
> +    compiler: clang
> +    env: TARGET=linux2628
> +#  - os: osx
> +#    compiler: clang
> +#    env: TARGET=osx SSL_LIB=/usr/local/opt/openssl/lib 
> SSL_INC=/usr/local/opt/openssl/include TMPDIR=/var/tmp
> +
> +install:
> +  - git clone https://github.com/VTest/VTest.git
> +  - make -C VTest

I've seen on your previous test report page that vtest fails to build
with the gcc version there due to some zealous checks on the return
value of "nice()" which expect to be verified in the include files.
I'd suggest to simply force the build options so that it's never an
issue and that we don't fail on other uninteresting errors, since
vtest is a utility and not the primary component we're testing :

   - make -C VTest FLAGS='-O2 -s -Wall -Wno-unused-result'

> +script:
> +  - make CC=$CC V=1 TARGET=$TARGET

So here you should pass your global options, since by default they will
not be used if explicitly disabled, or not be disabled if explicitly
enabled :

 - make CC=$CC V=1 TARGET=$TARGET USE_THREAD=$USE_THREAD 
USE_OPENSSL=$USE_OPENSSL USE_PCRE=$USE_PCRE USE_ZLIB=$USE_ZLIB 
USE_GETADDRINFO=$USE_GETADDRINFO

> +  - export PATH=${PATH}:${PWD}/VTest
> +  - export VTEST_PROGRAM="VTest/vtest -v" # "VTest/vtest -v"
> +  - make reg-tests

I'd suggest to do it this way to limit the number of useless output lines
in the test report :

  - make reg-tests PATH=${PATH}:${PWD}/VTest VTEST_PROGRAM="VTest/vtest -v"

Regards,
Willy

Reply via email to