From mboxrd@z Thu Jan  1 00:00:00 1970
From: Heiko Schocher <hs@denx.de>
Date: Fri, 21 Oct 2016 09:54:56 +0200
Subject: [U-Boot] [PATCH 7/8] travis-ci: Drop 'TEST_CMD'
In-Reply-To: <1476991566-2989-8-git-send-email-trini@konsulko.com>
References: <1476991566-2989-1-git-send-email-trini@konsulko.com>
 <1476991566-2989-8-git-send-email-trini@konsulko.com>
Message-ID: <5809C9D0.5000404@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hello Tom,

Am 20.10.2016 um 21:26 schrieb Tom Rini:
> We don't need to use TEST_CMD in order to run tests.  We need a BUILDMAN
> and TOOLCHAIN variable to avoid having to duplicate logic or write some
> wrapper function.  But this makes the tests harder as we add more
> complex examples.
>
> Signed-off-by: Tom Rini <trini@konsulko.com>
> ---
>   .travis.yml | 26 ++++++++++----------------
>   1 file changed, 10 insertions(+), 16 deletions(-)

Reviewed-by: Heiko Schocher <hs@denx.de>

bye,
Heiko

>
> diff --git a/.travis.yml b/.travis.yml
> index 8ebcfaf996ab..25b0023d631b 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -57,10 +57,6 @@ before_script:
>     - if [[ "${TOOLCHAIN}" == *xtensa* ]]; then ./tools/buildman/buildman --fetch-arch xtensa ; fi
>
>   script:
> - # the execution sequence for each test
> - - if [[ "${TEST_CMD}" != "" ]]; then
> -     ${TEST_CMD};
> -   fi
>    # Exit code 129 means warnings only.
>    - if [[ "${BUILDMAN}" != "" ]]; then
>        set +e;
> @@ -154,24 +150,22 @@ matrix:
>
>       # QA jobs for code analytics
>       # static code analysis with cppcheck (we can add --enable=all later)
> -    - env:
> -        - TEST_CMD="cppcheck --force --quiet --inline-suppr ."
> +    - script:
> +        - cppcheck --force --quiet --inline-suppr .
>       # search for TODO within source tree
> -    - env:
> -        - TEST_CMD="grep -r TODO ."
> +    - script:
> +        - grep -r TODO .
>       # search for FIXME within source tree
> -    - env:
> -        - TEST_CMD="grep -r FIXME ."
> +    - script:
> +        - grep -r FIXME .
>       # search for HACK within source tree and ignore HACKKIT board
> -    - env:
> -        - TEST_CMD="grep -r HACK . | grep -v HACKKIT"
>         script:
>           - grep -r HACK . | grep -v HACKKIT
>       # some statistics about the code base
> -    - env:
> -        - TEST_CMD="sloccount ."
> +    - script:
> +        - sloccount .
>       # test/py
> -    - env:
> -        - TEST_CMD="./test/py/test.py --bd sandbox --build"
> +    - script:
> +        - ./test/py/test.py --bd sandbox --build
>
>   # TODO make it perfect ;-r
>

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany