From 67bca9963772e85f25307b210e4446fac2625df0 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Michel=20D=C3=A4nzer?= <mdaenzer@redhat.com>
Date: Fri, 2 Sep 2022 11:13:20 +0200
Subject: [PATCH] ci: Move build job script to a separate file

Will make it easier to do more complex shell stuff.

No functional change intended.

v2:
* Use /bin/bash instead of /bin/sh (Peter Hutterer)
* Export environment variables on a separate line (Peter)
* Use "set" command instead of shell command line arguments, for
  consistency with debian-install.sh.
---
 .gitlab-ci.yml               |  4 +---
 .gitlab-ci/build-and-test.sh | 11 +++++++++++
 2 files changed, 12 insertions(+), 3 deletions(-)
 create mode 100755 .gitlab-ci/build-and-test.sh

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 7e436b234..d1b584a57 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -76,9 +76,7 @@ debian-bullseye:
 meson:
     extends: .common-build-and-test
     script:
-        - meson -Dc_args="-fno-common" -Dprefix=/usr -Dxephyr=true -Dwerror=true $MESON_EXTRA_OPTIONS build/
-        - ninja -j${FDO_CI_CONCURRENT:-4} -C build/ dist
-        - PIGLIT_DIR=/root/piglit XTEST_DIR=/root/xts ninja -j${FDO_CI_CONCURRENT:-4} -C build/ test
+        - .gitlab-ci/build-and-test.sh
         - .gitlab-ci/manpages-check
 
 meson-noglamor:
diff --git a/.gitlab-ci/build-and-test.sh b/.gitlab-ci/build-and-test.sh
new file mode 100755
index 000000000..5caf3b129
--- /dev/null
+++ b/.gitlab-ci/build-and-test.sh
@@ -0,0 +1,11 @@
+#!/bin/bash
+
+set -e
+set -o xtrace
+
+meson -Dc_args="-fno-common" -Dprefix=/usr -Dxephyr=true -Dwerror=true $MESON_EXTRA_OPTIONS build/
+
+ninja -j${FDO_CI_CONCURRENT:-4} -C build/ dist
+
+export PIGLIT_DIR=/root/piglit XTEST_DIR=/root/xts
+ninja -j${FDO_CI_CONCURRENT:-4} -C build/ test