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

[bug] conan remote remove "*" doesn't remove duplicate remotes #13569

Closed
cqc-alec opened this issue Mar 30, 2023 · 3 comments · Fixed by #13574
Closed

[bug] conan remote remove "*" doesn't remove duplicate remotes #13569

cqc-alec opened this issue Mar 30, 2023 · 3 comments · Fixed by #13574
Assignees

Comments

@cqc-alec
Copy link

Environment details

  • Operating System+version: all
  • Compiler+version: N/A
  • Conan version: 2.0.2
  • Python version: 3.10

Steps to reproduce

  1. conan remote add (some remote)
  2. conan remote add (exactly the same remote -- a warning is printed but no error)
  3. conan remote remove "*"
  4. conan remote list

The remote remains (only one instance has been removed).

Logs

No response

@memsharded
Copy link
Member

Hi @cqc-alec

Thanks for your report.

I can see that conan remote remove * is not removing repeated remotes names.
But I feel that it shouldn't be possible to add twice the same remote, that should be an error (if force, it should be overwritten), it doesn't make sense that there are 2 remotes with same name and same url. What do you think? Lets fix the original bug? I mean conan remote add .... --force will not add a new remote with same name and same url, but keep the existing one?

@cqc-alec
Copy link
Author

Hi @memsharded , I agree, and this was my original expectation: that I could use conan remote add ... --force to add the remote if it didn't exist, and do nothing otherwise (this was the conan 1 behaviour). It was only later I realized that I had a huge list of duplicated remotes!

@memsharded
Copy link
Member

Suggesting #13574 fix for next 2.0.3 release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants