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
onboard: Fix gobject-introspection #67884
Conversation
@@ -91,6 +91,7 @@ in python3.pkgs.buildPythonApplication rec { | |||
wrapGAppsHook | |||
xorg.libXtst | |||
xorg.libxkbfile | |||
gobject-introspection # Temporary fix, see https://github.com/NixOS/nixpkgs/issues/56943 |
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.
gobject-introspection # Temporary fix, see https://github.com/NixOS/nixpkgs/issues/56943 |
And use
strictDeps = false;
with the comment.
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.
@worldofpeace is it better to remove strictDeps
vs listing twice? If so, why?
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.
The issue is actually triggered preventing the setup hook from being ran because of strictDeps being used #56943 (comment).
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.
So this will be the best work around until the eventual issue is fixed.
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.
Correct, as python packages enabled strictDeps
this was the regression that followed for gobject-introspection using applications.
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.
Tested on non-nixos system with strictDeps = false;
, looks good.
I've fixed this in #67915 along with some other issues with onboard. |
Motivation for this change
#56943
Onboard was not working under
xmonad
(AFAIK it works fine undergnome
).Fixed as per ticket #56943 by adding
gobject-introspection
tobuildInputs
(it's already innativeBuildInputs
.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)Notify maintainers
cc @johnramsden