Change in osmo-gsm-tester[master]: util: Fix unloading of suite lib import path

2018-05-14 Thread Pau Espin Pedrol
Pau Espin Pedrol has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/9159 )

Change subject: util: Fix unloading of suite lib import path
..

util: Fix unloading of suite lib import path

Change-Id: Iac81151f5baec092a3d15bbe6541b5bc007e6a03
---
M src/osmo_gsm_tester/util.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Pau Espin Pedrol: Looks good to me, approved; Verified



diff --git a/src/osmo_gsm_tester/util.py b/src/osmo_gsm_tester/util.py
index c7efb77..27c71cf 100644
--- a/src/osmo_gsm_tester/util.py
+++ b/src/osmo_gsm_tester/util.py
@@ -119,7 +119,7 @@

 def import_path_remove(pathname):
 dir = os.path.realpath(pathname)
-if dir not in sys.path:
+if dir in sys.path:
 sys.path.remove(dir)

 class listdict(dict):

--
To view, visit https://gerrit.osmocom.org/9159
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: Iac81151f5baec092a3d15bbe6541b5bc007e6a03
Gerrit-Change-Number: 9159
Gerrit-PatchSet: 1
Gerrit-Owner: Pau Espin Pedrol 
Gerrit-Reviewer: Pau Espin Pedrol 


Change in osmo-gsm-tester[master]: util: Fix unloading of suite lib import path

2018-05-14 Thread Pau Espin Pedrol
Pau Espin Pedrol has posted comments on this change. ( 
https://gerrit.osmocom.org/9159 )

Change subject: util: Fix unloading of suite lib import path
..


Patch Set 1: Verified+1 Code-Review+2


--
To view, visit https://gerrit.osmocom.org/9159
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Iac81151f5baec092a3d15bbe6541b5bc007e6a03
Gerrit-Change-Number: 9159
Gerrit-PatchSet: 1
Gerrit-Owner: Pau Espin Pedrol 
Gerrit-Reviewer: Pau Espin Pedrol 
Gerrit-Comment-Date: Mon, 14 May 2018 13:37:51 +
Gerrit-HasComments: No
Gerrit-HasLabels: Yes


Change in osmo-gsm-tester[master]: util: Fix unloading of suite lib import path

2018-05-14 Thread Pau Espin Pedrol
Pau Espin Pedrol has uploaded this change for review. ( 
https://gerrit.osmocom.org/9159


Change subject: util: Fix unloading of suite lib import path
..

util: Fix unloading of suite lib import path

Change-Id: Iac81151f5baec092a3d15bbe6541b5bc007e6a03
---
M src/osmo_gsm_tester/util.py
1 file changed, 1 insertion(+), 1 deletion(-)



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

diff --git a/src/osmo_gsm_tester/util.py b/src/osmo_gsm_tester/util.py
index c7efb77..27c71cf 100644
--- a/src/osmo_gsm_tester/util.py
+++ b/src/osmo_gsm_tester/util.py
@@ -119,7 +119,7 @@

 def import_path_remove(pathname):
 dir = os.path.realpath(pathname)
-if dir not in sys.path:
+if dir in sys.path:
 sys.path.remove(dir)

 class listdict(dict):

--
To view, visit https://gerrit.osmocom.org/9159
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Iac81151f5baec092a3d15bbe6541b5bc007e6a03
Gerrit-Change-Number: 9159
Gerrit-PatchSet: 1
Gerrit-Owner: Pau Espin Pedrol