Skip to content

Refactored the --with-prrte configure option to reject building Open MPI against external installs of PRRTE #1

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.

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

Merged
merged 2 commits into from
Nov 20, 2024

Conversation

alexandria-sisk
Copy link

Refactored the --with-prrte configure option to only accept "yes" , "no", or nothing as arguments. The --with-prrte option previously allowed for building Open MPI against external installations of PRRTE which we no longer want to support. Any paths to external PRRTE builds given to this configure option will be rejected.

@alexandria-sisk alexandria-sisk force-pushed the alexsisk/refactor-prrte-config-option branch 4 times, most recently from 2fb05ce to 7adcecf Compare November 14, 2024 12:05
@jsquyres
Copy link

jsquyres commented Nov 17, 2024

@alexandria-sisk I think your code isn't working -- check out line 4043 in the macOS build CI results in the "Build Open MPI" section:

*** Configuring PRRTE
./configure: line 69195: test: -eq: unary operator expected

This suggests that the PRRTE configure script failed to run, and therefore OMPI thought it wasn't building PRRTE.

…l PRRTE builds

Signed-off-by: Alexandria Sisk <a0sisk02@louisville.edu>
@alexandria-sisk alexandria-sisk force-pushed the alexsisk/refactor-prrte-config-option branch from 44f6564 to 9222192 Compare November 19, 2024 23:44
@jsquyres jsquyres force-pushed the alexsisk/refactor-prrte-config-option branch from 9222192 to fae7249 Compare November 20, 2024 17:32
@alexandria-sisk alexandria-sisk force-pushed the alexsisk/refactor-prrte-config-option branch from fae7249 to 375245f Compare November 20, 2024 22:46
Signed-off-by: Alexandria Sisk <a0sisk02@louisville.edu>
@alexandria-sisk alexandria-sisk force-pushed the alexsisk/refactor-prrte-config-option branch from 375245f to a8e87f7 Compare November 20, 2024 23:04
@jsquyres jsquyres merged commit e9f654b into capstone-devel Nov 20, 2024
13 checks passed
@jsquyres jsquyres deleted the alexsisk/refactor-prrte-config-option branch November 20, 2024 23:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants