I haven't tested the 64-bit build functionality that is the original intent of the patch, but this fixes make install which was broken before.
thanks, Brad >>> On Wed, Nov 14, 2007 at 1:05 PM, in message <[EMAIL PROTECTED]>, "Brad Nicholes" <[EMAIL PROTECTED]> wrote: > 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 ------------------------------------------------------------------------- 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
