2011/3/20 Tanu Kaskinen <ta...@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-@PA_MAJORMINOR@.la >> libpulsecommon-@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@mail.0pointer.de > https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss > _______________________________________________ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss