From dbda9529a4c056c196d803e75f0b15fe7eacefdc Mon Sep 17 00:00:00 2001
From: Blazej Pindelski <bpindelski@dundee.ac.uk>
Date: Fri, 7 Sep 2012 15:30:07 +0100
Subject: [PATCH] Fix bug #9179: OMERO-cpp.bat fails on notification

Removing the "shell" argument from the Popen call removes the [Errno 22]
error. A check has been also added in "communicate()" to return when run
on Windows platforms, as an early exit from the function.
---
 build.py | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/build.py b/build.py
index 15c07e8823b..6e769c87411 100755
--- a/build.py
+++ b/build.py
@@ -18,13 +18,11 @@
 
 def popen(args, stdin=None, stdout=subprocess.PIPE, stderr=subprocess.PIPE):
         copy = os.environ.copy()
-        shell = (sys.platform == "win32")
         return subprocess.Popen(args,
                 env=copy,
                 stdin=stdin,
                 stdout=stdout,
-                stderr=stderr,
-                shell=shell)
+                stderr=stderr)
 
 
 def execute(args):
@@ -40,7 +38,7 @@ def notification(msg, prio):
     """
 
     # May want to revert this to be OMERO_BUILD_NOTIFICATION, or whatever.
-    if "OMERO_QUIET" in os.environ:
+    if "OMERO_QUIET" in os.environ or sys.platform == "win32":
         return
 
     try: