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

Allow linking against system libraries via pkg-config #63

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

WOnder93
Copy link
Contributor

@WOnder93 WOnder93 commented Jul 26, 2021

When 'link_pkgconfig' is added to CONFIG, chessx will now use pkg-config
to dynamically link against the quazip and zlib libraries installed on
the system, rather than bundling quazip or guessing the header file
location. Default behavior remains unchanged.

@WOnder93
Copy link
Contributor Author

I updated the PR to use the new QuaZip development interface (where we can take advantage of pkgconfig) used in version 1.0 and later. Previous version would work against the older 0.x releases.

@WOnder93 WOnder93 changed the title Allow linking against system quazip library Allow linking against system libraries via pkg-config Sep 14, 2021
@WOnder93
Copy link
Contributor Author

Updated to also use pkgconfig for zlib, which is used directly by kzip.cpp, which I missed before. Also I changed it to just react to the built-in link_pkgconfig config rather than introducing a new one.

@WOnder93
Copy link
Contributor Author

@Isarhamster Any feedback to this and the other PR? I'm currently carrying these as downstream patches in Fedora [1]. It would be nice to have them included in an official release.

[1] https://src.fedoraproject.org/rpms/chessx/tree/rawhide

When 'link_pkgconfig' is added to CONFIG, chessx will now use pkg-config
to dynamically link against the quazip and zlib libraries installed on
the system, rather than bundling quazip or guessing the header file
location. Default behavior remains unchanged.

Signed-off-by: Ondrej Mosnáček <[email protected]>
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.

1 participant