Fix exec in docker

Author: lucasssvaz

tools/config_editor/app.py

@@ -265,8 +265,8 @@ def main() -> None:
             print("Warning: Default Arduino path not found. Disabling copy to Arduino.")
             app.setting_enable_copy = False
         elif args.arduino_path == "/arduino-esp32": # Docker mount point
-            print("Error: Arduino folder not mounted to /arduino-esp32 in Docker container.")
-            exit(1)
+            print("Warning: Docker mount point not found. Disabling copy to Arduino.")
+            app.setting_enable_copy = False
         else:
             print("Error: Invalid path to Arduino core: " + os.path.abspath(args.arduino_path))
             exit(1)

tools/config_editor/compile.py

@@ -139,11 +139,13 @@ def compile_libs(self) -> None:
                 print("Error reading child process errors: " + str(e))
             label.update("Compilation failed for " + target)
         else:
-            regex = r"^[1-9][0-9]*:[1-9][0-9]*$"  # Regex to match the line:column format
+            regex = r"^[1-9][0-9]*:[1-9][0-9]*$"  # Regex to match the uid:gid format
             if self.app.setting_arduino_permissions and re.match(regex, self.app.setting_arduino_permissions):
+                print_info("Changing permissions of generated files")
                 chown_process = None
                 try:
                     chown_process = subprocess.run(["chown", "-R", self.app.setting_arduino_permissions, self.app.setting_arduino_path])
+                    chown_process.wait()
                 except Exception as e:
                     print("Error changing permissions: " + str(e))
 

tools/docker/entrypoint.sh

@@ -15,9 +15,11 @@ then
 	done
 fi
 
-exec "$@"
-
-if [ -d /arduino-esp32 ]; then
-	echo "Fixing permissions on /arduino-esp32"
-	chown -R `stat -c "%u:%g" /arduino-esp32` /arduino-esp32
+# Check if /arduino-esp32 exists
+if [ -d "/arduino-esp32" ]; then
+    # If it exists, add the -p argument
+    exec "$@" -p `stat -c "%u:%g" /arduino-esp32`
+else
+    # If it doesn't exist, just execute the command without the -p argument
+    exec "$@"
 fi