build: Error out on implicit function declarations.

[lopsub.git] / Makefile
diff --git a/Makefile b/Makefile

index 3c7bde206f1e977188ab753a6b86f53331cde74c..548c96ff0e6298e29d8f247940a4f446e6757d35 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -9,6 +9,13 @@ endif
  .ONESHELL:
  .SHELLFLAGS := -ec
  
+# Recipes which redirect stdout to the target of the rule (i.e., constructs
+# like cmd > $@) create empty or incomplete output files if the command fails,
+# for example when cmd was not found. Since the target exists and is uptodate
+# in this case, this may lead to all sorts of problems. The following target
+# makes sure that such files are removed.
+.DELETE_ON_ERROR:
+
  PREFIX ?= /usr/local
  M4 := m4
  LN := ln -f
@@ -18,9 +25,18 @@ AR := ar
  GROFF := groff
  CP := cp
  INSTALL := install
-GZIP := gzip -f9
+GZIP := gzip -fn9
  ZCAT := zcat
  
+dummy != $(M4) /dev/null || printf 'failed'
+ifeq ($(dummy), failed)
+$(error m4 is required to build this package)
+endif
+dummy != printf '%%%%\n' | $(LEX) -o /dev/null || printf 'failed'
+ifeq ($(dummy), failed)
+$(error (f)lex is required to build this package)
+endif
+
  DATE_FMT := +%B %Y
  # To get a reproducible build, we use $(SOURCE_DATE_EPOCH) instead of the
  # current time if this variable is set.
@@ -28,7 +44,7 @@ ifdef SOURCE_DATE_EPOCH
         DATE := $(shell LC_ALL=C date -u -d '@$(SOURCE_DATE_EPOCH)' \
                 '$(DATE_FMT)' 2>/dev/null || LC_ALL=C date -u '$(DATE_FMT)')
  else
-       DATE := $(shell date '+$(DATE_FMT)')
+       DATE := $(shell date '$(DATE_FMT)')
  endif
  GIT_VERSION := $(shell ./version-gen.sh)
  PLAIN_VERSION := $(firstword $(subst -, , $(GIT_VERSION)))
@@ -79,13 +95,17 @@ lsg_objs := lopsubgen.o lsg.o lopsubgen.lsg.o lopsub.o version.o
  liblopsub_objs := config_file.o lopsub.o version.o
  lopsubex_objs := lopsubex.o lopsubex.lsg.o $(liblopsub_objs)
  
+LLS_CFLAGS := -g -fPIC
+STRICT_CFLAGS := -Wall
+STRICT_CFLAGS += -Werror-implicit-function-declaration
+
  $(lsg_objs) $(liblopsub_objs) $(lopsubex_objs): %.o: %.c
  lopsubgen.o config_file.o:
-       $(CC) -g -c -fPIC -o $@ ${@:.o=.c}
+       $(CC) $(CPPFLAGS) $(LLS_CFLAGS) $(CFLAGS) -c -o $@ ${@:.o=.c}
  lsg1.o: lsg.c lsg.h
-       $(CC) -g -DSTAGE1 -Wall -g -c $< -o $@
+       $(CC) $(CPPFLAGS) $(LLS_CFLAGS) $(STRICT_CFLAGS) $(CFLAGS) -DSTAGE1 -c -o $@ $<
  %.o: %.c
-       $(CC) -Wall -I. -g -c -fPIC -o $@ $<
+       $(CC) -I. $(CPPFLAGS) $(LLS_CFLAGS) $(STRICT_CFLAGS) $(CFLAGS) -c -o $@ $<
  
  # linking
  lopsubgen-stage1: $(lsg1_objs)
@@ -99,17 +119,14 @@ lopsubex: $(lopsubex_objs) $(REALNAME)
  
  # web
  html := $(addprefix web/, $(addsuffix .html, \
-       index lopsub-api lopsubgen.1 lopsubex.1 $(m4_man_pages)))
-$(html): $(addprefix web/, $(addsuffix .html, header footer))
-
+       index lopsub-api lopsubgen.1 lopsubex.1 $(m4_man_pages:.gz=)))
  www: $(html)
-
-web/lopsub-api.html: lopsub.h.m4
+web/lopsub-api.html: lopsub.h.m4 web/header.html web/footer.html
         $(M4) -DOUTPUT_MODE=HTML web/header.html $(gendoc) \
                 $< web/footer.html > $@
  web/index.html: web/lopsub.7.html
         $(LN) -s $(notdir $<) $@
-web/%.html: %.gz
+web/%.html: %.gz web/header.html
         $(CP) web/header.html $@
         $(ZCAT) $< | $(GROFF) -m man -Thtml | sed -e '1,/^<body>/d' >> $@