libosmocore[master]: Cleanup jenkins build scripts
gerrit-no-reply at lists.osmocom.org
Sun Oct 22 19:29:54 UTC 2017
Patch Set 4:
Line 9: * reorder builds to avoid rm -rf invocation
Line 10: * avoid useless double autoreconf
the autoreconf is not useless, it only becomes so from your dropping 'rm -rf', a change with which I actually disagree. I would prefer ensuring a clean slate every time.
Line 6: $1 --enable-static \
(cosmetic: personally, I'd prefer passing ".." or "." as $1 instead of '../configure' and run "$1"/configure)
Line 13: $MAKE $PARALLEL_MAKE \
Line 18: mkdir -p builddir
removing the rm -rf: have you checked that the jenkins build job running this cleans up the workspace? Otherwise there would be built files lying around in the workspace and we'd also rebuild over and over in the same build dirs. We should make sure it's done from scratch every time, one way or the other.
An alternative if you really dislike 'rm -rf' would be 'git clean -dxf'?
Line 13: $1 --enable-static $2 CFLAGS="-Werror" CPPFLAGS="-Werror"
(could just write $ENABLE_SANITIZE instead of passing $2. As before: preference of ".." for $1)
Line 14: $MAKE $PARALLEL_MAKE check \
Line 21: mkdir -p builddir
again make sure we build cleanly every time.
(seems a lot of trouble to add a _common script for just three lines, but ok)
To view, visit https://gerrit.osmocom.org/3132
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-Owner: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: blobb <dr.blobb at gmail.com>
Gerrit-Reviewer: lynxis lazus <lynxis at fe80.eu>
Gerrit-Reviewer: neels <nhofmeyr at sysmocom.de>
More information about the gerrit-log