-
-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
gnomeExtensions.ddterm: fix gjs path and dependencies #298973
Conversation
b9c2fb1
to
0850395
Compare
0850395
to
d79d5a8
Compare
nativeBuildInputs = [ gjs gobject-introspection wrapGAppsHook ]; | ||
buildInputs = [ vte libhandy ]; |
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.
nativeBuildInputs = [ gjs gobject-introspection wrapGAppsHook ]; | |
buildInputs = [ vte libhandy ]; | |
nativeBuildInputs = [ gobject-introspection wrapGAppsHook ]; | |
buildInputs = [ gjs vte libhandy ]; |
Imagine you're cross-compiling from x86_64-linux to aarch64-linux, you'll certainly want gjs
to be native to aarch64-linux so that it runs on the host. So gjs
should be in buildInputs
.
As long as this PR is not merged #305991 is still open and an overlay would be necessary. How would that overlay look like on NixOS (without home-manager)? See my question here: #305991 (comment) |
The new version has #!@GJS_SHEBANG@ which ended up as #!/usr/bin/gjs -> #!/usr/bin//nix/store../bin/gjs. gnomeExtensions.ddterm: add libhandy as dependency libhandy was added as dependency on commit: ddterm/gnome-shell-extension-ddterm@eade14f
When will this fix be available for NixOS 24.05 ?? |
Successfully created backport PR for |
Wow this was quick, thanks :) |
The new version has
#!@GJS_SHEBANG@
which ended up as#!/usr/bin/gjs
->#!/usr/bin//nix/store../bin/gjs
. This change changes the whole line before building.Also added
libhandy
as dependency which is added in commit ddterm/gnome-shell-extension-ddterm@eade14f.Description of changes
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 馃憤 reaction to pull requests you find important.