Skip to content

Fix libgc pkg-config name for version discovery #15636

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

Merged

Conversation

straight-shoota
Copy link
Member

We define different names for pkg-config in the Link annotations of LibGC depending on target, but used a fixed name in version discovery.
This patch re-uses the name declared in the annotation. We need to filter for the appropriate annotation because there are multiple ones. Unfortunately, I haven't found an easier way to do this.

@straight-shoota straight-shoota added kind:bug A bug in the code. Does not apply to documentation, specs, etc. topic:stdlib:runtime labels Apr 4, 2025
@straight-shoota straight-shoota self-assigned this Apr 4, 2025
Co-authored-by: Quinton Miller <[email protected]>
@ysbaddaden ysbaddaden added this to the 1.16.1 milestone Apr 15, 2025
@straight-shoota straight-shoota merged commit 8a6ae41 into crystal-lang:master Apr 15, 2025
34 checks passed
@straight-shoota straight-shoota deleted the fix/libgc-pkgconfig-name branch April 15, 2025 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:bug A bug in the code. Does not apply to documentation, specs, etc. topic:stdlib:runtime
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants