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
mlt: 6.10.0 -> 6.12.0 #51370
mlt: 6.10.0 -> 6.12.0 #51370
Conversation
|
||
src = fetchFromGitHub { | ||
owner = "mltframework"; | ||
repo = "mlt"; | ||
rev = "v${version}"; | ||
sha256 = "0ki86yslr5ywa6sz8pjrgd9a4rn2rr4mss2zkmqi7pq8prgsm1fr"; | ||
sha256 = "0pzm3mjbbdl2rkbswgyfkx552xlxh2qrwzsi2a4dicfr92rfgq6w"; | ||
}; | ||
|
||
buildInputs = [ |
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.
At least makeWrapper
and pkgconfig
should be into nativeBuildInputs
The configure flag |
That's how you enable gpl x2 => gplv2 code, right?! 🤣 Cleaning up, thank you for review and apologies for missing these things! |
Seems we half-upgraded this by bumping qt-enabled variant in b0f912b or so. Rebasing/fixing underway... |
* needs 'which' to find bins from ladspa-sdk, add * cleanup a bit, qt seems fine as-is (?)
ff592c9
to
bce556c
Compare
Merged the two mlt expressions, which seems high-level good (as well as a few dependencies that seem no heavier than what's already pulled in) but should make sure shotcut and kdenlive both still work. I nuked the "fix motiv.dev" postInstall as likely kruft but will revisit. |
This seems to have suffered some bitrot, if we have a better PR or so feel free to close :). Hopefully will be revisited eventually if not :). |
Thank you for your contributions.
|
Master already has version 6.22.1. If you want to move any of the other changes over please open a new PR. |
Motivation for this change
https://github.com/mltframework/mlt/releases/tag/v6.12.0
Notes mention new VA-API support, and other goodies..
among many important fixes! :)
Should look through them more closely,
possibly adding new deps (for vaapi?).
(note to myself but open to anyone with a second :))
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)