Make SEGYImport work great again on windows (#159)

equinor · Mar 24, 2023 · afb37e9 · afb37e9
1 parent a7f5aea
commit afb37e9
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/fmu/sumo/uploader/_fileondisk.py b/src/fmu/sumo/uploader/_fileondisk.py
@@ -73,12 +73,13 @@ def _get_segyimport_cmdstr(blob_url, object_id, file_path, sample_unit):
     url_conn = '"Suffix=?' + blob_url["auth"] + '"'
     persistent_id = '"' + object_id + '"'
 
-    pythonPath = os.path.dirname(sys.executable)
-    path_to_SEGYImport = os.path.join(pythonPath, '..', 'bin', 'SEGYImport') 
+    python_path = os.path.dirname(sys.executable)
+    path_to_SEGYImport = os.path.join(python_path, '..', 'bin', 'SEGYImport') 
+    if sys.platform.startswith("win"):
+        path_to_SEGYImport = path_to_SEGYImport + ".exe"
     if not os.path.isfile(path_to_SEGYImport):
         path_to_SEGYImport = os.path.join(python_path, '..', 'shims', 'SEGYImport')
 
-
     cmdstr = ' '.join([path_to_SEGYImport, 
         '--compression-method', 'RLE',
         '--brick-size', '64', 

diff --git a/tests/test_uploader.py b/tests/test_uploader.py
@@ -210,6 +210,8 @@ def test_openvds_available(token):
     python_path = os.path.dirname(sys.executable)
     logger.info(python_path)
     path_to_SEGYImport = os.path.join(python_path, '..', 'bin', 'SEGYImport')
+    if sys.platform.startswith("win"):
+        path_to_SEGYImport = path_to_SEGYImport + ".exe"
     if not os.path.isfile(path_to_SEGYImport):
         path_to_SEGYImport = os.path.join(python_path, '..', 'shims', 'SEGYImport')