Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

v2.1.x: Open BSD 6.0 and patcher problems #2106

Open
jsquyres opened this issue Sep 22, 2016 · 5 comments
Open

v2.1.x: Open BSD 6.0 and patcher problems #2106

jsquyres opened this issue Sep 22, 2016 · 5 comments
Assignees
@hjelmn
Labels
bug
Milestone
Future

Comments

@jsquyres
Copy link
Member

Filing a duplicate issue to #2049 for v2.1.0 (it's a blocker on v2.0.2, so it needs to be a blocker on v2.1.0, too).

As reported by Paul H., Open BSD 6.0 disallows the patcher code's mprotect call.

See https://mail-archive.com/[email protected]/msg19527.html
@jsquyres jsquyres added this to the v2.1.0 milestone Sep 22, 2016
@jsquyres jsquyres mentioned this issue Sep 22, 2016
Closed
@hppritcha
Copy link
Member

@hjelmn could you get this working on v2.x?

@hppritcha
Copy link
Member

hppritcha commented Jan 24, 2017 •
edited by jsquyres
Loading

Waiting on #2107 to get merged in to master.

@jsquyres
Copy link
Member Author

Per discussion on the 2017-02-21 webex, @hjelmn said he would try to get this in today.

ggouaillardet added a commit to ggouaillardet/pmix that referenced this issue Feb 24, 2017
@ggouaillardet
commit d6530b0
f0ce179 
Author: Jeff Squyres <[email protected]>
Date:   Mon Mar 9 08:15:38 2015 -0700

pmix_check_package: use AC_SEARCH_LIBS instead of AC_CHECK_LIB

Per discussion on devel
(http://www.open-mpi.org/community/lists/devel/2015/02/17030.php), and
per Autoconf 2.69 docs, use the recommended AC_SEARCH_LIBS instead of
AC_CHECK_LIB (e.g., for functions that appear in libc on some
platforms and in a specific library on other platforms).

see open-mpi/ompi@d6530b0

Refs open-mpi/ompi#2106

Signed-off-by: Gilles Gouaillardet <[email protected]>
ggouaillardet added a commit to ggouaillardet/pmix that referenced this issue Feb 24, 2017
@ggouaillardet
ptl/base: add missing include file
cdc8537 
Refs. open-mpi/ompi#2106

Signed-off-by: Gilles Gouaillardet <[email protected]>
@ggouaillardet ggouaillardet mentioned this issue Feb 24, 2017
Merged
@hppritcha
Copy link
Member

@jsquyres should we push this to 2.1.1?

@jsquyres jsquyres modified the milestones: v2.1.1, v2.1.0 Feb 28, 2017
@jsquyres
Copy link
Member Author

Agreed. Pushed to v2.1.1.

@hppritcha hppritcha modified the milestones: v2.1.2, v2.1.1 Apr 24, 2017
@hppritcha hppritcha modified the milestones: Future, v2.1.2 Aug 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hjelmn hjelmn

Labels
bug
Projects
None yet
Milestone
Future
Development

No branches or pull requests
3 participants
@jsquyres @hjelmn @hppritcha