On Sun, Mar 20, 2011 at 7:14 PM, Tanu Kaskinen <ta...@iki.fi> wrote: > 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.
Yes, of course. I'll fix this. > Cosmetic: spaces around "+=" would be nice (it seems that it wasn't > entirely consistent before either...) I'd rather keep it in sync with the other lines. Someone could come up with a patch to fix them all :) Daniel _______________________________________________ pulseaudio-discuss mailing list pulseaudio-discuss@mail.0pointer.de https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss
