Martin Roth has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/46802 )
Change subject: util/docker: Add tests to coreboot-sdk build process ......................................................................
util/docker: Add tests to coreboot-sdk build process
This tests some of the basic targets that coreboot-sdk needs to be able to run.
I was running most of these tests manually after creating the sdk image, but adding it into the Dockerfile makes sure they get run.
Signed-off-by: Martin Roth martin@coreboot.org Change-Id: I0d4a2ad82042733a7966edb8ccf927676618977c --- M util/docker/coreboot-sdk/Dockerfile 1 file changed, 30 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/02/46802/1
diff --git a/util/docker/coreboot-sdk/Dockerfile b/util/docker/coreboot-sdk/Dockerfile index 3946814..26078e8 100644 --- a/util/docker/coreboot-sdk/Dockerfile +++ b/util/docker/coreboot-sdk/Dockerfile @@ -110,3 +110,33 @@ ENV SDK_VERSION={{SDK_VERSION}} ENV SDK_COMMIT={{DOCKER_COMMIT}} USER coreboot + +# Test the built image +RUN mkdir -p /tmp/work && \ + cd /tmp/work && \ + git clone https://review.coreboot.org/bios_extract.git && \ + make -C bios_extract && \ + git clone https://review.coreboot.org/memtest86plus.git && \ + make -C memtest86plus && \ + git clone https://review.coreboot.org/flashrom.git && \ + CONFIG_EVERYTHING=yes make -C flashrom && \ + git clone https://review.coreboot.org/em100.git && \ + make -C em100 && \ + git clone https://review.coreboot.org/coreboot.git && \ + make -C coreboot CPUS=$(nproc) test-abuild && \ + \ + make -C coreboot olddefconfig && \ + make -C coreboot all -j && \ + make -C coreboot printall && \ + make -C coreboot filelist && \ + make -C coreboot ctags-project && \ + make -C coreboot cscope-project && \ + \ + make -C coreboot doxygen -j && \ + make -C coreboot test-payloads&& \ + make -C coreboot test-tools -j && \ + make -C coreboot test-lint -j && \ + make -C coreboot test-cleanup -j && \ + make -C coreboot CPUS=$(nproc) crostools && \ + cd && \ + rm -rf /tmp/work/
Patrick Georgi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46802 )
Change subject: util/docker: Add tests to coreboot-sdk build process ......................................................................
Patch Set 1: Code-Review+2
Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/46802 )
Change subject: util/docker: Add tests to coreboot-sdk build process ......................................................................
util/docker: Add tests to coreboot-sdk build process
This tests some of the basic targets that coreboot-sdk needs to be able to run.
I was running most of these tests manually after creating the sdk image, but adding it into the Dockerfile makes sure they get run.
Signed-off-by: Martin Roth martin@coreboot.org Change-Id: I0d4a2ad82042733a7966edb8ccf927676618977c Reviewed-on: https://review.coreboot.org/c/coreboot/+/46802 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Patrick Georgi pgeorgi@google.com --- M util/docker/coreboot-sdk/Dockerfile 1 file changed, 30 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Patrick Georgi: Looks good to me, approved
diff --git a/util/docker/coreboot-sdk/Dockerfile b/util/docker/coreboot-sdk/Dockerfile index 3946814..26078e8 100644 --- a/util/docker/coreboot-sdk/Dockerfile +++ b/util/docker/coreboot-sdk/Dockerfile @@ -110,3 +110,33 @@ ENV SDK_VERSION={{SDK_VERSION}} ENV SDK_COMMIT={{DOCKER_COMMIT}} USER coreboot + +# Test the built image +RUN mkdir -p /tmp/work && \ + cd /tmp/work && \ + git clone https://review.coreboot.org/bios_extract.git && \ + make -C bios_extract && \ + git clone https://review.coreboot.org/memtest86plus.git && \ + make -C memtest86plus && \ + git clone https://review.coreboot.org/flashrom.git && \ + CONFIG_EVERYTHING=yes make -C flashrom && \ + git clone https://review.coreboot.org/em100.git && \ + make -C em100 && \ + git clone https://review.coreboot.org/coreboot.git && \ + make -C coreboot CPUS=$(nproc) test-abuild && \ + \ + make -C coreboot olddefconfig && \ + make -C coreboot all -j && \ + make -C coreboot printall && \ + make -C coreboot filelist && \ + make -C coreboot ctags-project && \ + make -C coreboot cscope-project && \ + \ + make -C coreboot doxygen -j && \ + make -C coreboot test-payloads&& \ + make -C coreboot test-tools -j && \ + make -C coreboot test-lint -j && \ + make -C coreboot test-cleanup -j && \ + make -C coreboot CPUS=$(nproc) crostools && \ + cd && \ + rm -rf /tmp/work/
-
Martin Roth (Code Review) -
Patrick Georgi (Code Review)