[m-dev.] cvs diff: bug fixes for runtests
Fergus Henderson
fjh at cs.mu.oz.au
Sat Mar 15 17:58:53 AEDT 1997
Tyson Richard DOWD, you wrote:
>
> > In fact it would be a good idea to change scripts/Mmake.vars.
> > It should define
> >
> > MCFLAGS = $(EXTRA_MCFLAGS)
> >
> > and similarly for all those other variables, we should document in the
> > user's guide that these EXTRA_* variables are intended to be used for
> > adding extra flags on the command line.
>
> I assume you mean ALL the flags variables in Mmake.vars.
Yep.
> This might be a good time to address the inconsistency in
> EXTRA_CFLAGS - really, it should be EXTRA_MGNUCFLAGS, otherwise
> it appears to be associated with CFLAGS.
Uh, that is sort of deliberate. The point is that there should (IMHO)
be a single variable for specifying extra options to the C compiler
on the command line, including times when the C compiler is invoked
directly, rather than via mgnuc.
Probably there ought to be both EXTRA_CFLAGS and EXTRA_MGNUCFLAGS,
and we should by default set
MGNUCFLAGS = $(EXTRA_MGNUCFLAGS) $(EXTRA_CFLAGS)
(Doing that would also ensure that our existing scripts and stuff that
uses EXTRA_CFLAGS wouldn't break.)
--
Fergus Henderson <fjh at cs.mu.oz.au> | "I have always known that the pursuit
WWW: <http://www.cs.mu.oz.au/~fjh> | of excellence is a lethal habit"
PGP: finger fjh at 128.250.37.3 | -- the last words of T. S. Garp.
More information about the developers
mailing list