Re: [PHP4BETA] cvs: /php4 php.map /php4 ChangeLog configure.in.in

On Wed, 8 Sep 1999, Sascha Schumann wrote: > > > +AC_MSG_CHECKING(whether to enable versioning) > > > +AC_ARG_ENABLE(versioning, > > > +[ --enable-versioning], > > > +[ > > > + if test "$enableval" = "yes" ; then > > > + RPATHS="-Wl,--version-script=\$(srcdir)/php.map $RPATHS" > > > + fi > > > +]) > > > > Are you sure you want to add this to the RPATHS variable ? rpaths can be > > quite a nuisance for system integrators, like debian, there were several > > threads/flamewars about this on the debian-devel list between the libtool > > maintainers and debian developers. This is because we don't need rpaths at > > all. So I just removed all references to @RPATHS@ in the php > > makefiles... But this versioning seems quite nice, I wouldn't want to > > remove it. I'm willing to make the modifications needed in the cvs... > > An easier way is to simply leave @RPATHS@ where they are and > remove the lines in acinclude.m4 which add -rpaths to RPATHS. Ok, so you suggest to leave this as is, and when building the debian package, modify acinclude.m4, run aclocal, autoconf, etc, right ? Sounds reasonable. But anyway I think this version flag does not belong to the RPATHS variable, and as I see it, it is just one place to put the new variable: Makefile.in in the main directory. Btw I was wondering... won't the shared modules have issues with this version stuff ? I'm about to check it now... Greg