fixup! Add protoc wrapper on Windows

tweag · Aug 22, 2023 · fa79a8e · fa79a8e
1 parent 1c73460
commit fa79a8e
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/tests/protoc.bzl b/tests/protoc.bzl
@@ -16,7 +16,9 @@ def _protoc_wrapper_impl(ctx):
  content = """\
 import os, subprocess, sys
 
-os.environ["PATH"] += os.pathsep + os.path.realpath(os.path.dirname({compiler_exe}))
+path = os.environ.get("PATH", "")
+path = os.pathsep.join([path, os.path.realpath(os.path.dirname({compiler_exe}))])
+os.environ["PATH"] = path
 subprocess.check_call([{protoc}] + sys.argv[1:])
 """.format(protoc = repr(ctx.executable._protoc.path), compiler_exe = repr(compiler)),
  is_executable = True,