Felix Singer submitted this change.

View Change



3 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.

Approvals: build bot (Jenkins): Verified Thomas Heijligen: Looks good to me, approved
test_build.sh: Delete old build dir when Meson is used

Make sure to start from a clean environment when Meson is used. Also,
rename the variable "builddir" to "build_dir" for better readability and
to avoid typos.

Signed-off-by: Felix Singer <felixsinger@posteo.net>
Change-Id: I050ed916685728cbed9c10601a6bcad6d59a4eda
Reviewed-on: https://review.coreboot.org/c/flashrom/+/67541
Reviewed-by: Thomas Heijligen <src@posteo.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
---
M test_build.sh
1 file changed, 24 insertions(+), 4 deletions(-)

diff --git a/test_build.sh b/test_build.sh
index 0a4506a..8f48ba1 100755
--- a/test_build.sh
+++ b/test_build.sh
@@ -37,10 +37,13 @@


build_meson () {
- builddir=out
- meson $builddir -Dtests=enabled
- ninja -C $builddir
- ninja -C $builddir test
+ build_dir=out
+
+ rm -rf ${build_dir}
+
+ meson $build_dir -Dtests=enabled
+ ninja -C $build_dir
+ ninja -C $build_dir test
}



To view, visit change 67541. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I050ed916685728cbed9c10601a6bcad6d59a4eda
Gerrit-Change-Number: 67541
Gerrit-PatchSet: 6
Gerrit-Owner: Felix Singer <felixsinger@posteo.net>
Gerrit-Reviewer: Felix Singer <felixsinger@posteo.net>
Gerrit-Reviewer: Thomas Heijligen <src@posteo.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-MessageType: merged