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

[addons] improve downgrading/version changing support #8038

Merged
merged 7 commits into from
Sep 19, 2015

Conversation

tamland
Copy link
Member

@tamland tamland commented Sep 13, 2015

This adds various client changes needed to support having multiple versions in the same repository and for installing specific versions from specific repositories.

On the gui side it changes the 'update' functionality in the info dialog to show a list of available versions and repositories which can be installed. Note that requires server side support. Currently there is only one version kept in the official repository.

@tamland tamland added Type: Feature non-breaking change which adds functionality Type: Cleanup non-breaking change which removes non-working or unmaintained functionality labels Sep 13, 2015
if (!m_localAddon)
return;

auto* dialog = static_cast<CGUIDialogSelect*>(g_windowManager.GetWindow(WINDOW_DIALOG_SELECT));

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

This comment was marked as spam.

@mkortstiege
Copy link
Member

You might want to remove the DialogSelect changes a you've PR'd that one separately.

@tamland
Copy link
Member Author

tamland commented Sep 15, 2015

jenkins build this please

@tamland
Copy link
Member Author

tamland commented Sep 16, 2015

@mkortstiege good to go?

@mkortstiege
Copy link
Member

Looks good. jenkins build and merge.

@MartijnKaijser MartijnKaijser added this to the Jarvis 16.0-alpha3 milestone Sep 19, 2015
@jenkins4kodi jenkins4kodi merged commit aab1345 into xbmc:master Sep 19, 2015
@MartijnKaijser
Copy link
Member

I wonder why the "Upgrade" button is enabled if i just installed the latest version from repo?
Opening it just gives me the same one i already have and clicking it just re-install the same. Doesn't really make sense to allow that.

@tamland
Copy link
Member Author

tamland commented Sep 27, 2015

It doesn't strictly do "upgrading" any more but opens a window where you can manage/see all available versions (i plan to put rollback in there as well). It doesn't make any sense to disable that in some special cases. It will show an empty list if it's in no repos, it shows one item highlighted you have the latest version. The reason it reinstalls is because we don't track what is installed and where they come from, so there's no way to know it's the same one; other than comparing the version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Cleanup non-breaking change which removes non-working or unmaintained functionality Type: Feature non-breaking change which adds functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants