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
untrunc: init at 2018.01.13 #36167
untrunc: init at 2018.01.13 #36167
Conversation
g++ -o $out/bin/untrunc \ | ||
-Wno-deprecated-declarations \ | ||
$src/file.cpp $src/main.cpp $src/track.cpp $src/atom.cpp $src/mp4.cpp \ | ||
-I$libavConfiguredSrc -lavformat -lavcodec -lavutil |
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.
Can you use buildPhase
and installPhase
here? We want to have at least fixup phase running unless there is a good reason not to.
pkgs/tools/video/untrunc/default.nix
Outdated
libavConfiguredSrc = libav_12.overrideAttrs (oldAttrs: { | ||
name = "libav-configured-src"; | ||
outputs = [ "out" ]; | ||
phases = [ "unpackPhase" "configurePhase" "copyConfiguredSrc" ]; |
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.
patchPhase
should be added here as well.
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.
Done.
Should I rename copyConfiguredSrc
to just installPhase
?
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.
Yes, that would be less confusing to people.
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.
Done. I've fixed all remaining quirks.
5279dbc
to
49315f0
Compare
@GrahamcOfBorg build untrunc |
Failure on x86_64-darwin (full log) Partial log (click to expand)
|
Success on x86_64-linux (full log) Partial log (click to expand)
|
Success on aarch64-linux (full log) Partial log (click to expand)
|
The Darwin build has failed. Should I add |
@erikarvstedt no, that is fine. The build is marked as unsupported on darwin as dependencies are supported. We can safely assume that platforms supporting libav are likely to support untrunc as well. |
Thanks for guiding me through this. 😃 |
Grüße aus Edinburgh zurück. |
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)