-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
udiskie: fix build #56783
udiskie: fix build #56783
Conversation
It was broken by b4acd97. Fixes NixOS#56771.
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.
LGTM!
librsvg # required for loading svg icons (udiskie uses svg icons) | ||
gobject-introspection |
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 |
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.
That yields
$ udiskie -t &
Typelib for 'libnotify' is not available. Possible causes include:
- libnotify is not installed
- the typelib is provided by a separate package
- libnotify was built with introspection disabled
Starting udiskie without notifications.
Typelib for 'Gtk 3.0' is not available. Possible causes include:
- GTK3 is not installed
- the typelib is provided by a separate package
- GTK3 was built with introspection disabled
Starting udiskie without tray icon.
That's why I put it in both.
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.
Weird, I would expect setup hooks to work front nativeBuildInputs
even with strictDeps
.
We have |
Binaries seem to work and it builds so fine by me. Thanks ! |
@dotlambda It does not seem broken on 19.03: https://hydra.nixos.org/job/nixos/release-19.03/nixpkgs.udiskie.x86_64-linux/all - should we backport anyway? |
It's broken on |
@dotlambda I'm not 100% sure about this, but can you explain why you moved libappindicator-gtk3 out of the propagated build inputs. My recollection is that the appindicator functionality does not work without it being there |
|
As stated above,
I simply moved all non-Python dependencies out of |
Motivation for this change
It was broken by b4acd97.
Fixes #56771.
I'm not sure I know enough about GTK to determine which dependencies belong where. Feedback is welcome.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)cc @teto