From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Tue, 17 Mar 2020 12:42:23 -0400 Subject: [PATCH v2 14/23] travis/gitlab/azure: Drop BUILDMAN variable with test.py In-Reply-To: References: <20200315234303.18598-1-sjg@chromium.org> <20200315234303.18598-11-sjg@chromium.org> <20200316211636.GG12423@bill-the-cat> Message-ID: <20200317164223.GT12423@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 Tue, Mar 17, 2020 at 10:29:42AM -0600, Simon Glass wrote: > Hi Tom, > > On Mon, 16 Mar 2020 at 15:16, Tom Rini wrote: > > > > 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! > > BUILDMAN is still set for the non-test builds, i.e. when we build a > lot of boards and don't run any tests. > > It is empty of the test build, but still used. Ah, OK, thanks. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 659 bytes Desc: not available URL: