Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit a67bf61

Browse files
committed
git cmd on windows now runs without the shell, see diff for explanation
1 parent 5593dc0 commit a67bf61

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

lib/git/cmd.py

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,13 @@
1717
'output_stream' )
1818

1919
extra = {}
20-
if sys.platform == 'win32':
21-
extra = {'shell': True}
20+
# NOTE: Execution through a shell appears to be slightly faster, but in fact
21+
# I consider it a problem whenever complex strings are passed and *interpreted*
22+
# by the shell beforehand. This can cause great confusion and reduces compatability
23+
# between the OS which is why the shell should not be used ( unless it does not work
24+
# otherwise )
25+
#if sys.platform == 'win32':
26+
# extra = {'shell': False}
2227

2328
def dashify(string):
2429
return string.replace('_', '-')

0 commit comments

Comments
 (0)