Re: Pretest compilation problem

[Glenn Morris]

Chong Yidong wrote:

> Adding $(CFLAGS) sounds correct,

ALL_CFLAGS used to be there in Emacs 23.1. It was removed in r1.461,
2009-12-16, by Andreas ("Don't compile prefix-args.c twice.")
I'm sure he will be happy to provide a detailed explanation.

http://cvs.savannah.gnu.org/viewvc/emacs/src/Makefile.in?root=emacs&r1=1.460&r2=1.461&pathrev=MAIN

>  except for a comment about this rule in Makefile.in, which I don't
> understand:
>
> /* We do not use ALL_LDFLAGS because LD_SWITCH_SYSTEM and LD_SWITCH_MACHINE
>    often contain options that have to do with using Emacs''s crt0,
>    which are only good with temacs.  */

IIRC, the Makefiles tried to make a distinction between flags that were
only needed to link temacs (see the various TEMACS variables, eg
LD_SWITCH_SYSTEM_TEMACS), and flags that were needed to compile separate
things like prefix-args. It's quite possible that the comment is
inaccurate or outdated though.