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

[dev.icinga.com #11369] Chocolatey package is missing uninstall function #4027

Closed
icinga-migration opened this issue Mar 13, 2016 · 3 comments
Closed
Labels
Milestone

Comments

@icinga-migration
Copy link
Member

@icinga-migration icinga-migration commented Mar 13, 2016

This issue has been migrated from Redmine: https://dev.icinga.com/issues/11369

Created by TheFlyingCorpse on 2016-03-13 22:09:48 +00:00

Assignee: TheFlyingCorpse
Status: Closed (closed on 2016-03-14 17:48:15 +00:00)
Target Version: 2.4.4
Last Update: 2016-03-15 12:18:14 +00:00 (in Redmine)

Icinga Version: 2.4.3
Backport?: Already backported
Include in Changelog: 1

When uninstalling with chocolatey, it doesnt actually remove the package when running silently, it just removes it from choco and not from the system itself.

See git for a fix to this :-)

Changesets

2016-03-13 22:13:01 +00:00 by TheFlyingCorpse 10a0774

Added uninstall to the chocolatey package

refs #11369

2016-03-14 05:28:11 +00:00 by TheFlyingCorpse f610ddd

Added uninstall to the chocolatey package

refs #11369

2016-03-14 17:42:10 +00:00 by TheFlyingCorpse 47c3253

Add uninstall to the chocolatey package

refs #11369

2016-03-15 08:35:53 +00:00 by (unknown) 08c5006

Update AUTHORS

refs #11369

2016-03-15 12:17:10 +00:00 by TheFlyingCorpse 901618d

Add uninstall to the chocolatey package

refs #11369

2016-03-15 12:17:55 +00:00 by (unknown) c4f9c34

Update AUTHORS

refs #11369
@icinga-migration
Copy link
Member Author

@icinga-migration icinga-migration commented Mar 14, 2016

Updated by gbeutner on 2016-03-14 07:15:42 +00:00

  • Status changed from New to Assigned
  • Assigned to set to TheFlyingCorpse
  • Target Version set to 2.4.4
  • Commit messages should use the imperative, present-tense

Other than that feel free to merge this into the master branch.

@icinga-migration
Copy link
Member Author

@icinga-migration icinga-migration commented Mar 14, 2016

Updated by TheFlyingCorpse on 2016-03-14 17:48:15 +00:00

  • Status changed from Assigned to Closed

Resolved, commited to master

@icinga-migration
Copy link
Member Author

@icinga-migration icinga-migration commented Mar 15, 2016

Updated by mfriedrich on 2016-03-15 12:18:14 +00:00

  • Backport? changed from Not yet backported to Already backported
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.