git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: remote#branch

From
Tom Prince <tom.prince@ualberta.net>
Date
Oct 30, 2007, 16:02 UTC
Message-ID
<20071030160232.GB2640@hermes.priv>
In-Reply-To
<alpine.LFD.0.999.0710300738550.30120@woody.linux-foundation.org>
On Tue, Oct 30, 2007 at 07:59:45AM -0700, Linus Torvalds wrote:
Show 13 quoted lines
> > Not that I care, but git should probably handle things consistently.
> 
> Git has been, and *is* entirely consistent. It uses convenient repo names. 
> If you don't want to call them url's, then call them "repository name". 
> Call them whatever. But they are 100% obvious, even if there are multiple 
> forms of them (and *none* of the forms do any quoting at all):
> 
>  - <remote shorthand> ("origin")
>  - <path> ("../git.git")
>  - <host>:<path> ("master.kernel.org:/pub/scm/...")
>  - <protocol>://<host>/<path> ("git://repo.or.cz/...")
> 
> See? We may not follow RFC's, but we follow "easy to use".

Well, only the last one actually looks like a URL, so that is the only this discussion is about. I don't think anyone is suggesting that the first three be changed at all. So, to use your terminology, git has a variety of ways to specify a repo name, one of which happens to be a URL (or looks like one). The suggestion is that we should make that way (and only that way) behave like a RFC URL.

And git should be consistent with web browsers, automatically quoting things it gets passed. I think the only point of contention is probably how to deal with URLs that git receives that are already quoted.

1. We ignore the quoting and re-encode everything for the http transport.
2. We honour the encoding and decode everything for the git transport.
3. We handle git:// and http:// different, so that the three git:// URLs below
refer to different repositories, while the three http:// URLs give refer to
the same repository.
Show 7 quoted lines
> > git://repo.or.cz/linux-2.6/linux acpi-2.6/ibm-acpi-2.6.git
> > git://repo.or.cz/linux-2.6/linux+acpi-2.6/ibm-acpi-2.6.git
> > git://repo.or.cz/linux-2.6/linux%20acpi-2.6/ibm-acpi-2.6.git
> 
> > http://repo.or.cz/linux-2.6/linux acpi-2.6/ibm-acpi-2.6.git
> > http://repo.or.cz/linux-2.6/linux+acpi-2.6/ibm-acpi-2.6.git
> > http://repo.or.cz/linux-2.6/linux%20acpi-2.6/ibm-acpi-2.6.git

The third possibility is probably what we do now, which is why I am suggesting git is inconsistent. The first will fall down when using a repository that is colocated, and somebody copies a URL from the web browsers location bar (which will be properly encoded). Which leaves the second.

  Tom
Previous: Linus TorvaldsNext: Linus Torvalds
Message 24 of 75 in “Git homepage: remove all the references to Cogito”
  1. Git homepage: remove all the references to CogitoPaolo Ciarrocchi, Oct 15, 2007
  2. Petr BaudisOct 16, 2007
  3. Matthieu MoyOct 16, 2007
  4. Paolo CiarrocchiOct 16, 2007
  5. gitweb: Speed up get_projects_list for large source treesLuke Lu, Oct 16, 2007
  6. Andreas EricssonOct 16, 2007
  7. Petr BaudisOct 16, 2007
  8. cogito and remote#branch, was Re: [PATCH] Git homepage: remove all the references to CogitoJohannes Schindelin, Oct 16, 2007
  9. Jan HudecOct 16, 2007
  10. Johannes SchindelinOct 16, 2007
  11. Jan HudecOct 27, 2007
  12. Johannes SchindelinOct 27, 2007
  13. Jan HudecOct 29, 2007
  14. Linus TorvaldsOct 29, 2007
  15. Theodore TsoOct 29, 2007
  16. Linus TorvaldsOct 29, 2007
  17. Johannes SchindelinOct 29, 2007
  18. Theodore TsoOct 30, 2007
  19. Junio C HamanoOct 30, 2007
  20. Theodore TsoOct 30, 2007
  21. Linus TorvaldsOct 30, 2007
  22. Tom PrinceOct 30, 2007
  23. Linus TorvaldsOct 30, 2007
  24. Tom PrinceOct 30, 2007
  25. Linus TorvaldsOct 30, 2007
  26. Matthieu MoyOct 30, 2007
  27. Linus TorvaldsOct 30, 2007
  28. Linus TorvaldsOct 30, 2007
  29. Pascal ObryOct 30, 2007
  30. Linus TorvaldsOct 30, 2007
  31. Randal L. SchwartzOct 30, 2007
  32. Linus TorvaldsOct 30, 2007
  33. Nicolas PitreOct 30, 2007
  34. Jeff KingOct 30, 2007
  35. Jakub NarebskiOct 31, 2007
  36. Jeff KingOct 31, 2007
  37. Jakub NarebskiOct 31, 2007
  38. Jeff KingOct 31, 2007
  39. Andreas EricssonOct 31, 2007
  40. Mike HommeyOct 31, 2007
  41. Andreas EricssonOct 31, 2007
  42. Mike HommeyOct 31, 2007
  43. Jakub NarebskiNov 1, 2007
  44. Theodore TsoNov 1, 2007
  45. Andreas EricssonNov 1, 2007
  46. Pascal ObryOct 31, 2007
  47. David KastrupOct 31, 2007
  48. Linus TorvaldsOct 31, 2007
  49. Jeff KingOct 31, 2007
  50. Linus TorvaldsOct 31, 2007
  51. Jeff KingOct 31, 2007
  52. Linus TorvaldsOct 31, 2007
  53. Andreas EricssonOct 31, 2007
  54. Jeff KingOct 31, 2007
  55. David KastrupOct 31, 2007
  56. Wincent ColaiutaOct 31, 2007
  57. Robin RosenbergOct 31, 2007
  58. Pascal ObryOct 31, 2007
  59. Petr BaudisOct 31, 2007
  60. Pascal ObryOct 30, 2007
  61. Jan HudecOct 30, 2007
  62. Linus TorvaldsOct 30, 2007
  63. Erik WarendorphOct 31, 2007
  64. Johannes SchindelinOct 30, 2007
  65. Martin LanghoffOct 31, 2007
  66. Linus TorvaldsOct 31, 2007
  67. Jeff KingOct 31, 2007
  68. Martin LanghoffOct 31, 2007
  69. Jeff KingOct 31, 2007
  70. Johannes SchindelinOct 31, 2007
  71. Linus TorvaldsOct 30, 2007
  72. Johannes SchindelinOct 29, 2007
  73. Jonas FonsecaOct 16, 2007
  74. Petr BaudisOct 31, 2007
  75. Jonas FonsecaOct 31, 2007

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.