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

choco tried to resolve disable repo #169

Closed
chevdor opened this issue Mar 13, 2015 · 2 comments
Closed

choco tried to resolve disable repo #169

chevdor opened this issue Mar 13, 2015 · 2 comments
Assignees
@ferventcoder
Labels
5 - Released Bug
Milestone
0.9.9.5

Comments

@chevdor
Copy link

chevdor commented Mar 13, 2015

choco 0.9.9.2

I am using the local repo in addition to the official chocolatey repo:
choco sources add -name local -source http://idontexist:4444/nuget

Obviously, this repo will not resolve...

When my local repo is unavailable, clist fails:

λ clist notepadplusplus
Chocolatey v0.9.9.2
The remote name could not be resolved: 'idontexist'

and I don´t even get the results from the other repos.

Since my local repo is the issue, I try disabling it:
choco sources disable -name local

even disabled, the repos is tested:

clist notepadplusplus
Chocolatey v0.9.9.2
The remote name could not be resolved: 'idontexist'

First of all, I think clist should still return the results from the other repos even if one cannot resolve.
Also, if a repo is disabled, the issue must definitely not occure.
@ferventcoder
Copy link
Member

Sounds like a bug. Also ugh.

@ferventcoder ferventcoder added this to the 0.9.9.3 milestone Mar 13, 2015
@ferventcoder ferventcoder modified the milestones: 0.9.9.3, 0.9.9.4, 0.9.9.5 Mar 29, 2015
ferventcoder added a commit that referenced this issue Apr 14, 2015
@ferventcoder
(GH-169) Do not resolve disabled sources
1c9ce4c
@ferventcoder
Copy link
Member

Thanks for reporting. This will be out in 0.9.9.5

@ferventcoder ferventcoder self-assigned this Apr 17, 2015
ferventcoder added a commit that referenced this issue Apr 20, 2015
@ferventcoder
Merge branch 'stable'
2ed0d63 
* stable: (22 commits)
  (GH-121) Making Uninstall-ChocolateyZipPackage more robust when
deleting files that were copied during installation of the Zip package
  (doc) update changelog/nuspec
  (GH-238) ApiKey source matching intuitive
  (maint) formatting
  (GH-240) Set CredentialProvider for NuGet
  (GH-240) ChocolateyNugetCredentialProvider
  (GH-240) Add default sources to machine sources
  (maint) Only warn subcommand list if not empty
  (GH-171) Use RedirectedHttpClient
  (GH-240) pass credentials at runtime
  (GH-240)(config) Add machine sources
  (doc) how to quote values
  (GH-230) Export all functions and aliases imported
  (GH-230) Fix Issues with Generate/Remove BinFile
  (GH-185) Remove console prompt default choices
  (GH-186) Uninstall - no prompt for one version
  (GH-182) Ask before printing ps1 scripts
  (GH-187) Show log file path in messages.
  (maint) formatting
  (GH-169) Do not resolve disabled sources
  ...

Conflicts:
	src/chocolatey/infrastructure.app/commands/ChocolateySourceCommand.cs
	src/chocolatey/infrastructure.app/runners/GenericRunner.cs
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
Projects
None yet
Milestone
0.9.9.5
Development

No branches or pull requests
3 participants
@ferventcoder @chevdor @gep13