Feature/293 show file links

CHANGELOG.md

@@ -1,5 +1,9 @@
 This is an overview of major changes. Refer to the git repository for a full log change.
 
+Version 0.1.29
+-------------
+- Fixed #293 - Link to files fails with ugly error message on Windows
+
 Version 0.1.28
 -------------
 - Fixed #291 - Alter travis to use Anaconda / Miniconda

inselect/__init__.py

@@ -1,6 +1,6 @@
 """Segmentation, validation and annotation of images of museum specimens
 """
-__version__ = '0.1.28'
+__version__ = '0.1.29'
 # distutils/version.py and win32/lib/win32verstamp.py have different
 # restrictions on the format of version numbers - both must be met.
 # Three numbers separated by dots is a safe format.

inselect/gui/info_widget.py

@@ -4,7 +4,7 @@
 import subprocess
 import sys
 
-from PySide.QtCore import Qt, QProcessEnvironment
+from PySide.QtCore import Qt
 from PySide.QtGui import QWidget, QFormLayout, QLabel, QGroupBox, QVBoxLayout
 
 from inselect.lib.utils import format_dt_display
@@ -17,16 +17,11 @@ def reveal_path(path):
     """Shows path in Finder (on Mac) or in Explorer (on Windows)
     """
     # http://stackoverflow.com/a/3546503
+    path = path.resolve()
     if sys.platform.startswith("win"):
-        explorer = QProcessEnvironment.systemEnvironment().searchInPath("explorer.exe")
-        if not explorer:
-            raise ValueError('Explorer could not be located')
-        else:
-            if not path.is_dir():
-                arg = u"/select,{0}".format(path)
-            else:
-                arg = unicode(path)
-        subprocess.check_call([explorer, arg])
+        res = subprocess.call(["explorer.exe", u"/select,{0}".format(path)])
+        if 1 != res:
+            raise ValueError('Unexpected exit code [{0}]'.format(res))
     elif 'Darwin' == platform.system():
         reveal = u'tell application "Finder" to reveal POSIX file "{0}"'
         activate = u'tell application "Finder" to activate "{0}"'