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

Re-enable --no-remote-name #8931

Closed
wants to merge 1 commit into from
Closed

Conversation

verhovsky
Copy link
Contributor

@verhovsky verhovsky commented May 28, 2022

The docs for --remote-name-all say

you must use "-o -" or --no-remote-name.

but with curl 7.83 on macOS I get

$ curl --no-remote-name
curl: option --no-remote-name: used '--no-' for option that isn't a boolean
curl: try 'curl --help' or 'curl --manual' for more information

So either the option should be put back (it was incorrectly marked as ARG_NONE instead of ARG_BOOL) or the docs should be updated.

@verhovsky verhovsky changed the title Undocument --no-remote-name Undocument deleted --no-remote-name option May 28, 2022
@verhovsky verhovsky changed the title Undocument deleted --no-remote-name option Re-enable --no-remote-name May 28, 2022
bagder
bagder approved these changes May 28, 2022
@bagder
Copy link
Member

@bagder bagder commented May 28, 2022

I would be great to also write up a test case that uses this feature so that we can make sure we don't break it again in the future...

@bagder
Copy link
Member

@bagder bagder commented May 31, 2022

Ok, I have written a test locally that verifies. I'll make a separate PR after I've merged this.

@bagder bagder closed this in 83ee5c4 May 31, 2022
@bagder
Copy link
Member

@bagder bagder commented May 31, 2022

Thanks!

bagder added a commit that referenced this issue May 31, 2022
bagder added a commit that referenced this issue May 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants