[m-rev.] For review: Fix Darwin shared libs on source distribution

[Ian MacLarty]

On 30 Oct 2004, at 12:51, Julien Fischer wrote:

>
> On Sat, 30 Oct 2004, Ian MacLarty wrote:
>
>> For review by anyone.
>>
>> Please note the comment for library/Mmakefile.  Is this acceptable?
>>
>> Estimated hours taken: 4
>> Branches: main
>>
>> Fix shared libs on Mac OS X for the source distribution.  The problem 
>> is that
>> the source distribution is generated on a Linux PC, so the 
>> Darwin-specific
>> -install_name option is not passed to the linker in the .dep files.
>>
> I guess this fixes the immediate problem but you will end up specifying
> -install_name twice if the source distribution is generated on Mac OS 
> X.
> (you've mentioned that below but it should be more prominent in the log
> message).
>
I'm beginning to think my original changes which added some extra 
options to the Mercury compiler might have been more trouble than 
they're worth.  The idea was to get the -install_name option to be 
passed correctly to the automatically generated .dep files, but this 
makes them non-portable since -install_name is a Darwin specific 
option.  (Are the .dep files meant to be portable?)

Maybe a better solution is to disable the new mmc options altogether 
and just pass the -install_name option manually in all the relevant 
Mmakefiles which would avoid the problem with it getting passed twice 
if the source distribution were generated and compiled on a Mac OS X 
system.

If a user wanted to build a shared lib on Darwin the new mmc options 
would still be passed - we just disable them for the Mercury build.  
Alternatively maybe the new mmc options should be removed altogether 
(since they result in non-portable .dep files) and replaced with some 
comments in the user guide about how to set up a Mmakefile to build 
shared libs on Mac OS X.

Any thoughts?

Ian.

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