-
-
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
filebot: init at 4.8.5 #67849
filebot: init at 4.8.5 #67849
Conversation
@GrahamcOfBorg build filebot |
@symphorien Addressed your comments, PTAL |
|
||
nativeBuildInputs = [ makeWrapper autoPatchelfHook | ||
# The following are lib dependencies for autoPatchelfHook to work. | ||
stdenv.cc.cc.lib zlib libzen libmediainfo curlWithGnuTls libmms glib ]; |
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.
These should be buildInputs (runtime dependencies), not nativeBuildInputs (build dependencies).
# The following are lib dependencies for autoPatchelfHook to work. | ||
stdenv.cc.cc.lib zlib libzen libmediainfo curlWithGnuTls libmms glib ]; | ||
|
||
buildPhase = ''''; |
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.
buildPhase = ''''; | |
dontBuild = true; |
ln -s $out/opt/filebot.sh $out/bin/filebot | ||
''; | ||
|
||
patchPhase = ''''; |
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.
Is this line really needed ?
ping @gleber, any update? |
OBE. See above PR ^^^ |
Motivation for this change
This is a non-free application to rename downloaded media files as per internet databases of medias.
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 @