-
Notifications
You must be signed in to change notification settings - Fork 38
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
Introspection #54
Introspection #54
Conversation
GIR will complain about it.
Without it, GIR thinks these are strings, instead of string arrays.
It's necessary for the visibility of the get_type()s generated by gnome.mkenum().
No functional changes, but will make next commit's diff a bit easier to read.
G_DECLARE_*_TYPE() already gives us the get_type() function of the object, declaring it again confuses the introspection scanner.
It's a necessary offering to the greater GIR gods.
No functional change, just will make things easier to deal with later.
99ba63c
to
fa4168a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Outside of that one question this all looks excellent, feel free to merge when ready.
Add instrospection to libportal, libportal-gtk3, and libportal-gtk4. Use Xdp as the namespace. Fixes flatpak#32
fa4168a
to
f456e41
Compare
We will want a CI setup for the gtk4 backend at some point too. |
This is in my queue already - together with generating Flatpak bundles out of the manifests here. But I'll do it in a separate pull request. |
To exercise the newly introduced introspection, rewrite the GTK4 test in JavaScript / GJS, and rearrange the build files accordingly. Sadly, I had to drop the POSIX file write test, since that's not supported on JavaScript. This can still be tested using the GTK3 test.
Doesn't hurt to be slightly more up-to-date.
f456e41
to
ab19e66
Compare
Bootstrapping the CI changes at #55. |
See commits. Needed suffling to expose XdpParent to GIR, but overall seems like a decently small changeset.
Fixes #32