espidf: fix subprocess related bugs

mlonmcu/platform/espidf.py

@@ -122,7 +122,7 @@ def invoke_idf_exe(self, *args, **kwargs):
             + " ".join([str(arg) for arg in args])
         )
         out = utils.exec_getout(
-            cmd, shell=True, env=env, **kwargs
+            cmd, shell=True, env=env, **kwargs, executable="/bin/bash"
         )  # TODO: using shell=True is insecure but right now we can not avoid it?
         return out
 
@@ -340,7 +340,9 @@ def _monitor_helper(*args, verbose=False, start_match=None, end_match=None, time
                     + f"> /dev/null && {self.idf_exe} "
                     + " ".join([str(arg) for arg in args])
                 )
-                process = subprocess.Popen(cmd, stdout=subprocess.PIPE, stderr=subprocess.STDOUT, shell=True, env=env)
+                process = subprocess.Popen(
+                    cmd, stdout=subprocess.PIPE, stderr=subprocess.STDOUT, shell=True, executable="bin/bash", env=env
+                )
                 try:
                     exit_code = None
                     for line in process.stdout:

mlonmcu/setup/tasks.py

@@ -1004,6 +1004,6 @@ def install_espidf(context: MlonMcuContext, params=None, rebuild=False, verbose=
         espidfInstallScript = Path(espidfSrcDir) / "install.sh"
         env = {}  # Do not use current virtualenv (TODO: is there a better way?)
         env["IDF_TOOLS_PATH"] = str(espidfInstallDir)
-        env["PATH"] = "/usr/bin"
+        env["PATH"] = "/usr/bin:/bin"  # TODO: find a better way to achieve this
         utils.exec_getout(espidfInstallScript, boards, live=verbose, env=env)
     context.cache["espidf.install_dir"] = espidfInstallDir