[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