Skip to content

Conversation

@rhc54
Copy link
Contributor

@rhc54 rhc54 commented Jun 23, 2015

Now that we require C99, and stdbool.h is part of C99, we no longer need to define our own bool types. Since bool is commonly used in a lot of places, just include stdbool.h in opal_config_bottom.h

@jsquyres please review

…eed to define our own bool types. Since bool is commonly used in a lot of places, just include stdbool.h in opal_config_bottom.h
@rhc54 rhc54 added this to the Future milestone Jun 23, 2015
@mellanox-github
Copy link

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

@jsquyres
Copy link
Member

@rhc54 Is there some stuff in configure.ac (or whatever.m4) that we can remove, too?

@mellanox-github
Copy link

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

@mellanox-github
Copy link

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

@jsquyres
Copy link
Member

👍

rhc54 pushed a commit that referenced this pull request Jun 25, 2015
Remove internal bool type definitions
@rhc54 rhc54 merged commit 1a767ed into open-mpi:master Jun 25, 2015
@rhc54 rhc54 deleted the topic/config branch June 25, 2015 16:10
jsquyres pushed a commit to jsquyres/ompi that referenced this pull request Nov 10, 2015
ft_removal: remove sstore/snapc frameworks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants