Commit 65ec78a7 authored by Swann Perarnau's avatar Swann Perarnau

Merge branch 'explicit-env' into 'master'

Pass environment explicitly

See merge request !21
parents 3a52922c 3fcf2f50
Pipeline #3751 passed with stages
in 20 seconds
......@@ -114,11 +114,13 @@ class NodeOSClient(object):
# to escape spaces from arguments before.
argv = [s.replace(' ', r'\ ') for s in argv]
cmd += " argv:'"+" ".join(argv)+"'"
env = ['{0}={1}'.format(envname, envval.replace(' ', r'\ '))
for envname, envval in environ.items()]
cmd += " env:'"+" ".join(env)+"'"
args.append(cmd)
return process.Subprocess(args, stdin=process.Subprocess.STREAM,
stdout=process.Subprocess.STREAM,
stderr=process.Subprocess.STREAM,
env=environ)
stderr=process.Subprocess.STREAM)
class ChrtClient(object):
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment