[PATCH] difftool: use perl built-ins when testing for msys
- From
David Aguilar <davvid@gmail.com>
- Date
- Mar 25, 2009, 06:13 UTC
- Message-ID
- <1237961633-29082-1-git-send-email-davvid@gmail.com>
I don't even know what $COMSPEC means so let's be safe and use the same perly $^O test add--interactive uses. While we're at it, make git-difftool match the prevalent git-perl style.
Signed-off-by: David Aguilar <davvid@gmail.com> --- git-difftool.perl | 7 +++++-- 1 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/git-difftool.perl b/git-difftool.perl index 0deda3a..207dd50 100755 --- a/git-difftool.perl +++ b/git-difftool.perl @@ -33,7 +33,10 @@ sub setup_environment sub exe { my $exe = shift; - return defined $ENV{COMSPEC} ? "$exe.exe" : $exe; + if ($^O eq 'MSWin32' || $^O eq 'msys') { + return "$exe.exe"; + } + return $exe; } sub generate_command @@ -47,7 +50,7 @@ sub generate_command $skip_next = 0; next; } - if ($arg eq '-t' or $arg eq '--tool') { + if ($arg eq '-t' || $arg eq '--tool') { usage() if $#ARGV <= $idx; $ENV{GIT_DIFF_TOOL} = $ARGV[$idx + 1]; $skip_next = 1;
-- 1.6.2.1.303.g63699