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
handbrake: 1.2.2 -> 1.3.0 #76112
handbrake: 1.2.2 -> 1.3.0 #76112
Conversation
Thanks to @jtojnar for showing the way forward with this comment. |
d1a0a82
to
760dcc8
Compare
I've done things differently to possibly better insulate our changes to the upstream build. |
3afde0e
to
2c5cebb
Compare
This update is important as handbrake 1.2.2 no longer builds with nixpkgs due to an unrelated update to pango-1.44.x as described [here](HandBrake/HandBrake#2226). Fixes NixOS#73399
2c5cebb
to
d46f197
Compare
These were included by the handbrake build because they are used by external dependencies. In nixpkgs, we build those dependencies (e.g. codecs) separately, and so excise the check for these tools from the handbrake build.
d46f197
to
7978601
Compare
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.
LGTM, program starts.
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.
tested with nix-review, seems to work, weird behavior when I try to open files
This update is important as handbrake 1.2.2 no longer builds with
nixpkgs due to an unrelated update to pango-1.44.x as described
here.
Fixes #73399
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @