[PATCH 3/5] osx: add -headerpad_max_install_names to LDFLAGS

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2011/3/20 Tanu Kaskinen <tanuk at iki.fi>:
> On Sun, 2011-03-20 at 18:39 +0100, Daniel Mack wrote:
>> This is needed for sufficient padding of library names in linked
>> binaries.
>> ---
>> ?src/Makefile.am | ? ?5 +++++
>> ?1 files changed, 5 insertions(+), 0 deletions(-)
>>
>> diff --git a/src/Makefile.am b/src/Makefile.am
>> index ff98ddb..c182483 100644
>> --- a/src/Makefile.am
>> +++ b/src/Makefile.am
>> @@ -82,6 +82,11 @@ FOREIGN_CFLAGS = -w
>> ?MODULE_LDFLAGS = $(AM_LDFLAGS) -module -disable-static -avoid-version $(LDFLAGS_NOUNDEFINED)
>> ?MODULE_LIBADD = $(AM_LIBADD) libpulsecore- at PA_MAJORMINOR@.la libpulsecommon- at PA_MAJORMINOR@.la libpulse.la
>>
>> +if OS_IS_DARWIN
>> +AM_LDFLAGS+=-headerpad_max_install_names
>> +MODULE_LDFLAGS+=-headerpad_max_install_names
>> +endif
>
> If you put this block before MODULE_LDFLAGS is defined, then you don't
> need to touch MODULE_LDFLAGS, because they include AM_LDFLAGS
> automatically.

Actually, I think MODULE_LD_FLAGS will pick up the switch twice,
because the update to AM_LDFLAGS gets into the previously defined
MODULE_LDFLAGS. Isn't that how make variables work?

Maarten

> Cosmetic: spaces around "+=" would be nice (it seems that it wasn't
> entirely consistent before either...)
>
> --
> Tanu
>
> _______________________________________________
> pulseaudio-discuss mailing list
> pulseaudio-discuss at mail.0pointer.de
> https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
>



[Index of Archives]     [Linux Audio Users]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux