Holger Freyther has submitted this change and it was merged. (
https://gerrit.osmocom.org/12103 )
Change subject: ms_driver: Do the existance check with the path
......................................................................
ms_driver: Do the existance check with the path
Change the existance test to use the full path and not just the
binary name.
Change-Id: Ibe28514df03efb7c302ac30411761a8d1a6dcf5d
---
M src/osmo_gsm_tester/ms_driver.py
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Pau Espin Pedrol: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/src/osmo_gsm_tester/ms_driver.py b/src/osmo_gsm_tester/ms_driver.py
index b29859b..bc1cce9 100644
--- a/src/osmo_gsm_tester/ms_driver.py
+++ b/src/osmo_gsm_tester/ms_driver.py
@@ -93,7 +93,7 @@
def check_and_return_binary(name):
"""Checks the binary exists and returns the path."""
binary = inst.child('bin', name)
- if not os.path.isfile(name):
+ if not os.path.isfile(binary):
raise RuntimeError('Binary missing: %r' % binary)
return binary
--
To view, visit https://gerrit.osmocom.org/12103
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ibe28514df03efb7c302ac30411761a8d1a6dcf5d
Gerrit-Change-Number: 12103
Gerrit-PatchSet: 1
Gerrit-Owner: Holger Freyther <[email protected]>
Gerrit-Reviewer: Holger Freyther <[email protected]>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Pau Espin Pedrol <[email protected]>
