Merge pull request #32 from lrauschning/master

remove unnecessary permission changes
vlasmirnov · Mar 5, 2024 · aba572d · aba572d
2 parents 2852dd9 + 78421b4
commit aba572d
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/magus_configuration.py b/magus_configuration.py
@@ -19,7 +19,10 @@ def retrieve_packaged_binary(p):
         for executable in glob(os.path.dirname(p) + "/**/*",recursive=True):
             if not os.path.isfile(executable):
                 continue
-            os.chmod(executable, os.stat(p).st_mode | stat.S_IEXEC)
+            # set to executable if not set already
+            # this will fail in containerized environments!
+            if not os.stat(executable) & stat.S_IEXEC:
+                os.chmod(executable, os.stat(executable).st_mode | stat.S_IEXEC)
         return p
     else:
         return None