Improve permission settings

Author: lucasssvaz

tools/config_editor/compile.py

@@ -139,21 +139,25 @@ 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 uid:gid format
-            if self.app.setting_output_permissions and re.match(regex, self.app.setting_output_permissions):
-                print_info("Changing permissions of generated files")
-                chown_process = None
-                try:
-                    chown_process = subprocess.run(["chown", "-R", self.app.setting_output_permissions, self.app.setting_arduino_path])
-                    chown_process.wait()
-                except Exception as e:
-                    print("Error changing permissions: " + str(e))
-
-                if chown_process and chown_process.returncode != 0:
-                    self.print_error("Error changing permissions.")
-                    self.print_error("Please change the ownership of generated files manually")
+            if self.app.setting_output_permissions:
+                regex = r"^[1-9][0-9]*:[1-9][0-9]*$"  # Regex to match the uid:gid format. Note that 0:0 (root) is not allowed
+                if re.match(regex, self.app.setting_output_permissions):
+                    print_info("Setting permissions to: " + self.app.setting_output_permissions)
+                    chown_process = None
+                    try:
+                        chown_process = subprocess.run(["chown", "-R", self.app.setting_output_permissions, self.app.setting_arduino_path])
+                        chown_process.wait()
+                    except Exception as e:
+                        print("Error changing permissions: " + str(e))
+
+                    if chown_process and chown_process.returncode != 0:
+                        self.print_error("Error changing permissions")
+                        self.print_error("Please change the ownership of generated files manually")
+                    else:
+                        self.print_success("Permissions changed successfully")
                 else:
-                    self.print_success("Permissions changed successfully")
+                    self.print_error("Invalid permissions format: " + self.app.setting_output_permissions)
+                    self.print_error("Please change the ownership of generated files manually")
 
             self.print_success("Compilation successful for " + target)
             label.update("Compilation successful for " + target)