Re: [PATCH] Avoid rebuilding everything if user changes CFLAGS on the make command line

[Junio C Hamano <junkio@cox.net>]

Johannes Sixt <J.Sixt@eudaptics.com> writes:

> IMHO, the value of $(CFLAGS) (which the Makefile declares as "for the
> users to override from the command line") should never take part in this
> build-flags-change-autodetection.

This on the surface looks nicer than the other Johannes's
approach, but I have reservations.  The user can always say
"make CFLAGS=-DNO_SYNLINK_HEAD=NoThanks" from the command line
to affect the behaviour, and in such a case this approach
breaks.

As long as you declare CFLAGS is there for users to override
only the way the programs are compiled (e.g. optimization) and
it is forbidden to use CFLAGS to affect the way the programs are
to behave (iow, "use of -D<FEATURE> in CFLAGS is forbidden"), it
would be Ok, but in effect it is forcing people to modify
config.mak even for "try out this configuration just this time,
one-shot" compilation, which makes me somewhat unhappy.

IGNORE_GIT_CFLAGS looks much uglier, but is much nicer from
design standpoint.  At least the user declares "I know what I am
doing, and I assume full responsibility".  That might be less
confusing in the end.

Undecided.