[m-rev.] for review: fix problems with mmc --make

Julien Fischer juliensf at cs.mu.OZ.AU
Sat Dec 3 16:12:19 AEDT 2005




On Fri, 2 Dec 2005, Julien Fischer wrote:

> > +:- pred shared_libraries_supported(bool::out, io::di, io::uo) is det.
> > +
> > +shared_libraries_supported(Supported, !IO) :-
> > +    globals__io_lookup_string_option(library_extension, LibExt, !IO),
> > +    globals__io_lookup_string_option(shared_library_extension, SharedLibExt,
> > +        !IO),
> > +    Supported = (if LibExt \= SharedLibExt then yes else no).
> > +
> This does not seem particularly robust.  I think a better approach here would
> be to have configure set a macro in mercury_conf.h (MR_HAVE_SHARED_LIB or
> something) and test that here.
>

Hmmm...that's not going to work either since it will prevent us
reconfiguring installations.

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