Skip to content
This repository was archived by the owner on Sep 30, 2022. It is now read-only.

Conversation

@hjelmn
Copy link
Member

@hjelmn hjelmn commented Jun 15, 2016

Newer versions of gcc have "poisoned" the __malloc_initialize_hook
name and it can no longer be used. Added a configure check and
protection around its usage.

Signed-off-by: Nathan Hjelm hjelmn@lanl.gov

(cherry picked from open-mpi/ompi@7018aed)

Signed-off-by: Nathan Hjelm hjelmn@lanl.gov

Newer versions of gcc have "poisoned" the __malloc_initialize_hook
name and it can no longer be used. Added a configure check and
protection around its usage.

Signed-off-by: Nathan Hjelm <hjelmn@lanl.gov>

(cherry picked from open-mpi/ompi@7018aed)

Signed-off-by: Nathan Hjelm <hjelmn@lanl.gov>
@hjelmn
Copy link
Member Author

hjelmn commented Jun 15, 2016

:bot🏷️bug
:bot:milestone:v2.0.0
:bot:assign: @hppritcha

@ompiteam-bot ompiteam-bot added this to the v2.0.0 milestone Jun 15, 2016
@hppritcha hppritcha modified the milestones: v2.0.1, v2.0.0 Jun 16, 2016
@hppritcha
Copy link
Member

this only occurs with gcc 6.1.

@hjelmn
Copy link
Member Author

hjelmn commented Jun 16, 2016

yup. they added the #pragma poison in 6.1

@mellanox-github
Copy link

Test PASSed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/1776/ for details.

@jsquyres
Copy link
Member

bot:retest

@mellanox-github
Copy link

Test FAILed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/1875/ for details.

@jsquyres
Copy link
Member

bot:retest

@mellanox-github
Copy link

Test FAILed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/1895/ for details.

@hjelmn
Copy link
Member Author

hjelmn commented Jul 18, 2016

:bot:retest

@jsquyres
Copy link
Member

bot:mellanox:retest

@hppritcha
Copy link
Member

travis has the known sed problem.

@mellanox-github
Copy link

Test PASSed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/1969/ for details.

@jsquyres jsquyres merged commit 29b837a into open-mpi:v2.x Jul 22, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants