darcs patch: When booting from HC files, search includes for headers

Matthias Kilian kili at outback.escape.de
Wed Nov 11 18:02:54 EST 2009


On Mon, Nov 09, 2009 at 10:53:18AM +0000, Simon Marlow wrote:
>  ifeq "$(BootingFromHc)" "YES"
> -SRC_CC_OPTS += -DNO_REGS -DUSE_MINIINTERPRETER -D__GLASGOW_HASKELL__=$(ProjectVersionInt)
> +SRC_CC_OPTS += -I$(GHC_INCLUDE_DIR) -DNO_REGS -DUSE_MINIINTERPRETER -D__GLASGOW_HASKELL__=$(ProjectVersionInt)
>  endif
> 
> I have a bad feeling about this: it looks like the wrong place to put a 
> -I option.  Do you have a record of the failure that you get without 
> that option?  Perhaps we can find a better place to put it.

Would it be ok to move the -I$(GHC_INCLUDE_DIR) portion to ghc/ghc.mk?

For example, add this one to the BootingFromHC section of ghc/ghc.mk:

	ghc_CC_OPTS += -I$(GHC_INCLUDE_DIR)

Ciao,
	Kili



More information about the Cvs-ghc mailing list