build_examples: Small cleanups

* Override MAKEFLAGS via make not via shell variables
* Remove build_examples in gitlab-ci
* Don't use mktemp for any logs

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index d22f55af2b45fc1ea2d1a264bc561ebd6cc3100e..56664145b0deb4cf2879956407cf35beb3e75035 100644 (file)
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -142,6 +142,7 @@ build_esp_idf_tests:
   script:
     # it's not possible to build 100% out-of-tree and have the "artifacts"
     # mechanism work, but this is the next best thing
+    - rm -rf build_examples
     - mkdir build_examples
     - cd build_examples
     # build some of examples

diff --git a/tools/ci/build_examples.sh b/tools/ci/build_examples.sh
index 8cdb05ae599788baa18af9d297583a6732016150..368c1efa89577a38e65c45d7bb99fb17516043f5 100755 (executable)
--- a/tools/ci/build_examples.sh
+++ b/tools/ci/build_examples.sh
@@ -44,6 +44,8 @@ die() {
 
 [ -z ${IDF_PATH} ] && die "IDF_PATH is not set"
 
+echo "build_examples running in ${PWD}"
+
 # only 0 or 1 arguments
 [ $# -le 1 ] || die "Have to run as $(basename $0) [<JOB_NAME>]"
 
@@ -55,7 +57,7 @@ RESULT=0
 FAILED_EXAMPLES=""
 
 RESULT_WARNINGS=22  # magic number result code for "warnings found"
-LOG_WARNINGS=$(mktemp -t example_all.XXXX.log)
+LOG_WARNINGS=${PWD}/build_warnings.log
 
 if [ $# -eq 0 ]
 then
@@ -114,7 +116,7 @@ build_example () {
         # build non-verbose first
         local BUILDLOG=${PWD}/examplebuild.${ID}.log
         (
-            MAKEFLAGS= make clean defconfig &> >(tee -a "${BUILDLOG}") &&
+            make MAKEFLAGS= clean defconfig &> >(tee -a "${BUILDLOG}") &&
             make all &> >(tee -a "${BUILDLOG}")
         ) || {
             RESULT=$?; FAILED_EXAMPLES+=" ${EXAMPLE_NAME}"