Just go ahead and check in the patch. It is easier than trying to patch the existing code and it really can't break the trunk build much more than it already is ;) If it doesn't work, then we can just try again.
Brad >>> On Wed, Nov 14, 2007 at 11:46 AM, in message <[EMAIL PROTECTED]>, "Bernard Li" <[EMAIL PROTECTED]> wrote: > Brad, Dan: > > Please let me know if the attached patch solves the problem (please > ignore the SNAPSHOT stuff): > > Index: configure.in > =================================================================== > --- configure.in (revision 868) > +++ configure.in (working copy) > @@ -45,7 +45,7 @@ > > # If we are releasing a snapshot (beta), set GANGLIA_SNAPSHOT to "yes" > # If we are creating an official release, set it to "no" > -GANGLIA_SNAPSHOT="no" > +GANGLIA_SNAPSHOT="yes" > > # If there are any changes to libganglia in this release, you need to... > # LIBGANGLIA_MICRO_VERSION += 1; > @@ -518,10 +518,14 @@ > > dnl Python module config files need to know where to look for > libraries (/usr/lib vs /usr/lib64) > AC_SUBST_FILE(libdir) > +if test "x$prefix" = xNONE; then > + prefix="$ac_default_prefix" > +fi > + > if test "x$host_cpu" = "xx86_64"; then > - libdir=/usr/lib64 > + libdir="$prefix/lib64" > else > - libdir=/usr/lib > + libdir="$prefix/lib" > fi > > AC_OUTPUT(Makefile > > Cheers, > > Bernard > > On 11/9/07, Bernard Li <[EMAIL PROTECTED]> wrote: >> Hi Brad: >> >> On 11/9/07, Brad Nicholes <[EMAIL PROTECTED]> wrote: >> >> > I think that the libdir variable that was added to configure.in is > getting in the way of the --prefix. I have set up my environment to install > the binaries in a different directory for debugging purposes. Now when I run > 'make install', it is trying to install the libraries to /usr/lib rather than > the --prefix location. The following script in configure.in needs to respect > --prefix as well. >> > >> > +dnl Python module config files need to know where to look for libraries > (/usr/lib vs /usr/lib64) >> > +AC_SUBST_FILE(libdir) >> > +if test "x$host_cpu" = "xx86_64"; then >> > + libdir=/usr/lib64 >> > +else >> > + libdir=/usr/lib >> > +fi >> >> Sorry about that -- I'm looking into a generic solution to solve this >> -- please feel free to fix it if you beat me to it. >> >> Thanks, >> >> Bernard >> ------------------------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configuration files using AJAX and a browser. Download your FREE copy of Splunk now >> http://get.splunk.com/ _______________________________________________ Ganglia-developers mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ganglia-developers
