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

add pkgconfig file to cmake build system #120

Merged
merged 1 commit into from
Apr 6, 2019
Merged

Conversation

loriab
Copy link
Collaborator

@loriab loriab commented Apr 6, 2019

A packager wanted this (psi4/psi4#1589) so that a CMake-built libint can be detected by both cmake and libtool downstream buildsystems. The CMake LibintConfig.cmake and friends are too involved to be imitated by libtool.

I constructed the .pc file by analogy to Libxc https://gitlab.com/libxc/libxc/blob/master/cmake/libxc.pc.in and https://gitlab.com/libxc/libxc/blob/master/build/libxc.pc.in . It builds and looks like the below, but I haven't tried out its consumption in a downstream libtool build.

>>> cat install-am4/lib64/pkgconfig/libint.pc 
prefix=/home/psilocaluser/gits/libint/install-am4
exec_prefix=/home/psilocaluser/gits/libint/install-am4
libdir=/home/psilocaluser/gits/libint/install-am4/lib64
includedir=/home/psilocaluser/gits/libint/install-am4/include

Name: libint
Description: a library for the evaluation of molecular integrals of many-body operators over Gaussian functions
Version: 1.2.1
Libs: -L${libdir} -lderiv -lint
Cflags: -I${includedir} -I${includedir}/libint -I${includedir}/libderiv

@evaleev
Copy link
Owner

evaleev commented Apr 6, 2019

looks good. do I need to bump the version?

@evaleev evaleev merged commit b349524 into evaleev:v1 Apr 6, 2019
@loriab
Copy link
Collaborator Author

loriab commented Apr 6, 2019

Thanks for the merge. There's Ninja, Windows, and pkgconfig changes since v1.2.1, so a version bump is justifiable but not necessary on Psi's account -- your call.

@loriab loriab deleted the pkgconfig branch April 6, 2019 13:31
@evaleev
Copy link
Owner

evaleev commented Apr 6, 2019 via email

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