From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Mon, 16 Mar 2020 17:16:36 -0400 Subject: [PATCH v2 14/23] travis/gitlab/azure: Drop BUILDMAN variable with test.py In-Reply-To: <20200315234303.18598-11-sjg@chromium.org> References: <20200315234303.18598-1-sjg@chromium.org> <20200315234303.18598-11-sjg@chromium.org> Message-ID: <20200316211636.GG12423@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Sun, Mar 15, 2020 at 05:42:54PM -0600, Simon Glass wrote: > This is not needed in the test.py part of the config, now since we use the > same name as the pytests. > > Drop BUILDMAN, retaining it only for the 'build' parts of the config, i.e. > where we build multiple boards and don't run any tests. > > Signed-off-by: Simon Glass [snip] > diff --git a/.travis.yml b/.travis.yml > index 3149ed6a15..42a3923565 100644 > --- a/.travis.yml > +++ b/.travis.yml > @@ -119,7 +119,7 @@ script: > # From buildman, exit code 129 means warnings only. If we've been asked to > # use clang only do one configuration. > # Build a selection of boards if TEST_PY_BD is empty > - - if [[ "${BUILDMAN}" != "" ]] && [[ "${TEST_PY_BD}" == "" ]]; then > + - if [[ "${BUILDMAN}" != "" ]]; then > ret=0; > tools/buildman/buildman -P -E ${BUILDMAN} ${OVERRIDE}|| ret=$?; > if [[ $ret -ne 0 && $ret -ne 129 ]]; then This looks wrong. We don't set BUILDMAN anymore anywhere, right? So shouldn't this just be the test for TEST_PY_BD is empty? Thanks! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 659 bytes Desc: not available URL: