Merge pull request #6 from ni/users/gregstoll/linux-generate-protobuf

Fix generate_protobuf_classes on Linux

.github/workflows/publish-to-pypi.yml

@@ -8,7 +8,7 @@ on:
 jobs:
   build-and-publish:
     if: github.repository == 'ni/niflexlogger-automation-python'
-    runs-on: windows-latest
+    runs-on: ubuntu-latest
 
     steps:
       - uses: actions/checkout@master
@@ -19,10 +19,10 @@ jobs:
           python-version: 3.8
 
       - name: Install setuptools and other tools
-        run: python -m pip install setuptools wheel twine grpcio grpcio-tools
+        run: python3 -m pip install setuptools wheel twine grpcio grpcio-tools
 
       - name: Build packages
-        run: python setup.py bdist_wheel
+        run: python3 setup.py bdist_wheel
 
       - name: Publish distribution to PyPI
         if: github.event.action == 'published'

generate_protobuf_classes.py

@@ -2,6 +2,7 @@
 import sys
 from itertools import chain
 from pathlib import Path
+from shlex import quote
 
 PROTO_PATHS = [
     "ConfigurationBasedSoftware/FlexLogger/Automation/FlexLogger.Automation.Protocols",
@@ -78,7 +79,9 @@ def _call_protoc() -> int:
     ]
 
     cwd = "./src" if _has_src_dir() else None
-    return subprocess.run([sys.executable, "-m", "grpc_tools.protoc"] + args, cwd=cwd).returncode
+    # Need to run with shell=True so the .proto files will get
+    # globbed on Linux.
+    return subprocess.run(' '.join([quote(sys.executable), "-m", "grpc_tools.protoc"] + args), cwd=cwd, shell=True).returncode
 
 
 if __name__ == "__main__":