APR 1.2.x should be all that is needed. The APR project is very good about making sure that there are no incompatibilities within a minor version.
Also, if you make --enable-static-build work for APR you will also need to make it work for libconfuse and libexpat. The real issue is that if --enable-static-build is specified, the result is that there won't be much difference between Ganglia 3.0.4 and 3.1.x. The reason why is because by building static, it will essentially disable the dynamic metric module capability. The metric modules need to have access to some of the APR functionality (such as memory pools, etc. and anything else the module decides to use) and if APR is statically linked, the dynamic modules don't have access to APR APIs. The --enable-static-build parameter is basically a backward compatibility feature. IMO, I think that we should rip all of the external libraries out of the ganglia tarball and if we still want to distribute them, do it in a separate tarball/RPM. That way we can be sure to install the external libraries in an /opt/ganglia location that will be out of the way of any conflicting distro libraries. Moving forward, the Ganglia binaries should be dynamically linking to any external libraries for technical reasons and to avoid redistributing forked external source. Of course the final call is up to the community, Brad >>> On 5/16/2007 at 1:03 PM, in message <[EMAIL PROTECTED]>, "Bernard Li" <[EMAIL PROTECTED]> wrote: > Hi Brad: > > What version of apr-devel is needed for the new functionalities to > work? I'm going to see if I can hack the spec file such that it will > do --enable-static-build if the particular version of apr is not > available. > > P.S. Are we still planning to update the version of apr we ship with > the tarball...? > > Thanks, > > Bernard > > On 5/9/07, Brad Nicholes <[EMAIL PROTECTED]> wrote: >> >>> On 5/9/2007 at 4:50 PM, in message >> <[EMAIL PROTECTED]>, >> <[EMAIL PROTECTED]> wrote: >> > Revision: 779 >> > http://svn.sourceforge.net/ganglia/?rev=779&view=rev >> > Author: bnicholes >> > Date: 2007-05-09 15:50:58 -0700 (Wed, 09 May 2007) >> > >> > Log Message: >> > ----------- >> > Converted to dynamically link all external libraries by default. Added >> > --with-libapr --with-libexpat --with-libconfuse to specify an alternate >> > location >> > for the headers and libraries. Also added --enable-static-build to link all >> > libraries statically as before. >> > >> > Modified Paths: >> > -------------- >> > trunk/monitor-core/Makefile.am >> > trunk/monitor-core/configure.in >> > trunk/monitor-core/ganglia.spec.in >> > trunk/monitor-core/gmetad/Makefile.am >> > trunk/monitor-core/gmetric/Makefile.am >> > trunk/monitor-core/gmond/Makefile.am >> > trunk/monitor-core/gmond/gstat/Makefile.am >> > trunk/monitor-core/gmond/modules/Makefile.am >> > trunk/monitor-core/lib/Makefile.am >> > trunk/monitor-core/srclib/Makefile.am >> > >> >> I just committed the above patch that converts the makefiles and .spec file > to build the binaries by linking all external libraries dynamically by > default. As stated in the log message, some new --with-libXXX options have > been > added to allow the build to search alternate locations for the external > libraries and headers. A backwards compatibility option has also been added > (--enable-static-build) to link the libraries statically as was done before. > When building dynamically, the make files no longer look for anything in the > /srclib directory unless specifically instructed using the --with-libXXX > options. However when building statically, the code in the /srclib directory > is built and used as before but the new metric module functionality is > disabled because modules depend on APR and must therefore, be linked > dynamically. As a side note, I decided not to replace libexpat with > libapr-util because by default, libapr-util will just defer to libexpat if > the .so has been inst > al >> led. Therefore, until the functionality of libapr-util is necessary, > converting to libapr-util would basically just introduce unnecessary overhead. >> >> Comments? >> >> Brad >> >> >> ------------------------------------------------------------------------- >> This SF.net email is sponsored by DB2 Express >> Download DB2 Express C - the FREE version of DB2 express and take >> control of your XML. No limits. Just data. Click to get it now. >> http://sourceforge.net/powerbar/db2/ >> _______________________________________________ >> Ganglia-developers mailing list >> [email protected] >> https://lists.sourceforge.net/lists/listinfo/ganglia-developers >> > > ------------------------------------------------------------------------- > This SF.net email is sponsored by DB2 Express > Download DB2 Express C - the FREE version of DB2 express and take > control of your XML. No limits. Just data. Click to get it now. > http://sourceforge.net/powerbar/db2/ > _______________________________________________ > Ganglia-developers mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/ganglia-developers ------------------------------------------------------------------------- This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now. http://sourceforge.net/powerbar/db2/ _______________________________________________ Ganglia-developers mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ganglia-developers
