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
[addons][inputstream] allow instance version check and fix inputstream addon way #16535
[addons][inputstream] allow instance version check and fix inputstream addon way #16535
Conversation
Is this the fix for xbmc/inputstream.adaptive#313 ? |
Seems to be connected because of your given commit examples this way came. |
…eck depends) By this also the "CAddon::GetDependencyVersion(..,)" is changed to, and CBinaryAddonBase becomes it added too, to have on running instances.
Before was the addon own version used where never match a inputstream instance version.
980f4fa
to
4142840
Compare
@candrews yes |
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 -> works fine, thx!
Confirmed also that it fixed my problem. Thank you! |
Why was this merged with a failed Jenkins build? |
The test was OK, only by Win64 on end a jenkins build script failure EDIT: Also as another was Win32 and Win64 UWP complete |
Description
The fist commit:
[addons] add new function "CAddonInfo::DependencyVersion(...)" (to check depends)
By this also the "CAddon::GetDependencyVersion(..,)" is changed to, and
CBinaryAddonBase becomes it added too, to have on running instances.
The second commit:
[addons][inputstream] fix depedency version check
Before was the addon own version used where never match a inputstream instance
version.
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of change
Checklist: