diff options
| author | Thomas Miedema <thomasmiedema@gmail.com> | 2015-05-28 12:38:21 +0200 |
|---|---|---|
| committer | Thomas Miedema <thomasmiedema@gmail.com> | 2015-06-04 23:58:29 +0200 |
| commit | 5dd02864a844bcf6fe0018755ff261affdef3fea (patch) | |
| tree | f63cbc4b1c95b91ade8b7357aae972772cf1c061 /rules/build-prog.mk | |
| parent | cac68d0d340f93738db79ad867fe3f0eec515075 (diff) | |
| download | haskell-5dd02864a844bcf6fe0018755ff261affdef3fea.tar.gz | |
Build: remove more unnecessary CLEANING/=YES checks
All these checks that CLEANING/=YES are no longer needed, because
nowadays $1_$2_PROGNAME is always set explicitly, and $1_$2_PROG isn't.
They were once introduce to allow `make clean` before `./configure`. I
checked, and it still works.
Remove the checks to make the build system a tiny bit shorter, and to
no longer wonder why they are there.
Differential Revision: https://phabricator.haskell.org/D941
Diffstat (limited to 'rules/build-prog.mk')
| -rw-r--r-- | rules/build-prog.mk | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/rules/build-prog.mk b/rules/build-prog.mk index 3e9bc89e29..f2f6ad2551 100644 --- a/rules/build-prog.mk +++ b/rules/build-prog.mk @@ -22,14 +22,12 @@ $(call profStart, build-prog($1,$2,$3)) # $2 = distdir # $3 = GHC stage to use (0 == bootstrapping compiler) -ifneq "$$(CLEANING)" "YES" ifeq "$$($1_$2_PROGNAME)" "" $$(error $1_$2_PROGNAME is not set) endif ifneq "$$($1_$2_PROG)" "" $$(error $1_$2_PROG is set) endif -endif $1_$2_PROG = $$($1_$2_PROGNAME)$$(exeext$3) ifeq "$$(findstring $3,0 1 2)" "" @@ -91,11 +89,9 @@ else $1_$2_INPLACE = endif else -ifneq "$$(CLEANING)" "YES" ifneq "$$($$($1_$2_PROGNAME)_INPLACE)" "" $$(error $$($1_$2_PROGNAME)_INPLACE defined twice) endif -endif ifeq "$$($1_$2_TOPDIR)" "YES" $$($1_$2_PROGNAME)_INPLACE = $$(INPLACE_TOPDIR)/$$($1_$2_PROG) else |