Commit abcd1f29 authored by Steve Beattie's avatar Steve Beattie

This patch makes the parser's makefile honor CFLAGS that have been

exported in the environment. Without it, merely setting the CFLAGS
environment variable would not affect the compilation of the parser,
though it was still possible to override it by passing the variable
as an argument (e.g. make all CFLAGS="-Oinsane -Wextra-special").

It also makes the default CFLAGS for the parser consistent with
the default for the C++ dfa library, and passes the flags on to
the library.

An audit of the other bits of C showed that they either supported
CFLAGS during configure or were otherwise honoring CFLAGS when set
as environment variable.
parent 5849c7ab
......@@ -45,11 +45,14 @@ WARNINGS += $(shell for warning in ${EXTRA_WARNINGS} ; do \
echo "$${warning}"; \
fi ; \
done)
CFLAGS = -O2 -pipe
ifndef CFLAGS
CFLAGS = -g -O2 -pipe
ifdef DEBUG
CFLAGS = -g
endif
endif #CFLAGS
EXTRA_CFLAGS = ${CFLAGS} ${WARNINGS} -D_GNU_SOURCE
#LEXLIB := -lfl
......@@ -232,7 +235,7 @@ check: tests
.SILENT: $(AAREOBJECTS)
.PHONY: $(AAREOBJECTS)
$(AAREOBJECTS):
make -C $(AAREDIR)
make -C $(AAREDIR) CFLAGS="$(CFLAGS)"
.PHONY: install-rhel4
install-rhel4: install-redhat
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment