From 31babaac5fa7c33126dad277d4e28b4ff5aef184 Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Thu, 24 Nov 2022 17:07:39 +0100
Subject: [PATCH 1/3] change how extensions are checked when finding executable

---
 openpype/lib/vendor_bin_utils.py | 58 +++++++++++++++++++-------------
 1 file changed, 34 insertions(+), 24 deletions(-)

diff --git a/openpype/lib/vendor_bin_utils.py b/openpype/lib/vendor_bin_utils.py
index 099f9a34ba..91ba94c60e 100644
--- a/openpype/lib/vendor_bin_utils.py
+++ b/openpype/lib/vendor_bin_utils.py
@@ -70,24 +70,21 @@ def find_executable(executable):
     low_platform = platform.system().lower()
     _, ext = os.path.splitext(executable)
 
-    # Prepare variants for which it will be looked
-    variants = [executable]
-    # Add other extension variants only if passed executable does not have one
-    if not ext:
-        if low_platform == "windows":
-            exts = [".exe", ".ps1", ".bat"]
-            for ext in os.getenv("PATHEXT", "").split(os.pathsep):
-                ext = ext.lower()
-                if ext and ext not in exts:
-                    exts.append(ext)
-        else:
-            exts = [".sh"]
+    # Prepare extensions to check
+    exts = set()
+    if ext:
+        exts.add(ext.lower())
 
-        for ext in exts:
-            variant = executable + ext
-            if is_file_executable(variant):
-                return variant
-            variants.append(variant)
+    else:
+        # Add other possible extension variants only if passed executable
+        #   does not have any
+        if low_platform == "windows":
+            exts |= {".exe", ".ps1", ".bat"}
+            for ext in os.getenv("PATHEXT", "").split(os.pathsep):
+                exts.add(ext.lower())
+
+        else:
+            exts |= {".sh"}
 
     # Get paths where to look for executable
     path_str = os.environ.get("PATH", None)
@@ -97,13 +94,26 @@ def find_executable(executable):
         elif hasattr(os, "defpath"):
             path_str = os.defpath
 
-    if path_str:
-        paths = path_str.split(os.pathsep)
-        for path in paths:
-            for variant in variants:
-                filepath = os.path.abspath(os.path.join(path, variant))
-                if is_file_executable(filepath):
-                    return filepath
+    if not path_str:
+        return None
+
+    paths = path_str.split(os.pathsep)
+    for path in paths:
+        if not os.path.isdir(path):
+            continue
+        for filename in os.listdir(path):
+            filepath = os.path.abspath(os.path.join(path, filename))
+            # Filename matches executable exactly
+            if filename == executable and is_file_executable(filepath):
+                return filepath
+
+            basename, ext = os.path.splitext(filename)
+            if (
+                basename == executable
+                and ext.lower() in exts
+                and is_file_executable(filepath)
+            ):
+                return filepath
     return None
 
 

From 3ca4c04a158b99e77d6f18b171ababd91d02eae0 Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Thu, 24 Nov 2022 17:08:45 +0100
Subject: [PATCH 2/3] added ability to fill only extension when is missing

---
 openpype/lib/vendor_bin_utils.py | 21 +++++++++++++++++++--
 1 file changed, 19 insertions(+), 2 deletions(-)

diff --git a/openpype/lib/vendor_bin_utils.py b/openpype/lib/vendor_bin_utils.py
index 91ba94c60e..16e2c197f9 100644
--- a/openpype/lib/vendor_bin_utils.py
+++ b/openpype/lib/vendor_bin_utils.py
@@ -60,9 +60,10 @@ def find_executable(executable):
             path to file.
 
     Returns:
-        str: Full path to executable with extension (is file).
-        None: When the executable was not found.
+        Union[str, None]: Full path to executable with extension which was
+            found otherwise None.
     """
+
     # Skip if passed path is file
     if is_file_executable(executable):
         return executable
@@ -86,6 +87,21 @@ def find_executable(executable):
         else:
             exts |= {".sh"}
 
+    # Executable is a path but there may be missing extension
+    #   - this can happen primarily on windows where
+    #       e.g. "ffmpeg" should be "ffmpeg.exe"
+    exe_dir, exe_filename = os.path.split(executable)
+    if exe_dir and os.path.isdir(exe_dir):
+        for filename in os.listdir(exe_dir):
+            filepath = os.path.join(exe_dir, filename)
+            basename, ext = os.path.splitext(filename)
+            if (
+                basename == exe_filename
+                and ext.lower() in exts
+                and is_file_executable(filepath)
+            ):
+                return filepath
+
     # Get paths where to look for executable
     path_str = os.environ.get("PATH", None)
     if path_str is None:
@@ -114,6 +130,7 @@ def find_executable(executable):
                 and is_file_executable(filepath)
             ):
                 return filepath
+
     return None
 
 

From 453cada172b5962921af9d3dc61c64b0b379d277 Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Thu, 24 Nov 2022 17:09:16 +0100
Subject: [PATCH 3/3] change how oiio tools executables are found

---
 openpype/lib/vendor_bin_utils.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/openpype/lib/vendor_bin_utils.py b/openpype/lib/vendor_bin_utils.py
index 16e2c197f9..b6797dbba0 100644
--- a/openpype/lib/vendor_bin_utils.py
+++ b/openpype/lib/vendor_bin_utils.py
@@ -299,8 +299,8 @@ def get_oiio_tools_path(tool="oiiotool"):
         oiio_dir = get_vendor_bin_path("oiio")
         if platform.system().lower() == "linux":
             oiio_dir = os.path.join(oiio_dir, "bin")
-        default_path = os.path.join(oiio_dir, tool)
-        if _oiio_executable_validation(default_path):
+        default_path = find_executable(os.path.join(oiio_dir, tool))
+        if default_path and _oiio_executable_validation(default_path):
             tool_executable_path = default_path
 
     # Look to PATH for the tool