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 file for C++ API bindings #194

Merged
merged 3 commits into from
Feb 22, 2023

Conversation

Kerilk
Copy link
Contributor

@Kerilk Kerilk commented Feb 15, 2023

This implements pkg-config support for C++ bindings, which would be good to have, as hinted by KhronosGroup/OpenCL-Headers#217

I will add a commit updating the path of the regular headers to share, but this will break the CI until KhronosGroup/OpenCL-Headers#218 is merged.

@Kerilk Kerilk requested a review from bashbaug February 22, 2023 20:30
Copy link
Contributor

@bashbaug bashbaug left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Usual caveat: I am not a pkg-config expert. This matches the similar changes in OpenCL-Headers though and CI tests are passing, so LGTM!

@bashbaug bashbaug merged commit ac7e7bd into KhronosGroup:main Feb 22, 2023
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