Skip to content

add pkg-config file #18

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

Closed
asdil12 opened this issue Jul 23, 2014 · 4 comments · Fixed by #42
Closed

add pkg-config file #18

asdil12 opened this issue Jul 23, 2014 · 4 comments · Fixed by #42

Comments

@asdil12
Copy link

asdil12 commented Jul 23, 2014

Eg. Autobuild relies on that. Please add an option to create this file on building.

Example:

/usr/lib/pkgconfig/jsoncpp.pc

prefix=/usr
libdir=${prefix}/lib
includedir=${prefix}/include/jsoncpp

version = 0.5.0

name = jsoncpp
pkgname = jsoncpp

Name: ${name}
Version: ${version}
Description: Jsoncpp library version 0.5.0
Cflags: -I${includedir}
Libs: -L${libdir} -ljsoncpp
@cdunn2001
Copy link
Contributor

I've used pkg-config myself, but I'm not sure where to put it or how to tell cmake to generate it. Could you submit a pull request?

@asdil12
Copy link
Author

asdil12 commented Jul 28, 2014

I actually don't know cmake enough as well.

@jplatte
Copy link
Contributor

jplatte commented Aug 13, 2014

I could do this as I've already generated .pc files with CMake. However I can't access the corresponding files right now; I will submit a PR as soon as I can (will take some weeks).

@bchretien
Copy link

👍

The idea would be to have a template file (similar to this one) that would be filled with CMake variables (with CONFIGURE_FILE). Note that in our custom CMake toolbox, we handle pretty much everything when generating pkg-config files for all of our projects, so the CMake scripts can become quite complex. Still, you can make something much more simple here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants