Skip to content
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

Try to detect circular dependencies #125

Open
MartijnKaijser opened this issue Oct 13, 2018 · 3 comments
Open

Try to detect circular dependencies #125

MartijnKaijser opened this issue Oct 13, 2018 · 3 comments

Comments

@MartijnKaijser
Copy link
Member

Sometimes developers add circular dependencies by accident which freaks out Kodi. Doesn't happen often but it's quite nasty and we had lots of problems with this since add-ons where added to XBMC

<addon id="plugin.video.fattoquotidianotv" name="Il Fatto Quotidiano TV" version="1.0.4" provider-name="NeverWise"> <requires> <import addon="xbmc.python" version="2.14.0"/> <import addon="script.module.neverwise" version="1.0.6"/> <import addon="plugin.video.youtube" version="4.4.10" optional="true"/> <import addon="plugin.video.serviziopubblico" version="1.0.6" optional="true"/> </requires>

<addon id="plugin.video.serviziopubblico" name="Servizio Pubblico" version="1.0.6" provider-name="NeverWise"> <requires> <import addon="xbmc.python" version="2.14.0"/> <import addon="script.module.neverwise" version="1.0.6"/> <import addon="plugin.video.fattoquotidianotv" version="1.0.4" optional="true"/> </requires>

@mzfr
Copy link
Contributor

mzfr commented Oct 15, 2018

This one could be a bit tricky because when we'll be checking the dependencies of an addon say plugin.video.fattoquotidianotv, we'll have to go on all the add-ons present in the requires section. And by go means sending requests to all require add-ons and getting the addon.xml files for them and then reading dependencies and matching it.

@MartijnKaijser
Copy link
Member Author

Now you know why Kodi freaks out if we have these addons in repo :)

@Rechi
Copy link
Member

Rechi commented Oct 15, 2018

addons.xml contains all the info. Besides that the checker doesn't consider anything else then addons.xml (from the server) and the local addon.xml of the currently checked addon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants