coreboot-gerrit November 2022

coreboot-gerrit@coreboot.org

1 participants
7286 discussions

[S] Change in coreboot[master]: util/testing: Call test-tools target from what-jenkins-does
by build bot (Jenkins) (Code Review) 20 Nov '22

20 Nov '22

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69838 ) Change subject: util/testing: Call test-tools target from what-jenkins-does ...................................................................... Patch Set 1: (1 comment) Commit Message: Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-164088): https://review.coreboot.org/c/coreboot/+/69838/comment/099695af_0a464fe2 PS1, Line 11: Possible repeated word: 'the' -- To view, visit https://review.coreboot.org/c/coreboot/+/69838 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Id62d6bb1e729892ec123ea970ca8a31e03a812d0 Gerrit-Change-Number: 69838 Gerrit-PatchSet: 1 Gerrit-Owner: Martin L Roth <gaumless(a)gmail.com> Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sun, 20 Nov 2022 17:12:51 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

[S] Change in coreboot[master]: fix abuild
by Martin L Roth (Code Review) 20 Nov '22

20 Nov '22

Martin L Roth has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/69862 ) Change subject: fix abuild ...................................................................... Abandoned Not meant to be pushed -- To view, visit https://review.coreboot.org/c/coreboot/+/69862 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Id3a2fe397875f6305dc673033142042b23147ba0 Gerrit-Change-Number: 69862 Gerrit-PatchSet: 1 Gerrit-Owner: Martin L Roth <gaumless(a)gmail.com> Gerrit-MessageType: abandon

1 0

[S] Change in coreboot[master]: util/testing: Call test-abuild target from what-jenkins-does
by build bot (Jenkins) (Code Review) 20 Nov '22

20 Nov '22

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69836 ) Change subject: util/testing: Call test-abuild target from what-jenkins-does ...................................................................... Patch Set 1: (1 comment) Commit Message: Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-164086): https://review.coreboot.org/c/coreboot/+/69836/comment/cc6c0616_0b0ddeac PS1, Line 14: Possible repeated word: 'the' -- To view, visit https://review.coreboot.org/c/coreboot/+/69836 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I4552193894c16301defb851eb3db4bdfbfa49803 Gerrit-Change-Number: 69836 Gerrit-PatchSet: 1 Gerrit-Owner: Martin L Roth <gaumless(a)gmail.com> Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sun, 20 Nov 2022 17:12:00 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

[S] Change in coreboot[master]: util/testing: Call test-lint target from what-jenkins-does
by build bot (Jenkins) (Code Review) 20 Nov '22

20 Nov '22

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69835 ) Change subject: util/testing: Call test-lint target from what-jenkins-does ...................................................................... Patch Set 1: (1 comment) Commit Message: Robot Comment from checkpatch (run ID jenkins-coreboot-checkpatch-164085): https://review.coreboot.org/c/coreboot/+/69835/comment/1271f829_7f5f3b50 PS1, Line 12: Possible repeated word: 'the' -- To view, visit https://review.coreboot.org/c/coreboot/+/69835 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I2f90df76126f453fbcd91f4c4af5d784ac2dbe88 Gerrit-Change-Number: 69835 Gerrit-PatchSet: 1 Gerrit-Owner: Martin L Roth <gaumless(a)gmail.com> Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Sun, 20 Nov 2022 17:11:34 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Gerrit-MessageType: comment

1 0

[M] Change in coreboot[master]: util/abuild: Add --name option to set name of abuild run
by Martin L Roth (Code Review) 20 Nov '22

20 Nov '22

Martin L Roth has uploaded a new patch set (#2). ( https://review.coreboot.org/c/coreboot/+/69859 ) Change subject: util/abuild: Add --name option to set name of abuild run ...................................................................... util/abuild: Add --name option to set name of abuild run Previously, the testclass variable was only updated with the chromeos or Kconfig option values, and the output directory and xml file names were updated independently. With the --name option, all of these can be set simultaneously. This also prevents jenkins from seeing clang and gcc tests as the same because the testclass variable wasn't updated. If --name is not set, all behavior is as it was previously. Signed-off-by: Martin Roth <gaumless(a)gmail.com> Change-Id: I8f52779b92d213386a3eb371d1f30ee32ed48b85 --- M util/abuild/abuild 1 file changed, 57 insertions(+), 11 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/59/69859/2 -- To view, visit https://review.coreboot.org/c/coreboot/+/69859 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I8f52779b92d213386a3eb371d1f30ee32ed48b85 Gerrit-Change-Number: 69859 Gerrit-PatchSet: 2 Gerrit-Owner: Martin L Roth <gaumless(a)gmail.com> Gerrit-MessageType: newpatchset

1 0

[S] Change in coreboot[master]: fix abuild
by Martin L Roth (Code Review) 20 Nov '22

20 Nov '22

Martin L Roth has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/69862 ) Change subject: fix abuild ...................................................................... fix abuild Change-Id: Id3a2fe397875f6305dc673033142042b23147ba0 --- M util/abuild/abuild 1 file changed, 11 insertions(+), 2 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/62/69862/1 diff --git a/util/abuild/abuild b/util/abuild/abuild index 270a492..89cc0d7 100755 --- a/util/abuild/abuild +++ b/util/abuild/abuild @@ -11,8 +11,8 @@ #set -x # Turn echo on.... -ABUILD_DATE="Mar 28, 2017" -ABUILD_VERSION="0.10.03" +ABUILD_DATE="Nov 19, 2022" +ABUILD_VERSION="0.11.00" TOP=$PWD -- To view, visit https://review.coreboot.org/c/coreboot/+/69862 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Id3a2fe397875f6305dc673033142042b23147ba0 Gerrit-Change-Number: 69862 Gerrit-PatchSet: 1 Gerrit-Owner: Martin L Roth <gaumless(a)gmail.com> Gerrit-MessageType: newchange

1 0

[S] Change in coreboot[master]: util/testing: Add scanbuild test build to what-jenkins-does
by Martin L Roth (Code Review) 20 Nov '22

20 Nov '22

Martin L Roth has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/69861 ) Change subject: util/testing: Add scanbuild test build to what-jenkins-does ...................................................................... util/testing: Add scanbuild test build to what-jenkins-does This tests building a single target with scanbuild so to make sure that option hasn't been broken. Since it's a different type of build, it hasn't previously been tested with what-jenkins-does. Signed-off-by: Martin Roth <gaumless(a)gmail.com> Change-Id: I8a74dac203f4d38c0cb30a0b64724e6f9095b9dd --- M util/testing/Makefile.inc 1 file changed, 16 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/61/69861/1 diff --git a/util/testing/Makefile.inc b/util/testing/Makefile.inc index f6bd20b..ea35851 100644 --- a/util/testing/Makefile.inc +++ b/util/testing/Makefile.inc @@ -109,6 +109,7 @@ util/lint/lint lint-extended $(JUNIT) test-abuild: + NAME=scanbuild; SCANBUILD_ARGS='-k -plist-html -maxloop 10' util/abuild/abuild -o $(COREBOOT_BUILD_DIR)/$${NAME} $(ABUILD_OPTIONS) -scan-build --target EMULATION_QEMU_X86_Q35 --exitcode --name $${NAME} NAME=gcc-chromeos; util/abuild/abuild -o $(COREBOOT_BUILD_DIR)/$${NAME} $(ABUILD_OPTIONS) -x --name $${NAME} NAME=gcc; util/abuild/abuild -o $(COREBOOT_BUILD_DIR)/$${NAME} $(ABUILD_OPTIONS) --name $${NAME} NAME=clang-chromeos; util/abuild/abuild -o $(COREBOOT_BUILD_DIR)/$${NAME} $(ABUILD_OPTIONS) -L -x --name $${NAME} @@ -143,6 +144,7 @@ test-cleanup: rm -rf $(COREBOOT_BUILD_DIR)/chromeos $(COREBOOT_BUILD_DIR)/default rm -rf $(COREBOOT_BUILD_DIR)/chromeos-clang $(COREBOOT_BUILD_DIR)/default-clang + rm -rf $(COREBOOT_BUILD_DIR)/scanbuild $(MAKE) clean $(foreach tool, $(TOOLLIST), $(MAKE) -C util/$(tool) clean ; ) $(MAKE) -C src/soc/nvidia/tegra124/lp0 clean -- To view, visit https://review.coreboot.org/c/coreboot/+/69861 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I8a74dac203f4d38c0cb30a0b64724e6f9095b9dd Gerrit-Change-Number: 69861 Gerrit-PatchSet: 1 Gerrit-Owner: Martin L Roth <gaumless(a)gmail.com> Gerrit-MessageType: newchange

1 0

[S] Change in coreboot[master]: util/testing: Use new --name argument for abuild
by Martin L Roth (Code Review) 20 Nov '22

20 Nov '22

Martin L Roth has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/69860 ) Change subject: util/testing: Use new --name argument for abuild ...................................................................... util/testing: Use new --name argument for abuild This gets rid of the duplicated directory and xml filename and uses the --name argument to abuild instead, which also updates the test name in the junit xml file. Signed-off-by: Martin Roth <gaumless(a)gmail.com> Change-Id: Ibe538da42280696190b0a7a0c63fd86a63e40214 --- M util/testing/Makefile.inc 1 file changed, 18 insertions(+), 4 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/60/69860/1 diff --git a/util/testing/Makefile.inc b/util/testing/Makefile.inc index a2f035e..f6bd20b 100644 --- a/util/testing/Makefile.inc +++ b/util/testing/Makefile.inc @@ -109,10 +109,10 @@ util/lint/lint lint-extended $(JUNIT) test-abuild: - util/abuild/abuild -o $(COREBOOT_BUILD_DIR)/chromeos $(ABUILD_OPTIONS) -x -X $(top)/abuild-chromeos.xml - util/abuild/abuild -o $(COREBOOT_BUILD_DIR)/default $(ABUILD_OPTIONS) - util/abuild/abuild -o $(COREBOOT_BUILD_DIR)/chromeos-clang $(ABUILD_OPTIONS) -x -X $(top)/abuild-chromeos_clang.xml -L - util/abuild/abuild -o $(COREBOOT_BUILD_DIR)/default-clang $(ABUILD_OPTIONS) -L + NAME=gcc-chromeos; util/abuild/abuild -o $(COREBOOT_BUILD_DIR)/$${NAME} $(ABUILD_OPTIONS) -x --name $${NAME} + NAME=gcc; util/abuild/abuild -o $(COREBOOT_BUILD_DIR)/$${NAME} $(ABUILD_OPTIONS) --name $${NAME} + NAME=clang-chromeos; util/abuild/abuild -o $(COREBOOT_BUILD_DIR)/$${NAME} $(ABUILD_OPTIONS) -L -x --name $${NAME} + NAME=clang; util/abuild/abuild -o $(COREBOOT_BUILD_DIR)/$${NAME} $(ABUILD_OPTIONS) -L --name $${NAME} test-payloads: $(MAKE) -C payloads/libpayload test-configs -j $(CPUS) V=$(V) Q=$(Q) MFLAGS= MAKEFLAGS= $(if $(TEST_NOCCACHE),,CONFIG_LP_CCACHE=y) -- To view, visit https://review.coreboot.org/c/coreboot/+/69860 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Ibe538da42280696190b0a7a0c63fd86a63e40214 Gerrit-Change-Number: 69860 Gerrit-PatchSet: 1 Gerrit-Owner: Martin L Roth <gaumless(a)gmail.com> Gerrit-MessageType: newchange

1 0

[M] Change in coreboot[master]: util/abuild: Add --name option to set name of abuild run
by Martin L Roth (Code Review) 20 Nov '22

20 Nov '22

Martin L Roth has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/69859 ) Change subject: util/abuild: Add --name option to set name of abuild run ...................................................................... util/abuild: Add --name option to set name of abuild run Previously, the testclass variable was only updated with the chromeos or Kconfig option values, and the output directory and xml file names were updated independently. With the --name option, all of these can be set simultaneously. This also prevents jenkins from seeing clang and gcc tests as the same because the testclass variable wasn't updated. If --name is not set, all behavior is as it was previously. Signed-off-by: Martin Roth <gaumless(a)gmail.com> Change-Id: I8f52779b92d213386a3eb371d1f30ee32ed48b85 --- M util/abuild/abuild 1 file changed, 55 insertions(+), 9 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/59/69859/1 diff --git a/util/abuild/abuild b/util/abuild/abuild index 43dd833..270a492 100755 --- a/util/abuild/abuild +++ b/util/abuild/abuild @@ -17,9 +17,15 @@ TOP=$PWD # Where shall we place all the build trees? -TARGET=${COREBOOT_BUILD_DIR:-coreboot-builds} -XMLFILE=$TOP/abuild.xml -REAL_XMLFILE=$XMLFILE +TARGET_DEFAULT=coreboot-builds +TARGET=${COREBOOT_BUILD_DIR:-${TARGET_DEFAULT}} +XML_DEFAULT="$TOP/abuild.xml" +XMLFILE="${XML_DEFAULT}" +REAL_XMLFILE="${XML_DEFAULT}" + +# Name associated with a run of abuild +TESTRUN_DEFAULT=default +TESTRUN="${TESTRUN_DEFAULT}" export KCONFIG_OVERWRITECONFIG=1 @@ -367,7 +373,7 @@ etime=$(perl -e 'print time();' 2>/dev/null || date +%s) duration=$(( etime - stime )) - junit " <testcase classname='board${testclass/#/.}' name='$BUILD_NAME' time='$duration' >" + junit " <testcase classname='${TESTRUN}${testclass/#/.}' name='$BUILD_NAME' time='$duration' >" if [ $MAKE_FAILED -eq 0 ]; then junit "<system-out>" @@ -465,7 +471,7 @@ fi if [ $BUILDENV_CREATED -ne 0 ] || [ $MAINBOARD_OK -ne 0 ] || [ $VENDOR_OK -ne 0 ] || [ $FORCE_ENABLED_CROS -eq 1 ]; then - junit " <testcase classname='board${testclass/#/.}' name='$BUILD_NAME' >" + junit " <testcase classname='${TESTRUN}${testclass/#/.}' name='$BUILD_NAME' >" junit "<failure type='BuildFailed'>" junitfile "$build_dir/config.log" @@ -581,7 +587,7 @@ $0 [-V|--version] $0 [-h|--help] -Options:\n" +Options:\n [-a|--all] Build previously succeeded ports as well [-A|--any-toolchain] Use any toolchain [-b|--board-variant <name>] Build specific board variant under the @@ -596,6 +602,8 @@ [-K|--kconfig <name>] Prepend file to generated Kconfig [-l|--loglevel <num>] Set loglevel [-L|--clang] Use clang on supported arch + [-n|--name] Set build name - also sets xmlfile if not + already set [-o|--outdir <path>] Store build results in path (defaults to $TARGET) [-p|--payloads <dir>] Use payloads in <dir> to build images @@ -663,12 +671,12 @@ # shellcheck disable=SC2086 if [ "${getoptbrand:0:6}" == "getopt" ]; then # Detected GNU getopt that supports long options. - args=$(getopt -l version,verbose,quiet,help,all,target:,board-variant:,payloads:,cpus:,silent,junit,config,loglevel:,remove,prefix:,update,scan-build,ccache,blobs,clang,any-toolchain,clean,clean-somewhat,outdir:,chromeos,xmlfile:,kconfig:,dir:,root:,recursive,checksum:,timeless,exitcode,asserts -o Vvqhat:b:p:c:sJCl:rP:uyBLAzZo:xX:K:d:R:Ie -- "$@") || exit 1 + args=$(getopt -l version,verbose,quiet,help,all,target:,board-variant:,payloads:,cpus:,silent,junit,config,loglevel:,remove,prefix:,update,scan-build,ccache,blobs,clang,any-toolchain,clean,clean-somewhat,outdir:,chromeos,xmlfile:,kconfig:,dir:,root:,recursive,checksum:,timeless,exitcode,asserts,name: -o Vvqhat:b:p:c:sJCl:rP:uyBLAzZo:xX:K:d:R:Ien: -- "$@") || exit 1 eval set -- $args retval=$? else # Detected non-GNU getopt - args=$(getopt Vvqhat:b:p:c:sJCl:rP:uyBLAZzo:xX:K:d:R:Ie "$@") + args=$(getopt Vvqhat:b:p:c:sJCl:rP:uyBLAZzo:xX:K:d:R:Ien: "$@") set -- $args retval=$? fi @@ -762,13 +770,19 @@ -o|--outdir) shift TARGET=$1; shift ;; + -n|--name) shift + TESTRUN=$1 + shift;; -x|--chromeos) shift chromeos=true testclass=chromeos customizing="${customizing}, chromeos" configoptions="${configoptions}CONFIG_CHROMEOS=y\nCONFIG_VBOOT_MEASURED_BOOT=y\n" ;; - -X|--xmlfile) shift; XMLFILE=$1; REAL_XMLFILE=$1; shift;; + -X|--xmlfile) shift + XMLFILE=$1 + REAL_XMLFILE=$1 + shift;; -I|--recursive) shift; recursive=true;; -K|--kconfig) shift testclass="$(basename "$1" | tr '.' '_' )" @@ -782,6 +796,18 @@ *) break;; esac done + +if [[ "${TESTRUN}" != "${TESTRUN_DEFAULT}" ]]; then + unset testclass + if [[ "${XML_UPDATED}" != "${XML_DEFAULT}" ]]; then + XMLFILE="abuild-${TESTRUN}.xml" + REAL_XMLFILE="${XMLFILE}" + fi + if [[ "${TARGET}" == "${TARGET_DEFAULT}" ]]; then + TARGET="${TESTRUN}" + fi +fi + if [ -n "$1" ]; then printf "Invalid option '%s'\n\n" "$1"; myhelp; exit 1; fi -- To view, visit https://review.coreboot.org/c/coreboot/+/69859 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: I8f52779b92d213386a3eb371d1f30ee32ed48b85 Gerrit-Change-Number: 69859 Gerrit-PatchSet: 1 Gerrit-Owner: Martin L Roth <gaumless(a)gmail.com> Gerrit-MessageType: newchange

1 0

[S] Change in coreboot[master]: util/testing: Call test-tools target from what-jenkins-does
by Martin L Roth (Code Review) 20 Nov '22

20 Nov '22

Martin L Roth has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/69838 ) Change subject: util/testing: Call test-tools target from what-jenkins-does ...................................................................... util/testing: Call test-tools target from what-jenkins-does Instead of having duplicate lines in the what-jenkins-does target and the test-tools target, make test-tools from what-jenkins-does. Now there's only one place to update when changing the the call. Signed-off-by: Martin Roth <gaumless(a)gmail.com> Change-Id: Id62d6bb1e729892ec123ea970ca8a31e03a812d0 --- M util/testing/Makefile.inc 1 file changed, 24 insertions(+), 9 deletions(-) git pull ssh://review.coreboot.org:29418/coreboot refs/changes/38/69838/1 diff --git a/util/testing/Makefile.inc b/util/testing/Makefile.inc index 5952393..a2f035e 100644 --- a/util/testing/Makefile.inc +++ b/util/testing/Makefile.inc @@ -95,14 +95,7 @@ cd 3rdparty/intel-sec-tools/ ; go mod vendor cd util/goswid ; go mod vendor $(MAKE) test-abuild - $(foreach tool, $(TOOLLIST), $(MAKE) CPUS=$(CPUS) V=$(V) Q=$(Q) BLD_DIR="util/$(tool)" BLD="$(tool)" MFLAGS= MAKEFLAGS= MAKETARGET= junit.xml; ) - unset COREBOOT_BUILD_DIR;$(MAKE) CPUS=$(CPUS) V=$(V) Q=$(Q) BLD_DIR=payloads/nvramcui BLD=nvramcui MFLAGS= MAKEFLAGS= MAKETARGET=all junit.xml - unset COREBOOT_BUILD_DIR;$(MAKE) CPUS=$(CPUS) V=$(V) Q=$(Q) BLD_DIR=payloads/coreinfo BLD=coreinfo MFLAGS= MAKEFLAGS= MAKETARGET=defaultbuild junit.xml - (cd payloads/libpayload; unset COREBOOT_BUILD_DIR; $(MAKE) $(if $(JENKINS_NOCCACHE),,CONFIG_LP_CCACHE=y) V=$(V) Q=$(Q) junit.xml) -# These targets don't use the standard build system, so we need to provide an xcompile to them. - $(MAKE) xcompile=$(COREBOOT_BUILD_DIR)/xcompile $(COREBOOT_BUILD_DIR)/xcompile - $(MAKE) CPUS=$(CPUS) V=$(V) Q=$(Q) BLD_DIR=src/soc/nvidia/tegra124/lp0 BLD=tegra124_lp0 MFLAGS= MAKEFLAGS=xcompile=$(COREBOOT_BUILD_DIR)/xcompile MAKETARGET=all junit.xml - $(MAKE) CPUS=$(CPUS) V=$(V) Q=$(Q) BLD_DIR=src/soc/nvidia/tegra210/lp0 BLD=tegra120_lp0 MFLAGS= MAKEFLAGS=xcompile=$(COREBOOT_BUILD_DIR)/xcompile MAKETARGET=all junit.xml + $(MAKE) test-tools +$(MAKE) junit.xml-unit-tests COV=1 +(cd payloads/libpayload; unset COREBOOT_BUILD_DIR; $(MAKE) junit.xml-unit-tests COV=1) +(cd payloads/libpayload; unset COREBOOT_BUILD_DIR; $(MAKE) coverage-report COV=1) @@ -132,7 +125,14 @@ test-tools: @echo "Build testing $(TOOLLIST)" - $(foreach tool, $(TOOLLIST), echo "Building $(tool)";export MFLAGS= ;export MAKEFLAGS= ;$(MAKE) -C util/$(tool) all V=$(V) Q=$(Q) || exit 1; ) + $(foreach tool, $(TOOLLIST), echo "Building $(tool)";$(MAKE) CPUS=$(CPUS) V=$(V) Q=$(Q) BLD_DIR="util/$(tool)" BLD="$(tool)" MFLAGS= MAKEFLAGS= MAKETARGET= junit.xml; ) + unset COREBOOT_BUILD_DIR;$(MAKE) CPUS=$(CPUS) V=$(V) Q=$(Q) BLD_DIR=payloads/nvramcui BLD=nvramcui MFLAGS= MAKEFLAGS= MAKETARGET=all junit.xml + unset COREBOOT_BUILD_DIR;$(MAKE) CPUS=$(CPUS) V=$(V) Q=$(Q) BLD_DIR=payloads/coreinfo BLD=coreinfo MFLAGS= MAKEFLAGS= MAKETARGET=defaultbuild junit.xml + (cd payloads/libpayload; unset COREBOOT_BUILD_DIR; $(MAKE) $(if $(JENKINS_NOCCACHE),,CONFIG_LP_CCACHE=y) V=$(V) Q=$(Q) junit.xml) + # The tegra targets don't use the standard build system, so we need to provide an xcompile to them. + $(MAKE) xcompile=$(COREBOOT_BUILD_DIR)/xcompile $(COREBOOT_BUILD_DIR)/xcompile + $(MAKE) CPUS=$(CPUS) V=$(V) Q=$(Q) BLD_DIR=src/soc/nvidia/tegra124/lp0 BLD=tegra124_lp0 MFLAGS= MAKEFLAGS=xcompile=$(COREBOOT_BUILD_DIR)/xcompile MAKETARGET=all junit.xml + $(MAKE) CPUS=$(CPUS) V=$(V) Q=$(Q) BLD_DIR=src/soc/nvidia/tegra210/lp0 BLD=tegra120_lp0 MFLAGS= MAKEFLAGS=xcompile=$(COREBOOT_BUILD_DIR)/xcompile MAKETARGET=all junit.xml @echo "Running gitconfig tests" @for test in $$(find util/gitconfig/test -maxdepth 1 \ -type f -executable); do \ -- To view, visit https://review.coreboot.org/c/coreboot/+/69838 To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings Gerrit-Project: coreboot Gerrit-Branch: master Gerrit-Change-Id: Id62d6bb1e729892ec123ea970ca8a31e03a812d0 Gerrit-Change-Number: 69838 Gerrit-PatchSet: 1 Gerrit-Owner: Martin L Roth <gaumless(a)gmail.com> Gerrit-MessageType: newchange

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

coreboot-gerrit November 2022