On Wed, Jul 02, 2008 at 11:14:17AM -0600, Brad Nicholes wrote: > >>> On 7/2/2008 at 11:23 AM, in message <[EMAIL PROTECTED]>, Carlo > Marcelo Arenas Belon <[EMAIL PROTECTED]> wrote: > > On Wed, Jul 02, 2008 at 10:30:13AM -0600, Brad Nicholes wrote: > >> >>> On 7/2/2008 at 4:34 AM, in message <[EMAIL PROTECTED]>, Carlo > > Marcelo > >> Arenas Belon <[EMAIL PROTECTED]> wrote: > >> > The following proposed patch for stable 3.1, workaround a bug in the APR > >> > headers in 32bit Linux that rely in off64_t being defined, which is only > >> > the case when _LARGEFILE64_SOURCE is defined, failing otherwise with : > >> > > >> > /usr/include/apr-1.0/apr.h:273: error: expected '=', ',', ';', 'asm' > >> > or > >> > '__attribute__' before 'apr_off_t' > >> > > >> > Contains changes from r1488 > >> > > >> > Carlo > >> > --- > >> > >> This is already being defined in the makefile.am's. Are we missing > > something somewhere? Should the -D_LARGEFILE64_SOURCE be removed from the > > makefile.am's in favor of a change to gm_metric.h? Or should it be defined > > in > > both places? > > > > for now in both places (hence why it is #ifndef), because this was aimed at > > out of tree C module builders which will have no way to know that > > -D_LARGEFILE64_SOURCE is needed for them to get a working build. > > > > The next step will be to remove it from the Makefile.am for mod_example and > > then from all in-tree modules. Removing it from all Makefile.am will be a > > nice to have as well, but not yet for stable 3.1, and specially not for the > > first release of it. > > > > OK, then I am +1
Committed revision 1499 Carlo ------------------------------------------------------------------------- Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW! Studies have shown that voting for your favorite open source project, along with a healthy diet, reduces your potential for chronic lameness and boredom. Vote Now at http://www.sourceforge.net/community/cca08 _______________________________________________ Ganglia-developers mailing list Ganglia-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ganglia-developers