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
gtk3: 3.24.8 -> 3.24.10, meson! #63874
Conversation
@GrahamcOfBorg build gtk3 |
fixupOutputHooks+=(_gtk3CleanComments) | ||
|
||
# Clean comments that link to generator of the file | ||
_gtk3CleanComments() { |
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.
Needed to add this because of how this file is created with post_install.py
it causes a circular reference.
Seems fine at the first glance. |
|
||
for program in ''${demos[@]}; do | ||
wrapProgram $dev/bin/$program \ | ||
--prefix XDG_DATA_DIRS : "$GSETTINGS_SCHEMAS_PATH:$dev/share/gsettings-schemas/${pname}-${version}" |
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.
Should these be wrapped for darwin?
Wait, why is this targeted to |
At the time of opening |
We now build gtk3 with the meson build system. Notably, at least for the expression, all the platforms logic for we had for configureFlags should be unneeded with meson.
This makes us generate the .css instead of using the pregenerated ones.
Currently the changes still look fine (and I still don't have time to do an in-depth analysis/testing, sorry) |
There are quite some gtk failures on staging-next. Could you have a look? |
@FRidh The path handling for their pkgconfig file's wasn't great and
That patch should be fairly simple I think, incoming soon. |
Motivation for this change
GTK can now be built with the meson build system.
Did this pretty blindly, not ready for review yet.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)