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

Basic uninstall clean-up #56

Merged
merged 1 commit into from
Mar 23, 2017
Merged

Basic uninstall clean-up #56

merged 1 commit into from
Mar 23, 2017

Conversation

macmpi
Copy link
Contributor

@macmpi macmpi commented Mar 13, 2017

Avoids kiosk starts even plugin is uninstalled.
Obviously, this would require fixing the following issue before, so that uninstall.sh is called.
volumio/Volumio2#851

While at it, other plugin uninstall thingy that needs consideration:
volumio/Volumio2#905 (not absolutely required for this specific one, but necessary too)

Avoids kiosk starts even plugin is uninstalled.
Obviously, this would require fixing the following issue before, so that uninstall.sh is called.
volumio/Volumio2#851

While at it, other plugin uninstall thingy that needs consideration:
volumio/Volumio2#905 (not absolutely required for this specific one, but necessary too)
@volumio
Copy link
Owner

volumio commented Mar 14, 2017

I think it won't start anyway, since the systemctl start volumiokiosk is invoked by the plugin anyway...

@macmpi
Copy link
Contributor Author

macmpi commented Mar 14, 2017

Well, I submitted it because it would still start after uninstall.
Similarly it started even if plugin was "disabled" (the reason why I uninstalled in first place).

@volumio volumio merged commit 055bcf4 into volumio:master Mar 23, 2017
@macmpi macmpi deleted the patch-1 branch March 23, 2017 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants