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
feh: 2.18.1 -> 2.18.2 #24028
feh: 2.18.1 -> 2.18.2 #24028
Conversation
stdenv.mkDerivation rec { | ||
name = "feh-2.18.1"; | ||
name = "feh-${version}"; | ||
version = "2.18.2"; |
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 version field doesn't seem to be used anywhere else so simpler to just leave it directly in the name.
--add-flags '--theme=feh' | ||
''; | ||
--add-flags '--theme=feh' | ||
''; |
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.
Some stray whitespace here.
|
||
buildInputs = [ xorg.libX11 xorg.libXinerama imlib2 libjpeg libpng curl libexif ]; | ||
|
||
preBuild = '' | ||
makeFlags="PREFIX=$out exif=1" | ||
''; | ||
|
||
libPath = makeLibraryPath ([ imlib2 curl xorg.libXinerama libjpeg libpng libexif xorg.libX11 ]); |
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 don't think the parentheses do anything here. Should work fine without them.
|
||
buildInputs = [ xorg.libX11 xorg.libXinerama imlib2 libjpeg libpng curl libexif ]; | ||
|
||
preBuild = '' | ||
makeFlags="PREFIX=$out exif=1" | ||
''; | ||
|
||
libPath = makeLibraryPath ([ imlib2 curl xorg.libXinerama libjpeg libpng libexif xorg.libX11 ]); |
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'm a bit curious why the libPath
field is required here. Did something change in 2.18.2 to need it or was there a problem with earlier versions as well?
Umm, I would have appreciated some response to my comments. |
It was attempt to fix it not finding that lib |
I will update it later think I know how to fix it now |
Sry didn't see msg |
No worries. |
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.
Actually I don't think I added that. I don't remember doing it anyway
Motivation for this change
Update
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)