-
Notifications
You must be signed in to change notification settings - Fork 885
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
pkgconfig files not installed with --enable-script-wrapper-compilers
#12609
Labels
Comments
This was referenced Jun 17, 2024
@jsquyres yes, that fixes it. Thank you! Sorry for the weird link, that fork keeps coming up first in my searches. Fixed it. |
Changes merged. Closing. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Background information
I'm exploring the consequences of enabling
--enable-script-wrapper-compilers
, and noticed that enabling the script wrappers prevents installation of pkg-config files.What version of Open MPI are you using? (e.g., v4.1.6, v5.0.1, git branch name and hash, etc.)
v5.0.3
Describe how Open MPI was installed (e.g., from a source/distribution tarball, from a git clone, from an operating system distribution package, etc.)
From source, with this script (for conda-forge, but currently running by hand for testing)
I think the only relevant argument there is
--enable-script-wrapper-compilers
, the rest should probably be ignored.Please describe the system on which you are running
Details of the problem
The openmpi pkg-config files are conditionally installed, and the condition is:
when presumably it should be:
Which means they don't get installed if
--enable-script-wrapper-compilers
is given. The files are generated correctly, they just get left out of installation, so I imagine this is not deliberate. Addingcp -v ompi/tools/wrappers/*.pc $PREFIX/lib/pkgconfig/
after installation produces the desired result.Perhaps relevant to #10572 where
--enable-script-wrapper-compilers
is discussed in reference to pkg-config, but it is not mentioned that enabling it removes all pkg-config files.The text was updated successfully, but these errors were encountered: