diff options
author | Sebastian Thiel <byronimo@gmail.com> | 2009-11-05 18:27:21 +0100 |
---|---|---|
committer | Sebastian Thiel <byronimo@gmail.com> | 2009-11-05 18:27:21 +0100 |
commit | a67bf61b5017e41740e997147dc88282b09c6f86 (patch) | |
tree | 97072b4ec625406982548d60071962a59a6b1511 /lib/git | |
parent | 5593dc014a41c563ba524b9303e75da46ee96bd5 (diff) | |
download | gitpython-a67bf61b5017e41740e997147dc88282b09c6f86.tar.gz |
git cmd on windows now runs without the shell, see diff for explanation
Diffstat (limited to 'lib/git')
-rw-r--r-- | lib/git/cmd.py | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/lib/git/cmd.py b/lib/git/cmd.py index ab34fa58..97d6613d 100644 --- a/lib/git/cmd.py +++ b/lib/git/cmd.py @@ -17,8 +17,13 @@ execute_kwargs = ('istream', 'with_keep_cwd', 'with_extended_output', 'output_stream' ) extra = {} -if sys.platform == 'win32': - extra = {'shell': True} +# NOTE: Execution through a shell appears to be slightly faster, but in fact +# I consider it a problem whenever complex strings are passed and *interpreted* +# by the shell beforehand. This can cause great confusion and reduces compatability +# between the OS which is why the shell should not be used ( unless it does not work +# otherwise ) +#if sys.platform == 'win32': +# extra = {'shell': False} def dashify(string): return string.replace('_', '-') |