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
kodi: 17.4 -> 17.6 and move to cmake #29830
Conversation
Upgrade to 17.5 works fine but test suite still takes about 4 hours to run (and has a few failures). |
|
||
let | ||
kodi_version = "17.4"; | ||
kodiReleaseDate = "20170422"; | ||
kodiVersion = "17.4"; |
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.
Shouldn't that be 17.5?
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.
@domenkozar - it's probably easier for everyone involved if I push the right branch...
Thanks @edwtjo - this PR disables the checks and I want to be sure everybody is cool with that. |
ffmpeg is now built as a separate derivation using the kodi makefile to avoid having to rebuild ffmpeg every time kodi is changed. Additionally, due to the far superior cmake output a number of dependencies were identified that have been added as well.
Supersedes #31787 if everyone is cool with this. |
It has a number of security patches (specifically with regards to ffmpeg) - we should probably backport this to 17.09. I'll do later during the weekend. |
Motivation for this change
Working on kodi was quite painful as it would rebuild ffmpeg every time. ffmpeg is now built as a separate derivation using the kodi makefile to avoid having to rebuild ffmpeg every time kodi is changed.
Additionally, due to the far superior cmake output a number of dependencies were identified that have been added as well.
This PR compiles but the tests fail (at least no more segfaulting). The main issue with troubleshooting the tests is that they take FOREVER to run now - approximately 4 hours as each test takes 30 seconds. My guess is that there is a timeout of some kind due to our sandbox.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)Cc: @domenkozar @titanous @edwtjo @joachifm