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.

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

CMake unconditionally installs libint2.pc into lib/pkgconfig #230

Closed
mbanck opened this issue Jan 1, 2022 · 6 comments · Fixed by #271
Closed

CMake unconditionally installs libint2.pc into lib/pkgconfig #230

mbanck opened this issue Jan 1, 2022 · 6 comments · Fixed by #271

Comments

@mbanck
Copy link
Contributor

mbanck commented Jan 1, 2022

The cmake build systems decides to override the standard CMAKE_INSTALL_LIBDIR with LIBINT2_INSTALL_LIBDIR (why?) for the libraries, but it uses a hardcoded lib/pkgconfig directory for libint2.pc (but see #225 - the generated libint2.pc is broken anyway):

# install cmake-processed files
install(FILES ${PROJECT_BINARY_DIR}/libint2.pc
        DESTINATION lib/pkgconfig)

This leads to libint2.pc not ending up in the same place as the library (they go into /usr/lib/<gnu-triplet>/ on most distributions in order to support multiarch).

@loriab
Copy link
Collaborator

loriab commented Jan 25, 2022

I think this is addressed in #233 .

@mbanck
Copy link
Contributor Author

mbanck commented Jan 25, 2022

Sounds good on first glance; I'm sorry, but I don't have the bandwidth right now to check out that #233 PR, I'll try to give it a spin with respect to distribution packaging ASAP (unless @susilehtola did already?)

@susilehtola
Copy link
Contributor

I haven't updated libint2 in Fedora since Aug 2019 when 2.6.0 was released...

@susilehtola
Copy link
Contributor

I've been waiting for the issues with Psi4 (including CMake in libint2) to be addressed

@loriab
Copy link
Collaborator

loriab commented Jan 26, 2022

Sounds good on first glance; I'm sorry, but I don't have the bandwidth right now to check out that #233 PR

Sure, no problem. #233 isn't ready for trial yet, and it needs EFV approval, too. I just wanted to record that this issue should be fixed up.

@susilehtola, yes, I'm working on the gmp/mpfr/eigen detection.

@loriab
Copy link
Collaborator

loriab commented Sep 6, 2023

This should be minimally addressed in #271 . The library is still going to LIBINT2_INSTALL_LIBDIR because I'm not making unnecessary changes, but the pc file goes to CMAKE_INSTALL_LIBDIR/pkgconfig. Thus locations are adjustable.

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 a pull request may close this issue.

3 participants