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 pkg-config support and deprecate cups-config #97

Closed
michaelrsweet opened this issue Feb 17, 2021 · 2 comments · Fixed by #122
Closed

Add pkg-config support and deprecate cups-config #97

michaelrsweet opened this issue Feb 17, 2021 · 2 comments · Fixed by #122
Assignees
Labels
enhancement New feature or request priority-high
Milestone

Comments

@michaelrsweet
Copy link
Member

Most open source libraries now provide support for the pkg-config utility. This also handles the addition of external dependencies (GNU TLS, zlib, etc.) and reporting project-specific values like directories.

We should add a pkg-config file for CUPS ("cups.pc") that provides all of the current cups-config options, and then deprecate cups-config.

@michaelrsweet michaelrsweet added enhancement New feature or request priority-high labels Feb 17, 2021
@michaelrsweet michaelrsweet added this to the v2.4.0 milestone Feb 17, 2021
@michaelrsweet michaelrsweet self-assigned this Feb 17, 2021
@surajkulriya
Copy link
Contributor

surajkulriya commented Feb 21, 2021

hello @michaelrsweet
I have a query regarding this
with the help of .pc files we can support specific options like --libs, --cflags etc.
But there are few other options in cups-config like --serverbin, --serverroot that I guess we won't be able to support using .pc files. Although one thing that I noticed is that we rarely use such options.(--serverbin,--serverroot)
So will it be okay if in our new .pc file we add support of only options like --libs, --cflags that we use frequently.
Thanks

@michaelrsweet
Copy link
Member Author

@surajkulriya The pkg-config command supports package-specific variables, e.g.:

pkg-config —variable=serverbin

to return the “serverbin” variable in the .pc file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority-high
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants