[m-rev.] for review: library additions for erlang

Julien Fischer juliensf at csse.unimelb.edu.au
Thu Aug 9 11:15:36 AEST 2007


On Wed, 8 Aug 2007, Peter Wang wrote:

> Julien pointed out that the problem is if configure generates/touches
> erlang_conf.m then make would try to use an installed Mercury compiler
> to build the .c file.
>
> Here is a proposed fix.
>
>
> Estimated hours taken: 0.5
> Branches: main
>
> Don't generate library/erlang_conf.m with configure as that introduces a
> bootstrapping problem -- if a .m file is generated then an installed Mercury
> compiler is required to generate .c files from it.  Instead, generate a .hrl
> header file which is only included when building for the Erlang backend.
>
> configure.in:
> library/library.hrl.in:
> 	Generate erlang_conf.hrl instead of erlang_conf.m.
>
> library/library.m:
> 	Make the Erlang implementation of library.version use the macros from
> 	erlang_conf.hrl.
>
> 	Prevent library.version from being inlined into other modules as that
> 	would require us to install erlang_conf.hrl for no gain.
>
> 	Remove references to erlang_conf.m.
>
> library/Mmakefile:
> 	Clean erlang_conf.hrl with the realclean_local target.
>
> library/erlang_conf.m.in:
> 	Delete this.
>

That fix is fine by me.

Julien.
--------------------------------------------------------------------------
mercury-reviews mailing list
Post messages to:       mercury-reviews at csse.unimelb.edu.au
Administrative Queries: owner-mercury-reviews at csse.unimelb.edu.au
Subscriptions:          mercury-reviews-request at csse.unimelb.edu.au
--------------------------------------------------------------------------



More information about the reviews mailing list