laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/37933?usp=email )

Change subject: testenv: fix testsuite.run() not passing env variables
......................................................................

testenv: fix testsuite.run() not passing env variables

Running start-testsuite.sh in a clean env fails on systems with non-
standard TTCN3_BIN_DIR and TITAN_LIBRARY_PATH paths.  This is the
case for Arch Linux and the eclipse-titan AUR package.  Let's call
cmd.generate_env() to allow overriding these variables.

Change-Id: I95c3cfe41ee230927bb5777631372597cbd3c051
---
M _testenv/testenv/testsuite.py
1 file changed, 2 insertions(+), 3 deletions(-)

Approvals:
  pespin: Looks good to me, but someone else must approve
  Jenkins Builder: Verified
  laforge: Looks good to me, approved




diff --git a/_testenv/testenv/testsuite.py b/_testenv/testenv/testsuite.py
index 88ea467..077d3f1 100644
--- a/_testenv/testenv/testsuite.py
+++ b/_testenv/testenv/testsuite.py
@@ -162,9 +162,8 @@
     start_testsuite = os.path.join(ttcn3_hacks_dir, "start-testsuite.sh")
     suite = os.path.join(ttcn3_hacks_dir, testenv.args.testsuite, 
section_data["program"])

-    env = {
-        "TTCN3_PCAP_PATH": os.path.join(testenv.testdir.testdir, "testsuite"),
-    }
+    pcap_path = os.path.join(testenv.testdir.testdir, "testsuite")
+    env = testenv.cmd.generate_env({ "TTCN3_PCAP_PATH" : pcap_path })

     cmd = [start_testsuite, suite, section_data["config"]]


--
To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/37933?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: merged
Gerrit-Project: osmo-ttcn3-hacks
Gerrit-Branch: master
Gerrit-Change-Id: I95c3cfe41ee230927bb5777631372597cbd3c051
Gerrit-Change-Number: 37933
Gerrit-PatchSet: 3
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: osmith <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>

Reply via email to