Skip to content

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 <omosnacek@gmail.com>
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