I merged your two patches and will commit once I come within range of
being online. I'll also file a CMR for 1.2.6 in case that ever
happens (had to resolve a trivial patch reject for the 1.2 branch).
On Feb 22, 2008, at 9:40 AM, Brian W. Barrett wrote:
> A second attempt, this time with less suck :).
> On Fri, 22 Feb 2008, Brian W. Barrett wrote:
>> Oops.. forgot to test the C++ bindings, which of course broke.
>> Let me
>> try again and I'll send a better patch.
>> On Fri, 22 Feb 2008, Brian W. Barrett wrote:
>>> On Thu, 21 Feb 2008, Jeff Squyres wrote:
>>>> While I agree that having AC actually define them to a value is a
>>>> Thing (better than just defining it to be empty), I do see the
>>>> that it has put us in. :-\ I don't see an obvious solution.
>>> I do :). Try the attached patch. It sneaks in at the last minute
>>> defines (for example) both SIZEOF_BOOL and OMPI_SIZEOF_BOOL.
>>> mpi.h only has
>>> the define for OMPI_SIZEOF_BOOL. opal_config.h has both, but a
>>> note saying
>>> not to use the OMPI_ version. Seems to work.
>> users mailing list
> users mailing list