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.

Sign up for GitHub

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

Jump to bottom

Uninstall ignores the version parameter #861

Closed
tmsns opened this issue Jul 12, 2016 · 0 comments
Closed

Uninstall ignores the version parameter #861

tmsns opened this issue Jul 12, 2016 · 0 comments
Assignees
@ferventcoder
Labels
5 - Released Bug Up For Grabs
Milestone
0.10.0

Comments

@tmsns
Copy link
Contributor

tmsns commented Jul 12, 2016 •
edited by ferventcoder

What You Are Seeing?

When executing an uninstall of a specfic version of a package which was side-by-side installed, Chocolatey will always uninstall the latest package.

eg. choco uninstall my-package --version 1.0.2

What is Expected?

We expect the specified version to be uninstalled.

How Did You Get This To Happen? (Steps to Reproduce)

  1. Install a (specific) version of a package. This is not required to be side-by-side.
    choco install my-package--version 2.0.0
  2. Install a second (previous) version of a package side-by-side
    choco install my-package --version 1.0.2 -sxs
  3. Uninstall the oldest version.
    choco uninstall my-package --version 1.0.2

Output Log

In the log we see that the the newest version has been selected (2.0.0)

Reference: https://groups.google.com/d/msgid/chocolatey/85aad436-547f-40c7-a4de-ad8310c13ed2%40googlegroups.com
tmsns added a commit to tmsns/choco that referenced this issue Jul 12, 2016
@tmsns
(chocolateyGH-861) Fix uninstall to use version parameter
fdec349 
Previously when executing an uninstall of a specific version of a
package which was side-by-side installed, Chocolatey would always
uninstall the latest package.

This fix will change the NugetService.uninstall_run to actually
use the version when selecting the package to remove.
tmsns added a commit to tmsns/choco that referenced this issue Jul 12, 2016
@tmsns
(chocolateyGH-861) Fix uninstall to use version parameter
bccfe33 
Previously when executing an uninstall of a specific version of a
package which was side-by-side installed, Chocolatey would always
uninstall the latest package.

This fix will change the NugetService.uninstall_run to actually
use the version when selecting the package to remove.
@ferventcoder ferventcoder self-assigned this Jul 12, 2016
@ferventcoder ferventcoder added this to the 0.9.10.4 milestone Jul 12, 2016
@tmsns tmsns mentioned this issue Jul 12, 2016
Closed
ferventcoder pushed a commit that referenced this issue Aug 3, 2016
@tmsns @ferventcoder
(GH-861) Fix uninstall to use version parameter
e9ecf80 
Previously when executing an uninstall of a specific version of a
package which was side-by-side installed, Chocolatey would always
uninstall the latest package.

This fix will change the NugetService.uninstall_run to actually
use the version when selecting the package to remove.
ferventcoder added a commit that referenced this issue Aug 3, 2016
@ferventcoder
Merge branch 'pr863' into stable
faed657 
* pr863:
  (GH-861) Fix uninstall to use version parameter
ferventcoder added a commit that referenced this issue Aug 3, 2016
@ferventcoder
Merge branch 'stable'
900ba95 
* stable:
  (GH-861) Fix uninstall to use version parameter
  (GH-889) Fix: Search by Id is Case Sensitive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ferventcoder ferventcoder

Labels
5 - Released Bug Up For Grabs
Projects
None yet
Milestone
0.10.0
Development

Successfully merging a pull request may close this issue.

(GH-861) Fix uninstall to use version parameter tmsns/choco
3 participants
@ferventcoder @gep13 @tmsns