osmo-gsm-tester[master]: fix build: properly remove artifacts

2017-09-13 Thread Neels Hofmeyr

Patch Set 1: Code-Review+2

-- 
To view, visit https://gerrit.osmocom.org/3933
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib7e5995aa6914e34f8b7bbb8a8ed46ff9230a913
Gerrit-PatchSet: 1
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr 
Gerrit-HasComments: No


[MERGED] osmo-gsm-tester[master]: fix build: properly remove artifacts

2017-09-13 Thread Neels Hofmeyr
Neels Hofmeyr has submitted this change and it was merged.

Change subject: fix build: properly remove artifacts
..


fix build: properly remove artifacts

The jenkins scripts are run by sh, which does not support the {a,b} syntax
shell glob. As a result, old build artifacts pile up in the workspaces.
Use two separate lines to remove .tgz and .md5.

Change-Id: Ib7e5995aa6914e34f8b7bbb8a8ed46ff9230a913
---
M contrib/jenkins-build-common.sh
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Neels Hofmeyr: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/contrib/jenkins-build-common.sh b/contrib/jenkins-build-common.sh
index a5b34af..39046ef 100644
--- a/contrib/jenkins-build-common.sh
+++ b/contrib/jenkins-build-common.sh
@@ -56,7 +56,8 @@
 env | grep -v "^LESS" | sort
 
 # clean the workspace
-rm -f "$base"/*.build-*.{tgz,md5}
+rm -f "$base"/*.build-*.tgz
+rm -f "$base"/*.build-*.md5
 rm -rf "$prefix_real"
 mkdir -p "$prefix_real"
 

-- 
To view, visit https://gerrit.osmocom.org/3933
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib7e5995aa6914e34f8b7bbb8a8ed46ff9230a913
Gerrit-PatchSet: 1
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr 


[PATCH] osmo-gsm-tester[master]: fix build: properly remove artifacts

2017-09-13 Thread Neels Hofmeyr

Review at  https://gerrit.osmocom.org/3933

fix build: properly remove artifacts

The jenkins scripts are run by sh, which does not support the {a,b} syntax
shell glob. As a result, old build artifacts pile up in the workspaces.
Use two separate lines to remove .tgz and .md5.

Change-Id: Ib7e5995aa6914e34f8b7bbb8a8ed46ff9230a913
---
M contrib/jenkins-build-common.sh
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester refs/changes/33/3933/1

diff --git a/contrib/jenkins-build-common.sh b/contrib/jenkins-build-common.sh
index a5b34af..39046ef 100644
--- a/contrib/jenkins-build-common.sh
+++ b/contrib/jenkins-build-common.sh
@@ -56,7 +56,8 @@
 env | grep -v "^LESS" | sort
 
 # clean the workspace
-rm -f "$base"/*.build-*.{tgz,md5}
+rm -f "$base"/*.build-*.tgz
+rm -f "$base"/*.build-*.md5
 rm -rf "$prefix_real"
 mkdir -p "$prefix_real"
 

-- 
To view, visit https://gerrit.osmocom.org/3933
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib7e5995aa6914e34f8b7bbb8a8ed46ff9230a913
Gerrit-PatchSet: 1
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr