Skip to content

Commit 69e18e2

Browse files
authored
Merge pull request #81 from rdeits/rd/environ
Pass the full environment to the subprocess rather than just PYTHONPATH
2 parents 7a63905 + 73fd480 commit 69e18e2

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

src/meshcat/servers/zmqserver.py

+5-2
Original file line numberDiff line numberDiff line change
@@ -54,8 +54,11 @@ def start_zmq_server_as_subprocess(zmq_url=None, server_args=[]):
5454
args.append(*server_args)
5555
# Note: Pass PYTHONPATH to be robust to workflows like Google Colab,
5656
# where meshcat might have been added directly via sys.path.append.
57-
env = {'PYTHONPATH': os.path.dirname(os.path.dirname(os.path.dirname(__file__)))}
58-
kwargs = {
57+
# Copy existing environmental variables as some of them might be needed
58+
# e.g. on Windows SYSTEMROOT and PATH
59+
env = dict(os.environ)
60+
env["PYTHONPATH"] = os.path.dirname(os.path.dirname(os.path.dirname(__file__)))
61+
kwargs = {
5962
'stdout': subprocess.PIPE,
6063
'stderr': subprocess.PIPE,
6164
'env': env

0 commit comments

Comments
 (0)