fixing a podman-related issue where it relies on the PATH environment variable including the directory where other executables are located

Author: robertpatrick

electron/app/js/imageBuilderUtils.js

@@ -137,7 +137,9 @@ async function doPushImage(currentWindow, stdoutChannel, stderrChannel, imageBui
 
 function getDockerEnv(httpsProxyUrl, bypassProxyHosts) {
   let env = {
-    DOCKER_BUILDKIT: '0'
+    DOCKER_BUILDKIT: '0',
+    // podman relies on the PATH including other executables (e.g., newuidmap)...
+    PATH: process.env.PATH
   };
 
   if (httpsProxyUrl) {

electron/app/js/kubectlUtils.js

@@ -601,13 +601,14 @@ async function apply(kubectlExe, fileData, options) {
 }
 
 function getKubectlEnvironment(options, httpsProxyUrl, bypassProxyHosts) {
-  const env = {};
+  const env = {
+    PATH: process.env.PATH
+  };
   if (options) {
     if (options.kubeConfig && Array.isArray(options.kubeConfig) && options.kubeConfig.length > 0) {
       env['KUBECONFIG'] = options.kubeConfig.join(path.delimiter);
       getLogger().debug('Set KUBECONFIG environment variable to %s', env['KUBECONFIG']);
     }
-    env['PATH'] = process.env.PATH;
     if (options.extraPathDirectories && Array.isArray(options.extraPathDirectories) && options.extraPathDirectories.length > 0) {
       env['PATH'] = env['PATH'] + path.delimiter + options.extraPathDirectories.join(path.delimiter);
     }