Harald Welte has submitted this change and it was merged.
Change subject: jenkins.sh: cleanup always the workspace
......................................................................
jenkins.sh: cleanup always the workspace
It was broken before becaues of checking emptyness against literal string
will never be zero.
However it should be cleaned always because the script is executed
recursive which would also end up, never being executed.
Change-Id: Ib228ff247a72b21b12e8dd4cbe3afe3e858c89d3
---
M contrib/jenkins.sh
1 file changed, 1 insertion(+), 3 deletions(-)
Approvals:
Pau Espin Pedrol: Looks good to me, but someone else must approve
Harald Welte: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index 8751abd..10fc2b1 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -83,6 +83,4 @@
$MAKE check \
|| cat-testlogs.sh
-if [ -z "x${INSIDE_CHROOT}" ]; then
- osmo-clean-workspace.sh
-fi
+osmo-clean-workspace.sh
--
To view, visit https://gerrit.osmocom.org/6911
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib228ff247a72b21b12e8dd4cbe3afe3e858c89d3
Gerrit-PatchSet: 2
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Owner: lynxis lazus <[email protected]>
Gerrit-Reviewer: Harald Welte <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Pau Espin Pedrol <[email protected]>
Gerrit-Reviewer: lynxis lazus <[email protected]>
