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
cmd/go: support for spaces or URL encoded spaces in git URL #37556
Comments
Could clarify if this is about spaces in repository URLs or package paths? Arguments to I think spaces in repository URLs are already allowed. I don't know of any reason they should be rejected, so if they're not working, we should fix that. It sounds like this is more about allowing spaces in package paths though.
|
Hi @jayconrod This is about spaces in the git repo URL, AFAIK. In my example above I forgot to add that I'm using the
|
@mbrancato, the argument to The package path often corresponds to the URL, but they are separate entities. (The |
Given that there are straightforward workarounds (either choose a project name without spaces, or set up a |
@mbrancato Your examples reference package paths on the command line. Repository URLs won't appear on the command line or in .go files; they'll only show up in As an example of how this is typically set up, consider the module
From that, I believe spaces are already supported in repository URLs, but if you find that's not the case, please give an example where it's not working. I'm not convinced spaces should be allowed in module and package paths. Package paths use a limited set of characters for clarity and simplicity. If the underlying repository URL has spaces, that's okay, but the package path will need to be different from the repository URL. |
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (
go env
)?go env
OutputWhat did you do?
What did you expect to see?
Azure DevOps supports
go get
butgo get
does not support spaces or HTTP encoded spaces (which is what most tools use to get around this).What did you see instead?
invalid char '%'
and
invalid char ' '
The text was updated successfully, but these errors were encountered: