SPEC-2962: move gen_info before the caller 94/24294/1
authorCorentin LABBE <clabbe@baylibre.com>
Thu, 2 Apr 2020 08:26:02 +0000 (10:26 +0200)
committerCorentin LABBE <clabbe@baylibre.com>
Thu, 2 Apr 2020 08:27:39 +0000 (10:27 +0200)
The gen_info() function is declared after the caller, this cannot work.
Let's fix that

Change-Id: I410dcd59bba1d428edbcdf4f997ecdf35ad23af4
Bug-AGL: SPEC-2962
Signed-off-by: Corentin LABBE <clabbe@baylibre.com>
jjb/common/include-agl-run-test-prepare.sh

index cb5be40..1c032f7 100644 (file)
@@ -20,51 +20,6 @@ if [ -e releng-scripts/utils ]; then
     RELENG=$PWD/releng-scripts
 fi
 
-# WARNING: we need gen_info in last, the device of the current JENKINS CIBT
-# example: for x86, the JENKINS CIBT job is for "qemu" and produce artifacts for all x86 boards.
-# so we need to gen_info for qemux86 and upsquare, this content will be used to copy artifacts just after.
-# but the LAVA job need to be generated only for qemu. So we need to have lava_device and releng_device for it.
-export BOARDOUT=$(mktemp --directory)
-case $MACHINE in
-qemux86-64)
-       lava_device=upsquare
-       releng_device=upsquare
-       gen_info || exit 0
-       lava_device=qemu
-       releng_device=qemux86-64
-       gen_info || exit 0
-;;
-raspberrypi4)
-       lava_device=bcm2711-rpi-4-b
-       releng_device=raspberrypi4
-       gen_info || exit 0
-;;
-qemuarm)
-       lava_device=qemu
-       releng_device=qemuarm
-       gen_info || exit 0
-;;
-qemuarm64)
-       lava_device=qemu
-       releng_device=qemuarm64
-       gen_info || exit 0
-;;
-h3ulcb-nogfx)
-       lava_device=r8a7795-h3ulcb-kf
-       releng_device=h3ulcb-kf
-       gen_info || exit 0
-;;
-upsquare)
-       lava_device=upsquare
-       releng_device=upsquare
-       # NO gen_info since, we use artifacts of the qemux86_64 CIBT
-;;
-*)
-       echo "ERROR: unknown machine $MACHINE"
-       exit 0
-;;
-esac
-
 #gen board info in $BOARDOUT
 gen_info() {
        if [ -z "${lava_device}" -o -z "${releng_device}" ]; then
@@ -113,5 +68,50 @@ gen_info() {
        return 0
 }
 
+# WARNING: we need gen_info in last, the device of the current JENKINS CIBT
+# example: for x86, the JENKINS CIBT job is for "qemu" and produce artifacts for all x86 boards.
+# so we need to gen_info for qemux86 and upsquare, this content will be used to copy artifacts just after.
+# but the LAVA job need to be generated only for qemu. So we need to have lava_device and releng_device for it.
+export BOARDOUT=$(mktemp --directory)
+case $MACHINE in
+qemux86-64)
+       lava_device=upsquare
+       releng_device=upsquare
+       gen_info || exit 0
+       lava_device=qemu
+       releng_device=qemux86-64
+       gen_info || exit 0
+;;
+raspberrypi4)
+       lava_device=bcm2711-rpi-4-b
+       releng_device=raspberrypi4
+       gen_info || exit 0
+;;
+qemuarm)
+       lava_device=qemu
+       releng_device=qemuarm
+       gen_info || exit 0
+;;
+qemuarm64)
+       lava_device=qemu
+       releng_device=qemuarm64
+       gen_info || exit 0
+;;
+h3ulcb-nogfx)
+       lava_device=r8a7795-h3ulcb-kf
+       releng_device=h3ulcb-kf
+       gen_info || exit 0
+;;
+upsquare)
+       lava_device=upsquare
+       releng_device=upsquare
+       # NO gen_info since, we use artifacts of the qemux86_64 CIBT
+;;
+*)
+       echo "ERROR: unknown machine $MACHINE"
+       exit 0
+;;
+esac
+
 # echo NEXT is rsync
 #exit 0