From db26055d563af7a92670aabd1c26121928527b80 Mon Sep 17 00:00:00 2001
From: iLLiCiTiT <jakub.trllo@gmail.com>
Date: Mon, 25 Oct 2021 19:17:40 +0200
Subject: [PATCH] use plist info to get full path to executable

---
 openpype/lib/applications.py | 48 ++++++++++++++++++++++++++++++------
 1 file changed, 41 insertions(+), 7 deletions(-)

diff --git a/openpype/lib/applications.py b/openpype/lib/applications.py
index cc8cb8e7be..b9bcecd3a0 100644
--- a/openpype/lib/applications.py
+++ b/openpype/lib/applications.py
@@ -461,13 +461,8 @@ class ApplicationExecutable:
         # On MacOS check if exists path to executable when ends with `.app`
         # - it is common that path will lead to "/Applications/Blender" but
         #   real path is "/Applications/Blender.app"
-        if (
-            platform.system().lower() == "darwin"
-            and not os.path.exists(executable)
-        ):
-            _executable = executable + ".app"
-            if os.path.exists(_executable):
-                executable = _executable
+        if platform.system().lower() == "darwin":
+            executable = self.macos_executable_prep(executable)
 
         self.executable_path = executable
 
@@ -477,6 +472,45 @@ class ApplicationExecutable:
     def __repr__(self):
         return "<{}> {}".format(self.__class__.__name__, self.executable_path)
 
+    @staticmethod
+    def macos_executable_prep(executable):
+        """Try to find full path to executable file.
+
+        Real executable is stored in '*.app/Contents/MacOS/<executable>'.
+
+        Having path to '*.app' gives ability to read it's plist info and
+        use "CFBundleExecutable" key from plist to know what is "executable."
+
+        Plist is stored in '*.app/Contents/Info.plist'.
+
+        This is because some '*.app' directories don't have same permissions
+        as real executable.
+        """
+        # Try to find if there is `.app` file
+        if not os.path.exists(executable):
+            _executable = executable + ".app"
+            if os.path.exists(_executable):
+                executable = _executable
+
+        # Try to find real executable if executable has `Contents` subfolder
+        contents_dir = os.path.join(executable, "Contents")
+        if os.path.exists(contents_dir):
+            executable_filename = None
+            # Load plist file and check for bundle executable
+            plist_filepath = os.path.join(contents_dir, "Info.plist")
+            if os.path.exists(plist_filepath):
+                import plistlib
+
+                parsed_plist = plistlib.readPlist(plist_filepath)
+                executable_filename = parsed_plist.get("CFBundleExecutable")
+
+            if executable_filename:
+                executable = os.path.join(
+                    contents_dir, "MacOS", executable_filename
+                )
+
+        return executable
+
     def as_args(self):
         return [self.executable_path]