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

Determine if nuget source name is case-sensitive and ensure disable and other uses works properly given that #8668

Closed
asherber opened this issue Oct 8, 2019 · 2 comments
Labels
Area:Settings NuGet.Config and related issues Product:NuGet.exe NuGet.exe RegressionFromPreviousRTM A regression from the last RTM. Example: worked in 6.2, doesn't work in 6.3 Triage:NeedsTriageDiscussion Type:Bug
Milestone

Comments

@asherber
Copy link

asherber commented Oct 8, 2019

Details about Problem

NuGet product used (NuGet.exe | VS UI | Package Manager Console | dotnet.exe): nuget.exe

NuGet version (x.x.x.xxx): 5.2.0.6090

dotnet.exe --version (if appropriate):

VS version (if appropriate):

OS version (i.e. win10 v1607 (14393.321)): win10 v1709 (16299.1268)

Worked before? If so, with which NuGet version: Don't know

Detailed repro steps so we can see the same problem

  1. nuget sources add -name FOO -source https://example.com
  2. nuget sources list (confirm new source is there)
  3. nuget sources disable -name foo (nuget reports Package source with Name: foo disabled successfully.)
  4. nuget sources list (source is still enabled!)
  5. nuget sources disable -name FOO (nuget reports Package source with Name: foo disabled successfully.)
  6. nuget sources list (source is disabled)

Note that nuget sources remove -name foo works just fine to remove a source whose name is FOO.

@kartheekp-ms kartheekp-ms added the Product:NuGet.exe NuGet.exe label Oct 9, 2019
@nkolev92 nkolev92 added the Area:Settings NuGet.Config and related issues label Oct 9, 2019
@kartheekp-ms kartheekp-ms added RegressionFromPreviousRTM A regression from the last RTM. Example: worked in 6.2, doesn't work in 6.3 Type:Bug Triage:NeedsTriageDiscussion labels Oct 9, 2019
@kartheekp-ms
Copy link
Contributor

Noticed that this issue was fixed in nuget.exe 4.8.2 version and broken in later releases.

@kartheekp-ms kartheekp-ms added this to the Backlog milestone Oct 10, 2019
@rrelyea rrelyea changed the title nuget sources disable and case-sensitivity Determine if nuget source name is case-sensitive and ensure disable and other uses works properly given that Nov 28, 2019
@rrelyea
Copy link
Contributor

rrelyea commented Jan 3, 2020

Looked into this a bit. Looks like nuget sources disable -name foo will persist a disabled package source. However, it looks like packageSource FOO and disabledPackageSource foo are not found to match...so our system treats FOO as enabled.

Haven't looked at pre-4.9 behavior. (@kartheekp-ms said that 4.8.2 worked), but 4.9 is where we did major config file (setting) system enhancements.
Doing work for dotnet nuget disable source now... won't address this as part of that work, but we should fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area:Settings NuGet.Config and related issues Product:NuGet.exe NuGet.exe RegressionFromPreviousRTM A regression from the last RTM. Example: worked in 6.2, doesn't work in 6.3 Triage:NeedsTriageDiscussion Type:Bug
Projects
None yet
Development

No branches or pull requests

4 participants