Set SIZEOF_BOOL in configure.ac

[rhc54]

When building --no-ompi, we don't run setup_cxx and hence sizeof_bool never gets set. So set it in configure.ac so everyone always has it

@jsquyres please review

[mellanox-github]

Refer to this link for build results (access rights to CI server needed):
http://bgate.mellanox.com/job/gh-ompi-master-pr/678/

[jsquyres]

@rhc54 I don't think this is right -- we can't set the language to C++ since we haven't setup the C++ compiler.

I think we should probably just add _Bool to the SIZEOF's that start at configure.ac:378. Additionally, per our conversation last week, should configure.ac:422 be changed to:

OPAL_C_GET_ALIGNMENT(_Bool, OPAL_ALIGNMENT_BOOL)

(i.e., _Bool vs. bool)

This would allow us to get right of the <stdbool.h> in the alignment test.

[rhc54]

Tried that and got error - you need c++ to get sizeof bool

May have to bring back the opal bottom code in configure

Sent from my iPhone

On Jun 29, 2015, at 9:30 AM, Jeff Squyres [email protected] wrote:

@rhc54 I don't think this is right -- we can't set the language to C++ since we haven't setup the C++ compiler.

I think we should probably just add _Bool to the SIZEOF's that start at configure.ac:378. Additionally, per our conversation last week, should configure.ac:422 be changed to:

OPAL_C_GET_ALIGNMENT(_Bool, OPAL_ALIGNMENT_BOOL)
(i.e., _Bool vs. bool)

This would allow us to get right of the <stdbool.h> in the alignment test.

—
Reply to this email directly or view it on GitHub.