Skip to content

Commit

Permalink
Don't warn about missing strip.exe on Windows
Browse files Browse the repository at this point in the history
We don't expect Windows systems to have the strip program anyway.
  • Loading branch information
dcoutts committed Aug 12, 2008
1 parent 03072ff commit 9076783
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion Distribution/Simple/GHC.hs
Expand Up @@ -728,7 +728,10 @@ stripExe :: Verbosity -> LocalBuildInfo -> FilePath -> FilePath -> IO ()
stripExe verbosity lbi name path = when (stripExes lbi) $
case lookupProgram stripProgram (withPrograms lbi) of
Just strip -> rawSystemProgram verbosity strip args
Nothing -> warn verbosity $ "Unable to strip executable '" ++ name
Nothing -> unless (buildOS == Windows) $
-- Don't bother warning on windows, we don't expect them to
-- have the strip program anyway.
warn verbosity $ "Unable to strip executable '" ++ name
++ "' (missing the 'strip' program)"
where
args = path : case buildOS of
Expand Down

0 comments on commit 9076783

Please sign in to comment.