[m-rev.] for review: lmc.in

Zoltan Somogyi zs at cs.mu.OZ.AU
Thu Apr 17 16:14:05 AEST 2003


On 17-Apr-2003, Fergus Henderson <fjh at cs.mu.OZ.AU> wrote:
> > +# compilation # process needs will come from that workspace: the libraries
> 
> s/# process/process/

Done.

> The name "SCRIPTS" is misleading, since almost everything in the
> tools directory is a script, but this variable only includes lmc.
> It would be a good idea to add a comment and/or change the name.

I changed the name to CONFIGURED_SCRIPTS.

> Why define both a generic rule and a special-case rule for lmc?

I deleted the special case rule.

> Also, shouldn't the top-level Mmakefile be modified to run mmake in
> the tools directory?

Done. I also added clean_local and realclean_local targets to tools/Mmakefile.

Zoltan.
--------------------------------------------------------------------------
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