Skip to content

Commit

Permalink
Manually merge paths (prepending the new section)
Browse files Browse the repository at this point in the history
This turned out to be the issue for the ride
jobs - I was adding in path before loading
modules and then the map update overwrote all
the changes with the initial path. It did not
show on other systems because they were not
manually adding paths.
  • Loading branch information
prwolfe committed Aug 14, 2019
1 parent 30df5f6 commit 6788e65
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions cmake/std/PullRequestLinuxDriverTest.py
Original file line number Diff line number Diff line change
Expand Up @@ -245,8 +245,7 @@ def setBuildEnviron(arguments):
'install',
'white-ride',
'ninja-1.8.2',
'bin') + os.pathsep +
os.environ['PATH']} }
'bin')} }

try:
moduleList = moduleMap[arguments.job_base_name]
Expand All @@ -269,7 +268,12 @@ def setBuildEnviron(arguments):
if 'OMPI_FC' in l_environMap:
l_environMap['OMPI_FC'] = os.environ.get('FC', '')

os.environ.update(l_environMap)
for key, value in l_environMap.items():
if key in os.environ:
# we are assuming these are paths to be prepended
os.environ[key] = value + os.pathsep + os.environ[key]
else:
os.environ[key] = value
confirmGitVersion()

print ("Environment:\n", file=sys.stdout)
Expand Down

0 comments on commit 6788e65

Please sign in to comment.