Re: [commit: ghc] master: Give hsc2hs different options in different stages; fixes #7705 (f1fcfff)

If the bootstrapping compiler is the Haskell Platform on Windows this
change can break the build if the Platofrm install directory includes
spaces.
Since before this change the intree gcc was always used on Windows, but now
the bootstrapping ghc's gcc is used for CC_STAGE0 for hsc2hs invocations.
Adding wrapping single quotes like $1_$2_HSC2HS_CC_OPTS and
$1_$2_HSC2HS_LD_OPTS
to the --cc and --ld options seems like the simplest and best solution.
Fyi,
Bill
On Sat, Mar 2, 2013 at 9:43 AM, Ian Lynagh
diff --git a/rules/distdir-opts.mk b/rules/distdir-opts.mk index 748e27e..50cc05e 100644 --- a/rules/distdir-opts.mk +++ b/rules/distdir-opts.mk @@ -79,10 +79,11 @@ $1_$2_HSC2HS_LD_OPTS:=$$(shell for i in $$($1_$2_DIST_LD_OPTS); do echo \'--lfla endif
$1_$2_ALL_HSC2HS_OPTS = \ - --cc=$$(WhatGccIsCalled) \ - --ld=$$(WhatGccIsCalled) \ + --cc=$$(CC_STAGE$3) \ + --ld=$$(CC_STAGE$3) \ $$(CONF_HSC2HS_OPTS) \ $$(SRC_HSC2HS_OPTS) \ + $$(SRC_HSC2HS_OPTS_STAGE$3) \ --cflag=-D__GLASGOW_HASKELL__=$$(if $$(filter 0,$3),$$(GhcCanonVersion),$$(ProjectVersionInt)) \ --cflag=-D$$(HostArch_CPP)_HOST_ARCH=1 \ --cflag=-D$$(HostOS_CPP)_HOST_OS=1 \
_______________________________________________ ghc-commits mailing list ghc-commits@haskell.org http://www.haskell.org/mailman/listinfo/ghc-commits

On Wed, Mar 13, 2013 at 12:18:02AM -0400, Bill Tutt wrote:
If the bootstrapping compiler is the Haskell Platform on Windows this change can break the build if the Platofrm install directory includes spaces.
Ah, thanks. Should be fixed now. Thanks Ian
participants (2)
-
Bill Tutt
-
Ian Lynagh