Remove GhcDynamic (in favor of DYNAMIC_GHC_PROGRAMS)
authorReid Barton <rwbarton@gmail.com>
Wed, 12 Apr 2017 18:10:12 +0000 (14:10 -0400)
committerBen Gamari <ben@smart-cactus.org>
Wed, 12 Apr 2017 18:53:06 +0000 (14:53 -0400)
DYNAMIC_GHC_PROGRAMS and GhcDynamic both tried to control whether
the ghc binary was built as a dynamic executable, with confusing
results. In particular, setting GhcDynamic=NO has no effect on
systems where DYNAMIC_GHC_PROGRAMS defaults to YES.

DYNAMIC_GHC_PROGRAMS is more fully-featured (it ensures that the
correct flavor of the libraries is built, for example) so let's
keep it and remove GhcDynamic to reduce confusion.

This effectively reverts commit 3c6190b0.

Test Plan: tested locally; harbormaster

Reviewers: simonmar, austin, bgamari

Reviewed By: bgamari

Subscribers: thomie, snowleopard

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

ghc/ghc.mk
mk/config.mk.in

index 8d4c1df..319f969 100644 (file)
@@ -53,9 +53,8 @@ ghc_stage2_MORE_HC_OPTS += -debug
 ghc_stage3_MORE_HC_OPTS += -debug
 endif
 
-ifeq "$(GhcDynamic)" "YES"
-ghc_stage2_MORE_HC_OPTS += -dynamic
-ghc_stage3_MORE_HC_OPTS += -dynamic
+ifneq "$(GhcDynamic)" ""
+$(error GhcDynamic is no longer supported, use DYNAMIC_GHC_PROGRAMS instead)
 endif
 
 ifeq "$(GhcThreaded)" "YES"
index 5d1f94a..1f7353c 100644 (file)
@@ -95,7 +95,6 @@ endif
 #   The stage number refers to the compiler stage the options are passed to.
 
 GhcDebugged=NO
-GhcDynamic=NO
 
 # GhcProfiled=YES means compile a profiled stage-2 compiler
 GhcProfiled=NO