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: implement kodiWithPlugins
, add more plugins
#46196
Conversation
/cc @titanous @edwtjo @cpages @worldofpeace from git-blame of |
5d296a9
to
f02153c
Compare
f02153c
to
f62a31b
Compare
025279d
to
ec7d00e
Compare
Success on x86_64-linux (full log) Attempted: kodi, kodiPlugins.simpleplugin, kodiPlugins.yatp The following builds were skipped because they don't evaluate on x86_64-linux: kodiPlugins.steam-launcher Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: kodiPlugins.simpleplugin, kodiPlugins.yatp The following builds were skipped because they don't evaluate on aarch64-linux: kodi, kodiPlugins.steam-launcher Partial log (click to expand)
|
with 3.8k rebuilds this should go to staging. |
ec7d00e
to
031e1cf
Compare
@xeji
Ugh, I see, `pythonPackages.requests` is a mass rebuilt.
I don't want to move this to staging as I rebased this several times already, it touches a lot of kodi stuff and I'm tired of fixing those conflicts.
I removed YATP from here, changes to plugins will go to `staging` after this gets merged to `master`.
@worldofpeace
Please drop this plugin as it no longer exists.
Ok, will do in that second PR.
This is now became a purely infrastructure PR.
|
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: kodi, kodiPlugins.steam-launcher Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: kodi The following builds were skipped because they don't evaluate on x86_64-linux: kodiPlugins.steam-launcher Partial log (click to expand)
|
I've rebuilt everything (with mass-rebuild YATP patches included) on top of this revision. I can see no issues. LGTM. Merge at will. |
@xeji Thanks! |
Changes to plugins: #46403. |
No loner exists, I couldn't find an official replacement repo, should be removed according to NixOS#46196 (comment)
No loner exists, I couldn't find an official replacement repo, should be removed according to #46196 (comment)
No loner exists, I couldn't find an official replacement repo, should be removed according to NixOS#46196 (comment)
Motivation for this change
I tried to add YATP to Kodi, ended up rewriting a bunch of stuff.
Known issues
I did not touch
wrapKodi
and the default wrappedkodi
, but, really, we should probably deprecate and then remove it.Things done
kodi
with its configuration vianixpkgs.config
and the newkodiWithPlugins
work.Don't merge sooner than 2018-09-09 23:59 UTC.Merge at will.