Don't use uname -o
authorBen Gamari <ben@smart-cactus.org>
Tue, 15 Aug 2017 00:45:40 +0000 (20:45 -0400)
committerBen Gamari <ben@smart-cactus.org>
Tue, 15 Aug 2017 00:45:40 +0000 (20:45 -0400)
Summary:
It's not required by the POSIX specification and OS X doesn't support
it; instead use uname -s.

Test Plan: V

Reviewers: O26 nofib, michalt, mpickering

Reviewed By: O26 nofib, michalt, mpickering

Subscribers: mpickering

GHC Trac Issues: #13711

Differential Revision: https://phabricator.haskell.org/D3594

mk/boilerplate.mk
runstdtest/runstdtest.prl

index 45344ca..c2a5cd6 100644 (file)
@@ -23,7 +23,7 @@ CONTEXT_DIFF_RAW = diff -U 1
 EXECUTABLE_FILE = chmod +x
 
 # Windows MSYS specific settings
-ifeq ($(shell uname -o), Msys)
+ifeq ($(shell uname -s | grep -c MSYS), 1)
        exeext=.exe
        CONTEXT_DIFF=$(CONTEXT_DIFF_RAW) --strip-trailing-cr
 else
index 8af3c0b..f1ebbd8 100644 (file)
@@ -57,7 +57,7 @@ if ( $ENV{'TMPDIR'} ) { # where to make tmp file names
 }
 $cmp = "cmp -s";
 # If this is Msys, ignore eol and CR characters.
-if ( `uname -o | grep Msys` ) {
+if ( `uname -s | grep MSYS` ) {
     $CONTEXT_DIFF=$CONTEXT_DIFF . " --strip-trailing-cr" ;
     $cmp = $CONTEXT_DIFF . " -q";
 }