Change in osmo-e1-hardware[master]: Revert "contrib/jenkins.sh: ugly work-around for PATH problems"

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

laforge gerrit-no-reply at lists.osmocom.org
Wed Jan 13 17:30:42 UTC 2021


laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-e1-hardware/+/22159 )

Change subject: Revert "contrib/jenkins.sh: ugly work-around for PATH problems"
......................................................................

Revert "contrib/jenkins.sh: ugly work-around for PATH problems"

Revert, because we don't overwrite PATH anymore. This reverts commit
5ab3bfdba4de2af32aeffdc92de963cb34c9715a.

Related: OS#4911
Change-Id: I981511f406b80b80e2a863c94b6a60755d824fec
---
M contrib/jenkins.sh
1 file changed, 0 insertions(+), 9 deletions(-)

Approvals:
  Jenkins Builder: Verified
  laforge: Looks good to me, approved; Verified



diff --git a/contrib/jenkins.sh b/contrib/jenkins.sh
index d69fcbb..171d368 100755
--- a/contrib/jenkins.sh
+++ b/contrib/jenkins.sh
@@ -5,15 +5,6 @@
 # * WITH_MANUALS: build manual PDFs if set to "1"
 # * PUBLISH: upload manuals after building if set to "1" (ignored without WITH_MANUALS = "1")
 
-# ugly, ugly hack to work around the fact that we cannot _extend_ the path when executing
-# a docker container.  We can either override it (and loose our /opt/... toolchain paths)
-# or we can not specify the /build_bin whcih means the osmo-build-dep.sh is not found
-# See https://osmocom.org/issues/4911
-if [ -d /build_bin ]; then
-	PATH=$PATH:/build_bin
-	export PATH
-fi
-
 if ! [ -x "$(command -v osmo-build-dep.sh)" ]; then
 	echo "Error: We need to have scripts/osmo-deps.sh from http://git.osmocom.org/osmo-ci/ in PATH !"
 	exit 2

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-e1-hardware/+/22159
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-e1-hardware
Gerrit-Branch: master
Gerrit-Change-Id: I981511f406b80b80e2a863c94b6a60755d824fec
Gerrit-Change-Number: 22159
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge at osmocom.org>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20210113/c3078e12/attachment.htm>


More information about the gerrit-log mailing list