[m-rev.] diff: make sure that configure creates the library Mmakefile

Fergus Henderson fjh at cs.mu.OZ.AU
Thu Feb 5 18:38:20 AEDT 2004


On 05-Feb-2004, Fergus Henderson <fjh at cs.mu.OZ.AU> wrote:
> On 05-Feb-2004, Julien Fischer <juliensf at students.cs.mu.OZ.AU> wrote:
> > 
> > James' recent change that placed library/Mmakefile under the control
> > of configure left out the following.
> > 
> > configure.in:
> > 	Have the configure script create library/Mmakefile.
> 
> Thanks.
> 
> But I think a better fix would be to not put library/Mmakefile under the
> control of configure.  According to the log message, the reason it was
> put under the control of configure was so that it would have access to
> FULLARCH, but FULLARCH is already defined in scripts/Mmake.vars(.in),
> so it shouldn't be necessary to define FULLARCH in library/Mmakefile.

I have therefore committed the following patch.

Branches: main

library/Mmakefile.in:
library/Mmakefile:
	Rename Mmakefile.in back to Mmakefile, and delete the line
	"FULLARCH = @FULLARCH@".  The FULLARCH variable is already
	defined by scripts/Mmake.vars, and so doesn't need to be
	defined here in library/Mmakefile, and as a consequence
	library/Mmakefile doesn't need to be generated by configure.

-- 
Fergus Henderson <fjh at cs.mu.oz.au>  |  "I have always known that the pursuit
The University of Melbourne         |  of excellence is a lethal habit"
WWW: <http://www.cs.mu.oz.au/~fjh>  |     -- the last words of T. S. Garp.
--------------------------------------------------------------------------
mercury-reviews mailing list
post:  mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe:   Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------



More information about the reviews mailing list