When calling subprocess.Popen on windows, don't open command line windows.

theano/configdefaults.py

@@ -6,6 +6,7 @@ from theano.configparser import (
         AddConfigVar, BoolParam, ConfigParam, EnumStr, IntParam,
         TheanoConfigParser)
 from theano.misc.cpucount import cpuCount
+from theano.misc.windows import call_subprocess_Popen
 
 _logger = logging.getLogger('theano.configdefaults')
 
@@ -99,8 +100,9 @@ enum = EnumStr("g++", "")
 # in an unusual Python 2.4.4 Windows environment with the default stdin=None.
 dummy_stdin = open(os.devnull)
 try:
-    subprocess.Popen('g++', stdout=subprocess.PIPE, stderr=subprocess.PIPE,
-                     stdin=dummy_stdin.fileno())
+    call_subprocess_Popen('g++', stdout=subprocess.PIPE,
+                          stderr=subprocess.PIPE,
+                          stdin=dummy_stdin.fileno())
     # Keep the default linker the same as the one for the mode FAST_RUN
     AddConfigVar('linker',
                  ("Default linker used if the theano flags mode is Mode "

theano/gof/cmodule.py

@@ -22,6 +22,7 @@ import theano
 from theano.gof.utils import flatten
 from theano.configparser import config
 from theano.gof.cc import hash_from_code
+from theano.misc.windows import call_subprocess_Popen
 
 # we will abuse the lockfile mechanism when reading and writing the registry
 import compilelock
@@ -1592,7 +1593,7 @@ class GCC_compiler(object):
             print >> sys.stderr, ' '.join(cmd)
 
         try:
-            p = subprocess.Popen(cmd, stderr=subprocess.PIPE)
+            p = call_subprocess_Popen(cmd, stderr=subprocess.PIPE)
             compile_stderr = p.communicate()[1]
         except Exception:
             # An exception can occur e.g. if `g++` is not found.

theano/gof/compiledir.py

@@ -3,8 +3,8 @@ import errno
 import os
 import platform
 import re
-import subprocess
 import shutil
+import subprocess
 import sys
 import textwrap
 
@@ -13,6 +13,7 @@ import numpy
 import theano
 from theano.configparser import config, AddConfigVar, ConfigParam, StrParam
 from theano.gof.utils import flatten
+from theano.misc.windows import call_subprocess_Popen
 
 # Using the dummy file descriptors below is a workaround for a crash
 # experienced in an unusual Python 2.4.4 Windows environment with the default
@@ -21,8 +22,10 @@ dummy_in = open(os.devnull)
 dummy_err = open(os.devnull, 'w')
 p = None
 try:
-    p = subprocess.Popen(['g++', '-dumpversion'], stdout=subprocess.PIPE,
-                         stdin=dummy_in.fileno(), stderr=dummy_err.fileno())
+    p = call_subprocess_Popen(['g++', '-dumpversion'],
+                              stdout=subprocess.PIPE,
+                              stdin=dummy_in.fileno(),
+                              stderr=dummy_err.fileno())
     p.wait()
     gcc_version_str = p.stdout.readline().strip()
 except OSError:

theano/gof/op.py

@@ -20,6 +20,7 @@ import warnings
 
 import theano
 from theano import config
+from theano.misc.windows import call_subprocess_Popen
 
 import cc
 import graph
@@ -788,10 +789,10 @@ class OpenMPOp(Op):
                 os.write(fd, code)
                 os.close(fd)
                 fd = None
-                proc = subprocess.Popen(['g++', '-fopenmp', path],
-                                        stdout=subprocess.PIPE,
-                                        stderr=subprocess.PIPE,
-                                        stdin=dummy_stdin.fileno())
+                proc = call_subprocess_Popen(['g++', '-fopenmp', path],
+                                             stdout=subprocess.PIPE,
+                                             stderr=subprocess.PIPE,
+                                             stdin=dummy_stdin.fileno())
                 proc.wait()
                 if proc.returncode != 0:
                     default_openmp = False

theano/sandbox/cuda/nvcc_compiler.py

@@ -15,6 +15,7 @@ from theano.gof.cmodule import (std_libs, std_lib_dirs,
                                 std_include_dirs, dlimport,
                                 get_lib_extension, local_bitwidth)
 from theano.gof.python25 import any
+from theano.misc.windows import call_subprocess_Popen
 
 _logger = logging.getLogger("theano.sandbox.cuda.nvcc_compiler")
 _logger.setLevel(logging.WARN)
@@ -64,8 +65,9 @@ nvcc_version = None
 def is_nvcc_available():
     """Return True iff the nvcc compiler is found."""
     def set_version():
-        p = subprocess.Popen([nvcc_path, '--version'], stdout=subprocess.PIPE,
-                             stderr=subprocess.PIPE)
+        p = call_subprocess_Popen([nvcc_path, '--version'],
+                                  stdout=subprocess.PIPE,
+                                  stderr=subprocess.PIPE)
         p.wait()
         s = p.stdout.readlines()[-1].split(',')[1].strip().split()
         assert s[0] == 'release'

theano/tests/run_tests_in_batch.py

@@ -60,6 +60,7 @@ import subprocess
 import sys
 import datetime
 import theano
+from theano.misc.windows import call_subprocess_Popen
 
 
 def main(stdout=None, stderr=None, argv=None, theano_nose=None,
@@ -264,7 +265,7 @@ def run(stdout, stderr, argv, theano_nose, batch_size, time_profile,
                                                          data["ids"][test_id]))
                 f_rawlog.flush()
 
-                proc = subprocess.Popen(
+                proc = call_subprocess_Popen(
                     ([python, theano_nose, '-v', '--with-id']
                     + [str(test_id)] + argv +
                      ['--disabdocstring']),