Merge pull request #1 from blurstudio/shell-option

Add "shell" keyword argument to ExifTool initialization

Author: RootLUG

exiftool.py

@@ -149,7 +149,8 @@ class ExifTool(object):
        associated with a running subprocess.
     """
 
-    def __init__(self, executable_=None):
+    def __init__(self, executable_=None, shell=True):
+        self.shell = shell
         if executable_ is None:
             self.executable = executable
         else:
@@ -168,11 +169,16 @@ def start(self):
             warnings.warn("ExifTool already running; doing nothing.")
             return
         with open(os.devnull, "w") as devnull:
+            startupInfo = subprocess.STARTUPINFO()
+            if not self.shell:
+                # Adding enum 11 (SW_FORCEMINIMIZE in win32api speak) will
+                # keep it from throwing up a DOS shell when it launches.
+                startupInfo.dwFlags |= 11
             self._process = subprocess.Popen(
                 [self.executable, "-stay_open", "True",  "-@", "-",
                  "-common_args", "-G", "-n"],
                 stdin=subprocess.PIPE, stdout=subprocess.PIPE,
-                stderr=devnull)
+                stderr=devnull, startupinfo=startupInfo)
         self.running = True
 
     def terminate(self):

setup.py

@@ -17,12 +17,12 @@
 from distutils.core import setup
 
 setup(name="PyExifTool",
-      version="0.1",
+      version="0.1.1.post1",
       description="Python wrapper for exiftool",
       license="GPLv3+",
       author="Sven Marnach",
       author_email="[email protected]",
-      url="http://github.com/smarnach/pyexiftool",
+      url="https://github.com/blurstudio/pyexiftool/tree/shell-option",
       classifiers=[
           "Development Status :: 3 - Alpha",
           "Intended Audience :: Developers",