fixeria submitted this change.

View Change

Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved
fixup: coverity: compile tests, but not execute them

In change 82265bc46df2f5c54be71274330e43664cf6c7fc [1] I attempted
to fix aaeab8c976a478309477e4200c1aef51407f6152 [2], however both
pushd and popd commands are not available in plain shell.

Change-Id: Ib63ded66781c774e16773537c3b943237904b03e
Fixes: [1] I92b648cd1e80399561067eae99426faed1ff1e25
Fixes: [2] Idf0714a59356633021be21acc47f9471d059b7fe
---
M coverity/prepare_source_Osmocom.sh
1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/coverity/prepare_source_Osmocom.sh b/coverity/prepare_source_Osmocom.sh
index ba5f13a..11b7e5c 100755
--- a/coverity/prepare_source_Osmocom.sh
+++ b/coverity/prepare_source_Osmocom.sh
@@ -52,9 +52,7 @@
# instead of 'check_PROGRAMS' allows building test binaries during 'make all'.
files="$(git -C $proj grep -l check_PROGRAMS)"
if [ -n "$files" ]; then
- pushd $proj
- sed -i "s/check_PROGRAMS/noinst_PROGRAMS/" $files
- popd
+ (cd $proj && sed -i "s/check_PROGRAMS/noinst_PROGRAMS/" $files)
fi
done


To view, visit change 28515. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-ci
Gerrit-Branch: master
Gerrit-Change-Id: Ib63ded66781c774e16773537c3b943237904b03e
Gerrit-Change-Number: 28515
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged