summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNeels Hofmeyr <neels@hofmeyr.de>2017-10-23 14:57:56 +0200
committerNeels Hofmeyr <nhofmeyr@sysmocom.de>2017-10-24 15:31:07 +0000
commit754e6877cccddd0a279659c32fa594b23369dc61 (patch)
treefcb985b62a5b37b1b196f9dd874fc86f299b475f
parenta568af25d2a8382265b44c9b2b3a72ca504d2443 (diff)
fixups for recent "Cleanup jenkins build scripts"
Change I24e500e132f5c8e8133d35548cb7b4e4552331d0 was merged, but IMHO needs improvement. Fix these: - clean the git source tree before each build step, in common prep_build(). - fix indenting inside the build() macros. - change build() arg to be build_dir, to absorb cleanup steps into build(). - in jenkins.sh, use $ENABLE_SANITIZE as global env, not passed as arg. - in jenkins.sh, don't do 'make distcheck' twice. It is not necessary to do it from source tree as well as separately from source tree, since distcheck already moves to a different build dir. Change-Id: I09d306350602f21943d5bd45f7388c83ede9b524
-rwxr-xr-xcontrib/jenkins-arm.sh21
-rwxr-xr-xcontrib/jenkins.sh26
-rw-r--r--contrib/jenkins_common.sh18
3 files changed, 43 insertions, 22 deletions
diff --git a/contrib/jenkins-arm.sh b/contrib/jenkins-arm.sh
index e0c75fa6..ef116b12 100755
--- a/contrib/jenkins-arm.sh
+++ b/contrib/jenkins-arm.sh
@@ -2,8 +2,13 @@
. $(dirname "$0")/jenkins_common.sh
+src_dir="$PWD"
build() {
- $1 --enable-static \
+ build_dir="$1"
+
+ prep_build "$src_dir" "$build_dir"
+
+ "$src_dir"/configure --enable-static \
--prefix=/usr/local/arm-none-eabi \
--host=arm-none-eabi \
--enable-embedded \
@@ -11,15 +16,11 @@ build() {
--disable-shared \
CFLAGS="-Os -ffunction-sections -fdata-sections -nostartfiles -nodefaultlibs -Werror"
-$MAKE $PARALLEL_MAKE \
- || cat-testlogs.sh
+ $MAKE $PARALLEL_MAKE \
+ || cat-testlogs.sh
}
# verify build in dir other than source tree
-mkdir -p builddir
-cd builddir
-build ../configure
-
-cd ..
-build ./configure
-
+build builddir
+# verify build in source tree
+build .
diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index c397d528..d7315040 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -9,19 +9,23 @@ if [ "x$label" = "xFreeBSD_amd64" ]; then
ENABLE_SANITIZE=""
fi
+src_dir="$PWD"
build() {
- $1 --enable-static $2 CFLAGS="-Werror" CPPFLAGS="-Werror"
-$MAKE $PARALLEL_MAKE check \
- || cat-testlogs.sh
-$MAKE distcheck \
- || cat-testlogs.sh
+ build_dir="$1"
+
+ prep_build "$src_dir" "$build_dir"
+
+ "$src_dir"/configure --enable-static $ENABLE_SANITIZE CFLAGS="-Werror" CPPFLAGS="-Werror"
+ $MAKE $PARALLEL_MAKE check \
+ || cat-testlogs.sh
}
# verify build in dir other than source tree
-mkdir -p builddir
-cd builddir
-build ../configure $ENABLE_SANITIZE
-
-cd ..
-build ./configure $ENABLE_SANITIZE
+build builddir
+# verify build in source tree
+build .
+# do distcheck only once, which is fine from built source tree, since distcheck
+# is well separated from the source tree state.
+$MAKE distcheck \
+ || cat-testlogs.sh
diff --git a/contrib/jenkins_common.sh b/contrib/jenkins_common.sh
index e52a96a9..6cfa3347 100644
--- a/contrib/jenkins_common.sh
+++ b/contrib/jenkins_common.sh
@@ -4,4 +4,20 @@ set -ex
verify_value_string_arrays_are_terminated.py $(find . -name "*.[hc]")
-autoreconf --install --force
+prep_build() {
+ _src_dir="$1"
+ _build_dir="$2"
+
+ cd "$_src_dir"
+
+ # a failed 'make distcheck' may leave files without write permissions
+ chmod -R a+w .
+ git clean -dxf
+ # make absolutely sure no src files have modifications
+ git checkout -f HEAD
+
+ autoreconf --install --force
+
+ mkdir -p "$_build_dir"
+ cd "$_build_dir"
+}