]> granicus.if.org Git - nethack/commitdiff
hints/*.2020 vs CFLAGS, take 3...
authorPatR <rankin@nethack.org>
Wed, 9 Sep 2020 21:40:22 +0000 (14:40 -0700)
committerPatR <rankin@nethack.org>
Wed, 9 Sep 2020 21:40:22 +0000 (14:40 -0700)
Avoid use of GNU make's 'override' feature by requiring
'make CCFLAGS=-O' to replace -g from the make command line instead
of 'make CFLAGS=-O'.  Note the extra 'C' in the spelling.

Revert the previous umpteen MORECFLAGS+= back to normal CFLAGS+=.

sys/unix/hints/linux.2020
sys/unix/hints/macOS.2020

index e5cefb7770b7dbebe36c19de229176b5abb40c82..6928df837f616aeb4c81a7648576bd6d0114a20f 100755 (executable)
@@ -1,4 +1,4 @@
-# NetHack 3.7  linux.2020 $NHDT-Date: 1599593762 2020/09/08 19:36:02 $  $NHDT-Branch: NetHack-3.7 $:$NHDT-Revision: 1.10 $
+# NetHack 3.7  linux.2020 $NHDT-Date: 1599687610 2020/09/09 21:40:10 $  $NHDT-Branch: NetHack-3.7 $:$NHDT-Revision: 1.11 $
 # Copyright (c) Kenneth Lorber, Kensington, Maryland, 2007.
 # NetHack may be freely redistributed.  See license for details. 
 #
 # linux.2020 hints file provides a single-user build for Linux (such
 # as Ubuntu focal).
 
-# compiler flags:  various -I, -D, and -W get appended below;
+# compiler flags:  CCFLAGS is used to construct a value for CFLAGS with
+#      various -I, -D, and -W settings appended below;
 #      these are the settings of most interest for an end-user build
 #      (clang doesn't support '-Og', gcc needs 4.x or later)
-CFLAGS = -g
-#CFLAGS = -g -Og
-#CFLAGS = -O2
+CCFLAGS = -g
+#CCFLAGS = -g -Og
+#CCFLAGS = -O2
+#      Note: this is not the usual 'CFLAGS' which is used in default
+#      rules for compiling C code; specifying a value for that on the
+#      'make' command line should be avoided.
 
 # note: '#-INCLUDE' is not just a comment; multiw-1 contains sections 1 to 3
 #-INCLUDE multiw-1.2020
@@ -53,42 +57,41 @@ GAMEGRP  = games
 
 #-INCLUDE multiw-2.2020
 
-# MORECFLAGS is eventually appended to CFLAGS
-MORECFLAGS=-I../include -DNOTPARMDECL
+CFLAGS=$(CCFLAGS) -I../include -DNOTPARMDECL
 
 ifeq "$(CCISCLANG)" ""
 # get the version of gcc
 GCCGTEQ9 := $(shell expr `$(CC) -dumpversion | cut -f1 -d.` \>= 9)
 ifeq "$(GCCGTEQ9)" "1"
-MORECFLAGS+=-Wno-format-overflow
+CFLAGS+=-Wno-format-overflow
 endif  #gcc version greater than or equal to 9
 endif  #not clang
 # As of LLVM build 2336.1.00, this gives dozens of spurious messages, so
 # leave it out by default.
-#MORECFLAGS+=-Wunreachable-code
-#MORECFLAGS+=-Wall -Wextra -Wno-missing-field-initializers -Wimplicit \
+#CFLAGS+=-Wunreachable-code
+#CFLAGS+=-Wall -Wextra -Wno-missing-field-initializers -Wimplicit \
 #      -Wreturn-type -Wunused -Wformat -Wswitch -Wshadow -Wwrite-strings
-#MORECFLAGS+=-DGCC_WARN
+#CFLAGS+=-DGCC_WARN
 
 # NetHack sources control
-MORECFLAGS+=-DDLB
-MORECFLAGS+=-DHACKDIR=\"$(HACKDIR)\"
-MORECFLAGS+=-DDEFAULT_WINDOW_SYS=\"$(WANT_DEFAULT)\"
-MORECFLAGS+=-DSYSCF -DSYSCF_FILE=\"$(HACKDIR)/sysconf\" -DSECURE
-MORECFLAGS+=-DTIMED_DELAY
-MORECFLAGS+=-DDUMPLOG
-MORECFLAGS+=-DCONFIG_ERROR_SECURE=FALSE
-#MORECFLAGS+=-DGREPPATH=\"/usr/bin/grep\"
-MORECFLAGS+=-DCOMPRESS=\"/bin/gzip\" -DCOMPRESS_EXTENSION=\".gz\"
-#MORECFLAGS+=-DNOMAIL
-#MORECFLAGS+=-DEXTRA_SANITY_CHECKS
-#MORECFLAGS+=-DEDIT_GETLIN
-#MORECFLAGS+=-DSCORE_ON_BOTL
-#MORECFLAGS+=-DMSGHANDLER
-#MORECFLAGS+=-DTTY_TILES_ESCCODES
-#MORECFLAGS+=-DTTY_SOUND_ESCCODES
-
-MORECFLAGS+= $(WINCFLAGS)      #WINCFLAGS set from multiw-2.2020
+CFLAGS+=-DDLB
+CFLAGS+=-DHACKDIR=\"$(HACKDIR)\"
+CFLAGS+=-DDEFAULT_WINDOW_SYS=\"$(WANT_DEFAULT)\"
+CFLAGS+=-DSYSCF -DSYSCF_FILE=\"$(HACKDIR)/sysconf\" -DSECURE
+CFLAGS+=-DTIMED_DELAY
+CFLAGS+=-DDUMPLOG
+CFLAGS+=-DCONFIG_ERROR_SECURE=FALSE
+#CFLAGS+=-DGREPPATH=\"/usr/bin/grep\"
+CFLAGS+=-DCOMPRESS=\"/bin/gzip\" -DCOMPRESS_EXTENSION=\".gz\"
+#CFLAGS+=-DNOMAIL
+#CFLAGS+=-DEXTRA_SANITY_CHECKS
+#CFLAGS+=-DEDIT_GETLIN
+#CFLAGS+=-DSCORE_ON_BOTL
+#CFLAGS+=-DMSGHANDLER
+#CFLAGS+=-DTTY_TILES_ESCCODES
+#CFLAGS+=-DTTY_SOUND_ESCCODES
+
+CFLAGS+= $(WINCFLAGS)  #WINCFLAGS set from multiw-2.2020
 
 VARDATND =
 VARDATND0 =
@@ -122,7 +125,7 @@ X11CFLAGS = -I/opt/X11/include
 # avoid repeated complaints about _X_NONNULL(args...) in <X11/Xfuncproto.h>
 X11CFLAGS += -Wno-variadic-macros
 ifdef USE_XPM
-MORECFLAGS += -DUSE_XPM
+CFLAGS += -DUSE_XPM
 WINX11LIB += -lXpm
 VARDATND0 += rip.xpm
 endif
@@ -161,13 +164,6 @@ WINOBJ = $(sort $(WINOBJ0))
 # prevent duplicates in VARDATND if both X11 and Qt are being supported
 VARDATND += $(sort $(VARDATND0))
 
-# if CFLAGS=foo was specified on the command line, CFLAGS+=$(MORECFLAGS)
-# would be ignored without 'override'; we want to allow CFLAGS=-g vs -O
-# on the command line without having to specify all the other stuff
-#
-override CFLAGS+=$(MORECFLAGS)
-#
-
 
 #PREFIX=/usr
 PREFIX=$(wildcard ~)/nh/install
index 212c8581be7fca9cf579609f73f605e88e8d4d5d..937e5515163ed3b794be88841b7463a5c7492b4a 100755 (executable)
@@ -1,4 +1,4 @@
-# NetHack 3.7  macOS.2020 $NHDT-Date: 1599593767 2020/09/08 19:36:07 $  $NHDT-Branch: NetHack-3.7 $:$NHDT-Revision: 1.72 $
+# NetHack 3.7  macOS.2020 $NHDT-Date: 1599687615 2020/09/09 21:40:15 $  $NHDT-Branch: NetHack-3.7 $:$NHDT-Revision: 1.73 $
 # Copyright (c) Kenneth Lorber, Kensington, Maryland, 2015.
 # NetHack may be freely redistributed.  See license for details.
 #
 # macOS X hints file
 #
 
-# compiler flags:  various -I, -D, and -W get appended below;
+# compiler flags:  CCFLAGS is used to construct a value for CFLAGS with
+#      various -I, -D, and -W settings appended below;
 #      these are the settings of most interest for an end-user build
 #      (clang doesn't support '-Og', gcc needs 4.x or later)
-CFLAGS = -g
-#CFLAGS = -g -Og
-#CFLAGS = -O2
+CCFLAGS = -g
+#CCFLAGS = -g -Og
+#CCFLAGS = -O2
+#      Note: this is not the usual 'CFLAGS' which is used in default
+#      rules for compiling C code; specifying a value for that on the
+#      'make' command line should be avoided.
 
 # note: '#-INCLUDE' is not just a comment; multiw-1 contains sections 1 to 3
 #-INCLUDE multiw-1.2020
@@ -55,12 +59,11 @@ endif
 
 #-INCLUDE multiw-2.2020
 
-# MORECFLAGS is eventually appended to CFLAGS
-MORECFLAGS=-I../include -DNOTPARMDECL
+CFLAGS=$(CCFLAGS) -I../include -DNOTPARMDECL
 
 ifndef WANT_WIN_QT
 #      these are normally used when compiling nethack's core
-MORECFLAGS+=-ansi -pedantic -Wno-long-long
+CFLAGS+=-ansi -pedantic -Wno-long-long
 #      but -ansi forces -std=c90 for C or -std=c++98 for C++;
 #      win/Qt/qt_*.cpp compiled with C++98 semantics trigger
 #In file included from .../qt5/include/QtCore/qglobal.h:105:
@@ -70,30 +73,30 @@ MORECFLAGS+=-ansi -pedantic -Wno-long-long
 endif
 # As of LLVM build 2336.1.00, this gives dozens of spurious messages, so
 # leave it out by default.
-#MORECFLAGS+=-Wunreachable-code
-MORECFLAGS+=-Wall -Wextra -Wno-missing-field-initializers -Wimplicit \
+#CFLAGS+=-Wunreachable-code
+CFLAGS+=-Wall -Wextra -Wno-missing-field-initializers -Wimplicit \
        -Wreturn-type -Wunused -Wformat -Wswitch -Wshadow -Wwrite-strings
-MORECFLAGS+=-DGCC_WARN
+CFLAGS+=-DGCC_WARN
 
 # NetHack sources control
-MORECFLAGS+=-DDLB
-MORECFLAGS+=-DHACKDIR=\"$(HACKDIR)\"
-MORECFLAGS+=-DDEFAULT_WINDOW_SYS=\"$(WANT_DEFAULT)\" -DDLB
-MORECFLAGS+=-DSYSCF -DSYSCF_FILE=\"$(HACKDIR)/sysconf\" -DSECURE
-#MORECFLAGS+=-DTIMED_DELAY
-#MORECFLAGS+=-DDUMPLOG
-#MORECFLAGS+=-DCONFIG_ERROR_SECURE=FALSE
-MORECFLAGS+=-DGREPPATH=\"/usr/bin/grep\"
-#MORECFLAGS+=-DCOMPRESS=\"/bin/gzip\" -DCOMPRESS_EXTENSION=\".gz\"
-MORECFLAGS+=-DNOMAIL
-#MORECFLAGS+=-DEXTRA_SANITY_CHECKS
-#MORECFLAGS+=-DEDIT_GETLIN
-#MORECFLAGS+=-DSCORE_ON_BOTL
-#MORECFLAGS+=-DMSGHANDLER
-#MORECFLAGS+=-DTTY_TILES_ESCCODES
-#MORECFLAGS+=-DTTY_SOUND_ESCCODES
-
-MORECFLAGS+= $(WINCFLAGS)   #WINCFLAGS set from multiw-2.2020
+CFLAGS+=-DDLB
+CFLAGS+=-DHACKDIR=\"$(HACKDIR)\"
+CFLAGS+=-DDEFAULT_WINDOW_SYS=\"$(WANT_DEFAULT)\" -DDLB
+CFLAGS+=-DSYSCF -DSYSCF_FILE=\"$(HACKDIR)/sysconf\" -DSECURE
+#CFLAGS+=-DTIMED_DELAY
+#CFLAGS+=-DDUMPLOG
+#CFLAGS+=-DCONFIG_ERROR_SECURE=FALSE
+CFLAGS+=-DGREPPATH=\"/usr/bin/grep\"
+#CFLAGS+=-DCOMPRESS=\"/bin/gzip\" -DCOMPRESS_EXTENSION=\".gz\"
+CFLAGS+=-DNOMAIL
+#CFLAGS+=-DEXTRA_SANITY_CHECKS
+#CFLAGS+=-DEDIT_GETLIN
+#CFLAGS+=-DSCORE_ON_BOTL
+#CFLAGS+=-DMSGHANDLER
+#CFLAGS+=-DTTY_TILES_ESCCODES
+#CFLAGS+=-DTTY_SOUND_ESCCODES
+
+CFLAGS+= $(WINCFLAGS)   #WINCFLAGS set from multiw-2.2020
 
 VARDATND =
 VARDATND0 =
@@ -127,7 +130,7 @@ X11CFLAGS = -I/opt/X11/include
 # avoid repeated complaints about _X_NONNULL(args...) in <X11/Xfuncproto.h>
 X11CFLAGS += -Wno-variadic-macros
 ifdef USE_XPM
-MORECFLAGS += -DUSE_XPM
+CFLAGS += -DUSE_XPM
 WINX11LIB += -lXpm
 VARDATND0 += rip.xpm
 endif
@@ -165,13 +168,6 @@ WINOBJ = $(sort $(WINOBJ0))
 # prevent duplicates in VARDATND if both X11 and Qt are being supported
 VARDATND += $(sort $(VARDATND0))
 
-# if CFLAGS=foo was specified on the command line, CFLAGS+=$(MORECFLAGS)
-# would be ignored without 'override'; we want to allow CFLAGS=-g vs -O
-# on the command line without having to specify all the other stuff
-#
-override CFLAGS+=$(MORECFLAGS)
-#
-
 
 WANT_BUNDLE=1
 ifdef WANT_SHARE_INSTALL