Re: OS X, enable embed problem, patch

Hi Chris, I'm very interested in having --enable-embed working on OS X. My experience with libtool is that it abstracts this sort of thing, so I'm a little bit confused by your work-around. Could you explain why you added --enable-embed=dylib instead of fixing --enable-embed=shared? Thanks, Paul On Wed, Apr 30, 2008 at 6:14 PM, Christopher Thompson <cthompson@nexopia.com> wrote: > Although there's obviously some debate as to the usefulness of my embed > patch for OS X, here's a replacement for the Makefile.global bit. It adds > $(PHP_LDFLAGS) to the compile line for libphp5.dylib, necessary in many > cases. > > > --- php-5.2.5.clean/Makefile.global 2007-08-03 08:01:56.000000000 -0600 > +++ php-5.2.5/Makefile.global 2008-04-29 17:00:20.000000000 -0600 > > @@ -17,6 +17,10 @@ > $(LIBTOOL) --mode=link $(CC) $(CFLAGS) $(EXTRA_CFLAGS) -rpath > $(phptempdir) $(EXTRA_LDFLAGS) $(LDFLAGS) $(PHP_RPATHS) $(PHP_GLOBAL_OBJS) > $(PHP_SAPI_OBJS) $(EXTRA_LIBS) $(ZEND_EXTRA_LIBS) -o $@ > -@$(LIBTOOL) --silent --mode=install cp $@ $(phptempdir)/$@ > >/dev/null 2>&1 > > +libphp$(PHP_MAJOR_VERSION).dylib: $(PHP_GLOBAL_OBJS) $(PHP_SAPI_OBJS) > + $(LIBTOOL) --mode=link $(CC) -dynamiclib -install_name > $(INSTALL_ROOT)$(prefix)/lib/$@ -current_version $(PHP_VERSION) > -compatibility_version $(PHP_MAJOR_VERSION) -undefined dynamic_lookup > $(PHP_RPATHS) $(PHP_LDFLAGS) $(PHP_GLOBAL_OBJS:.lo=.o) > $(PHP_SAPI_OBJS:.lo=.o) $(EXTRA_LIBS) $(ZEND_EXTRA_LIBS) -o $@ > > + -@$(LIBTOOL) --silent --mode=install cp $@ $(phptempdir)/$@ > >/dev/null 2>&1 > + > libs/libphp$(PHP_MAJOR_VERSION).bundle: $(PHP_GLOBAL_OBJS) > $(PHP_SAPI_OBJS) > $(CC) $(MH_BUNDLE_FLAGS) $(CFLAGS_CLEAN) $(EXTRA_CFLAGS) $(LDFLAGS) > $(EXTRA_LDFLAGS) $(PHP_GLOBAL_OBJS:.lo=.o) $(PHP_SAPI_OBJS:.lo=.o) > $(PHP_FRAMEWORKS) $(EXTRA_LIBS) $(ZEND_EXTRA_LIBS) -o $@ && cp $@ > libs/libphp$(PHP_MAJOR_VERSION).so > > > > -- > PHP Internals - PHP Runtime Development Mailing List > To unsubscribe, visit: http://www.php.net/unsub.php > > -- Paul Biggar paul.biggar@gmail.com