Skip to content

vala: Pass --shared-library to valac when generating a .gir file #14564

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

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

Conversation

tintou
Copy link
Contributor

@tintou tintou commented May 6, 2025

This is required to make sure that the generated .gir file actually contains all the information to be used dynamically.

Valac supports this argument since 0.29.3 released in 2015.

Fixes: #14562

@tintou tintou requested a review from jpakkane as a code owner May 6, 2025 10:13
@bugaevc
Copy link

bugaevc commented May 6, 2025

I can confirm this works for me!

@bugaevc
Copy link

bugaevc commented May 6, 2025

It does say shared-library="libdemo.a" if I use it with static_library(). It should probably not pass anything in that case.

@tintou tintou force-pushed the tintou/vala-shared-lib branch from d584d2c to 476d815 Compare May 6, 2025 12:03
@tintou
Copy link
Contributor Author

tintou commented May 6, 2025

Here is a new version that only pass the argument for shared libraries

This is required to make sure that the generated .gir file actually contains all
the information to be used dynamically.

Valac supports this argument since 0.29.3 released in 2015.
@tintou tintou force-pushed the tintou/vala-shared-lib branch from 476d815 to 3ddc884 Compare May 6, 2025 12:19
@bonzini bonzini added this to the 1.9 milestone May 7, 2025
@bonzini bonzini modified the milestones: 1.9, 1.8.2 May 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pass --shared-library to Vala
3 participants