Re: [OMPI devel] [OMPI svn-full] svn:open-mpi r23998

2010-11-08 Thread Ralph Castain
Agreed - just remove the stdbool.h from libevent207.h. Your changes to event.h look fine as we can't push opal_config.h upstream anyway. On Nov 8, 2010, at 6:31 AM, Terry Dontje wrote: > Hmmm, it looks like you are right so my original change probably is the right > thing then. > > --td > >

Re: [OMPI devel] [OMPI svn-full] svn:open-mpi r23998

2010-11-08 Thread Terry Dontje
Hmmm, it looks like you are right so my original change probably is the right thing then. --td On 11/08/2010 08:13 AM, Jeff Squyres wrote: It doesn't look like is needed at all in libevent207.h. Should it just be removed? On Nov 8, 2010, at 6:18 AM, Terry Dontje wrote: In light of the p

Re: [OMPI devel] [OMPI svn-full] svn:open-mpi r23998

2010-11-08 Thread Jeff Squyres
It doesn't look like is needed at all in libevent207.h. Should it just be removed? On Nov 8, 2010, at 6:18 AM, Terry Dontje wrote: > In light of the push event changes upstream to libevent the changes to > libevent207.h probably should be modified to look like event.h. That is wrap > the i

Re: [OMPI devel] [OMPI svn-full] svn:open-mpi r23998

2010-11-08 Thread Terry Dontje
In light of the push event changes upstream to libevent the changes to libevent207.h probably should be modified to look like event.h. That is wrap the include with some ifdef for C++. I did not do this in the original fix because everything pulling it in was also pulling in opal_config.h an

Re: [OMPI devel] [OMPI svn-full] svn:open-mpi r23998

2010-11-05 Thread Jeff Squyres
This patch should be pushed upstream to libevent. Terry / Ralph? On Nov 5, 2010, at 2:54 PM, t...@osl.iu.edu wrote: > Author: tdd > Date: 2010-11-05 14:54:19 EDT (Fri, 05 Nov 2010) > New Revision: 23998 > URL: https://svn.open-mpi.org/trac/ompi/changeset/23998 > > Log: > corrected stdbool.h i