Merge pull request #445 from ExpressLRS/ignore-pythonhome-pythonpath-env

Ignore PYTHONPATH & PYTHONHOME env variables when using platformio

src/api/src/library/Platformio/index.ts

@@ -31,14 +31,33 @@ const prependPATH = (pth: string, item: string): string => {
   return item;
 };
 
+const envFilter = (
+  env: NodeJS.ProcessEnv,
+  blacklist: string[]
+): NodeJS.ProcessEnv => {
+  const result: NodeJS.ProcessEnv = {};
+  Object.keys(env).forEach((key) => {
+    if (!blacklist.includes(key)) {
+      result[key] = env[key];
+    }
+  });
+  return result;
+};
+
 export default class Platformio {
+  private env: NodeJS.ProcessEnv;
+
   constructor(
     private getPlatformioPath: string,
     private stateTempStoragePath: string,
     private PATH: string,
-    private env: NodeJS.ProcessEnv,
+    env: NodeJS.ProcessEnv,
     private logger: LoggerService
-  ) {}
+  ) {
+    // Fix for https://github.com/ExpressLRS/ExpressLRS-Configurator/issues/440
+    const blacklistedEnvKeys = ['PYTHONPATH', 'PYTHONHOME'];
+    this.env = envFilter(env, blacklistedEnvKeys);
+  }
 
   async install(onUpdate: OnOutputFunc = NoOpFunc): Promise<CommandResult> {
     const pyExec = await this.findPythonExecutable(this.PATH);