Mercurial Repositories > eric / changeset

--- a/src/eric7/SystemUtilities/FileSystemUtilities.py	Sun Dec 10 17:06:00 2023 +0100
+++ b/src/eric7/SystemUtilities/FileSystemUtilities.py	Sun Dec 10 17:49:42 2023 +0100
@@ -740,18 +740,19 @@
     """
     filePath = str(filePath)
 
-    if OSUtilities.isWindowsPlatform():
-        return os.startfile(filePath)  # secok
-
-    elif OSUtilities.isMacPlatform():
-        return subprocess.call(("open", filePath))  # secok
+    with contextlib.suppress(OSError):
+        if OSUtilities.isWindowsPlatform():
+            os.startfile(filePath)  # secok
+            return True
 
-    elif OSUtilities.isLinuxPlatform() or OSUtilities.isFreeBsdPlatform():
-        return subprocess.call(("xdg-open", filePath))  # secok
+        elif OSUtilities.isMacPlatform():
+            return subprocess.call(("open", filePath)) == 0  # secok
 
-    else:
-        # unsupported platform
-        return False
+        elif OSUtilities.isLinuxPlatform() or OSUtilities.isFreeBsdPlatform():
+            return subprocess.call(("xdg-open", filePath)) == 0  # secok
+
+    # unsupported platform or OSError
+    return False
 
 
 ################################################################################

--- a/src/eric7/UI/Browser.py	Sun Dec 10 17:06:00 2023 +0100
+++ b/src/eric7/UI/Browser.py	Sun Dec 10 17:49:42 2023 +0100
@@ -599,9 +599,20 @@
         )
         for itm in itmList:
             if isinstance(itm, BrowserDirectoryItem):
-                QDesktopServices.openUrl(QUrl(itm.dirName()))
+                directory = itm.dirName()
             else:
-                QDesktopServices.openUrl(QUrl(os.path.dirname(itm.fileName())))
+                directory = os.path.dirname(itm.fileName())
+            ok = FileSystemUtilities.startfile(directory)
+
+            if not ok:
+                EricMessageBox.warning(
+                    self,
+                    self.tr("Show in File Manager"),
+                    self.tr(
+                        "<p>The directory of the selected item (<b>{0}</b>could not be"
+                        " shown in a file manager application.</p>"
+                    ).format(directory),
+                )
 
     def __showMimeType(self):
         """