Skip to content

Commit df17e77

Browse files
j6tgitster
authored andcommitted
add--interactive: fix external command invocation on Windows
Back in 21e9757 (Hack git-add--interactive to make it work with ActiveState Perl, 2007-08-01), the invocation of external commands was changed to use qx{} on Windows. The rationale was that the command interpreter on Windows is not a POSIX shell, but rather Windows's CMD. That patch was wrong to include 'msys' in the check whether to use qx{} or not: 'msys' identifies MSYS perl as shipped with Git for Windows, which does not need the special treatment; qx{} should be used only with ActiveState perl, which is identified by 'MSWin32'. Signed-off-by: Johannes Sixt <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent e230c56 commit df17e77

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

git-add--interactive.perl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -169,7 +169,7 @@ sub colored {
169169
my %patch_mode_flavour = %{$patch_modes{stage}};
170170

171171
sub run_cmd_pipe {
172-
if ($^O eq 'MSWin32' || $^O eq 'msys') {
172+
if ($^O eq 'MSWin32') {
173173
my @invalid = grep {m/[":*]/} @_;
174174
die "$^O does not support: @invalid\n" if @invalid;
175175
my @args = map { m/ /o ? "\"$_\"": $_ } @_;

0 commit comments

Comments
 (0)