diff mbox series

[05/20] gitlab: Use the -w option for sandbox_spl

Message ID	20200306200644.5.I31d30fb8031fe55a91aa0877c6754ccdf2ca1a50@changeid
State	New
Headers	show List-Id: U-Boot discussion <u-boot.lists.denx.de> From: sjg at chromium.org (Simon Glass) Date: Fri, 6 Mar 2020 20:07:19 -0700 Subject: [PATCH 05/20] gitlab: Use the -w option for sandbox_spl In-Reply-To: <20200307030734.237401-1-sjg@chromium.org> References: <20200307030734.237401-1-sjg@chromium.org> Message-ID: <20200306200644.5.I31d30fb8031fe55a91aa0877c6754ccdf2ca1a50@changeid>
Series	gitlab: Simplify the test script \| expand [00/20] gitlab: Simplify the test script [01/20] sandbox: Add documentation about required/useful packages [02/20] main: Drop show_boot_progress() prototype [03/20] buildman: Document the members of BuilderJob [04/20] bulidman: Add support for a simple build [05/20] gitlab: Use the -w option for sandbox_spl [06/20] azure: Use the -w option for sandbox_spl [07/20] gitlab: Use the --board buildman flag [08/20] gitlab: Drop the BUILDMAN variable [09/20] buildman: Update help for -d [10/20] gitlab: Drop the buildman -d flag [11/20] gitlab: Drop unnecessary if..fi [12/20] gitlab: Use -w flag for all builds [13/20] gitlab: Use bash to avoid needing a_test_which_does_not_exist [14/20] buildman: Allow ignoring warnings in the return code [15/20] gitlab: Use the buildman -W flag [16/20] gitlab: Enable test_handoff [17/20] buildman: Be more selective about which directories to remove [18/20] buildman: Allow building within a subdir of the current dir [19/20] test/py: Use buildman to build U-Boot [20/20] gitlab: Simplify the exit code for test.py

Commit Message

Simon Glass March 7, 2020, 3:07 a.m. UTC

Avoid needing to know about the internal .bm-work directory, by passing
the -w flag to buildman.

Signed-off-by: Simon Glass <sjg at chromium.org>
---

 .gitlab-ci.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Tom Rini March 9, 2020, 5:50 p.m. UTC | #1

On Fri, Mar 06, 2020 at 08:07:19PM -0700, Simon Glass wrote:

> Avoid needing to know about the internal .bm-work directory, by passing
> the -w flag to buildman.
> 
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
> 
>  .gitlab-ci.yml | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

What I'd like to see is GitLab / Azure / Travis changing in-sync with
these steps, rather than one commit for each (and Travis getting out of
sync).  Thanks!

diff mbox series

Patch

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 55943bb3a2..248e0530d2 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -167,10 +167,10 @@  Run binman, buildman, dtoc and patman testsuites:
       virtualenv -p /usr/bin/python3 /tmp/venv;
       . /tmp/venv/bin/activate;
       pip install pyelftools;
-      export UBOOT_TRAVIS_BUILD_DIR=/tmp/.bm-work/sandbox_spl;
+      export UBOOT_TRAVIS_BUILD_DIR=/tmp/sandbox_spl;
       export PYTHONPATH="${UBOOT_TRAVIS_BUILD_DIR}/scripts/dtc/pylibfdt";
       export PATH="${UBOOT_TRAVIS_BUILD_DIR}/scripts/dtc:${PATH}";
-      ./tools/buildman/buildman -o /tmp -P sandbox_spl;
+      ./tools/buildman/buildman -o ${UBOOT_TRAVIS_BUILD_DIR} -w sandbox_spl;
       ./tools/binman/binman --toolpath ${UBOOT_TRAVIS_BUILD_DIR}/tools test;
       ./tools/buildman/buildman -t;
       ./tools/dtoc/dtoc -t;