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
vimb: 3.1.0 -> 3.3.0 #51232
vimb: 3.1.0 -> 3.3.0 #51232
Conversation
|
||
src = fetchurl { | ||
url = "https://github.com/fanglingsu/vimb/archive/${version}.tar.gz"; | ||
sha256 = "1gws028c2v1zh6r142hmjvi2m447lwqqh65m6z3dzcar2yw35z3f"; | ||
sha256 = "0v3daxs10nndxvcpvx8377aylfdismzkys5n5cs8m89c3fdy6vsw"; |
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 be also fetchFromGitHub
.
nativeBuildInputs = [ pkgconfig ]; | ||
buildInputs = [ makeWrapper gtk2 libsoup webkit gsettings-desktop-schemas ]; | ||
nativeBuildInputs = [ makeWrapper pkgconfig ]; | ||
buildInputs = [ gtk3 libsoup webkit gsettings-desktop-schemas ]; |
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.
I just noticed the makeWrapper
:
If you add nativeBuildInputs = [ wrapGAppsHook ];
and buildInputs = [ glib gnome3.gsettings_desktop_schemas ];
then you don't makeWrapper
and the result
is less fragile.
https://github.com/NixOS/nixpkgs/blob/master/pkgs/applications/networking/instant-messengers/rambox/default.nix#L33
wrapGAppsHook per reviewer suggestion, ty! :)
e54564c
to
d57d414
Compare
@Mic92 look good? |
https://github.com/fanglingsu/vimb/releases/tag/3.2.0
https://github.com/fanglingsu/vimb/releases/tag/3.3.0
Motivation for this change
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)