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
libxkbcommon: fix build on darwin #65010
Conversation
08eac1c
to
95fadbc
Compare
Could you please make this pull request against |
@GrahamcOfBorg build libxkbcommon |
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.
Seems good, borg confirms the fix. Only rebuilds what's broken already. Upstreaming shouldn't block our merge.
@vcunat You're correct in that upstreaming shouldn't block the merge. Typically I'd think merge would be blocked until a patch is submitted upstream. |
Guess I'll go ahead and submit one. |
I think it was introduced in xkbcommon/libxkbcommon#68 ... but perhaps it's best to submit a PR directly. |
I can't see why such a hard rule should exist. Here I see very little risk in going ahead of upstream, and submitting would give no "validation" by itself (though I'd expect feedback soon after that). |
It's not exactly a hardrule, it's essentially a good idea to get feedback from them. |
Motivation for this change
meson tries to build a example file which uses glib's
program_invocation_short_name
, which is not available on darwinThings 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)