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
gnome: Add post_install() method #8272
Conversation
CC @ebassi See corresponding MR in gtk4: https://gitlab.gnome.org/GNOME/gtk/-/merge_requests/3135 |
Also need a patch in glib in case it's a subproject: https://gitlab.gnome.org/GNOME/glib/-/merge_requests/1909. |
It is going to be used in Meson's gnome module, and if glib is built as subproject it's better to use it. See mesonbuild/meson#8272.
fd3b1ed
to
9e31fa9
Compare
I don't know enough about Gnome internals to say anything about the UX but featurewise looks good. |
9e31fa9
to
3760fee
Compare
Various GNOME projects have scripts that does similar task, better do it directly in meson. This ensures it's done correctly regarding usage of subprojects and pkg-config. See for example this gtk bug: https://gitlab.gnome.org/GNOME/gtk/-/issues/3626. Fixes: mesonbuild#8268
3760fee
to
bcdec4d
Compare
I decided to rename the method to |
@ebassi could validate the idea maybe? |
I am not @mclasen. Please remove
Michael Clasen
Mclasen@mclasen.com
Mclasen@gmail.com
Thanks
Sent from my iPhone 6+
… On Feb 5, 2021, at 12:09 PM, Xavier Claessens ***@***.***> wrote:
Merged #8272 into master.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
That was for @matthiasclasen, sorry. |
Various GNOME projects have scripts that does similar task, better do it directly in meson. This ensures it's done correctly regarding usage of subprojects and pkg-config. See for example this gtk bug:
https://gitlab.gnome.org/GNOME/gtk/-/issues/3626.
Fixes: #8268