[m-rev.] for review: mmc --make for external foreign files improvements

Simon Taylor stayl at cs.mu.OZ.AU
Fri Dec 20 10:13:33 AEDT 2002


On 19-Dec-2002, Peter Ross <pro at missioncriticalit.com> wrote:
> On Thu, Dec 19, 2002 at 04:18:05PM +1100, Simon Taylor wrote:
> > On 18-Dec-2002, Peter Ross <pro at missioncriticalit.com> wrote:
> > > Estimated hours taken: 24
> > > Branches: main
> > > 
> > > Change `mmc --make' so that it no longer builds the external foreign
> > > object files at the same time as it builds the target object file.
> > > 
> > > This allows one to build on the IL backend where building an external
> > > foreign file assembly depends on having all the imported Mercury
> > > assemblies built first.
> > > 
> > > Various fixes were also added so that `mmc --make --grade il' could make
> > > an executable.
> > 
> > > compiler/make.module_target.m:
> > > 	Always create a directory to hold the target file before
> > >	building the target file.
> > 
> > Why?
> > 
> because when building tests/hard_coded/factt.m Mercury/cs isn't created
> and one gets an error.  This happens with the current ROTD.

The bug here is that modules__fact_table_file_name should take
an argument to control whether the directory is created, just
as module_name_to_file_name does.

> It also seems to me that this the is the correct time to ensure that the
> directory exists when building the target file.

It's not, because the directory needs to be created even when
compiling without `mmc --make'.

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