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 >
