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. Cosmetic: spaces around "+=" would be nice (it seems that it wasn't entirely consistent before either...) -- Tanu
