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

[PVR] Fix: pvr client addons must be reloaded on profile switch to re-read addon config etc. #10577

Merged
merged 1 commit into from Sep 28, 2016

Conversation

@ksooo
Copy link
Member

commented Sep 28, 2016

When switching user profiles, pvr client addons must be reloaded to re-read addon config etc.

This should fix a problem reported in the forum http://forum.kodi.tv/showthread.php?tid=282907&pid=2422637#pid2422637

@Jalle19 for review?

@ksooo ksooo added this to the Krypton 17.0-beta3 milestone Sep 28, 2016
@ksooo ksooo force-pushed the ksooo:pvr-fix-profile-switch branch from 31c62b8 to c7e430e Sep 28, 2016
…-read addon config etc.
@ksooo ksooo force-pushed the ksooo:pvr-fix-profile-switch branch from c7e430e to 66c7cc8 Sep 28, 2016
@ksooo

This comment has been minimized.

Copy link
Member Author

commented Sep 28, 2016

jenkins build this please

@da-anda

This comment has been minimized.

Copy link
Member

commented Sep 28, 2016

I guess this should apply to other binary add-ons as well. Did anybody check if it's the case for these already?

@ksooo

This comment has been minimized.

Copy link
Member Author

commented Sep 28, 2016

I guess this should apply to other binary add-ons as well.

yes

Did anybody check if it's the case for these already?

i don't think so.

@ksooo ksooo merged commit 2f7ce6e into xbmc:master Sep 28, 2016
2 of 3 checks passed
2 of 3 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
jenkins.kodi.tv You did a great job. Have a cookie.
Details
@ksooo ksooo deleted the ksooo:pvr-fix-profile-switch branch Sep 28, 2016
@Jalle19

This comment has been minimized.

Copy link
Member

commented Sep 28, 2016

I was thinking the same when looking at the code. @garbear and @AlwinEsch might be interested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.