Skip to content

Commit

Permalink
Merge pull request #1229 from lamblin/update_comment
Browse files Browse the repository at this point in the history
Update comment after code review.
  • Loading branch information
Olivier Delalleau committed Feb 14, 2013
2 parents 9bb801c + 2c60952 commit 1806b02
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions theano/misc/windows.py
Expand Up @@ -14,8 +14,12 @@ def call_subprocess_Popen(command, **params):
except AttributeError:
startupinfo.dwFlags |= subprocess._subprocess.STARTF_USESHOWWINDOW

# Under Windows 7 64-bits, Anaconda's g++ is not found unless
# specifying "shell=True".
# Anaconda for Windows does not always provide .exe files
# in the PATH, they also have .bat files that call the corresponding
# executable. For instance, "g++.bat" is in the PATH, not "g++.exe"
# Unless "shell=True", "g++.bat" is not executed when trying to
# execute "g++" without extensions.
# (Executing "g++.bat" explicitly would also work.)
params['shell'] = True
proc = subprocess.Popen(command, startupinfo=startupinfo, **params)
return proc

0 comments on commit 1806b02

Please sign in to comment.