{"thread":{"id":"23507","subject":"[PATCH 2/5] Documentation/urls: Rewrite to accomodate ::
","startedAt":"2010-04-18T00:54:18Z","lastAt":"2010-04-18T21:56:55Z","messageCount":9,"participants":["Ramkumar Ramachandra","Jonathan Nieder","Junio C Hamano","Sverre Rabbelier"],"isPatch":true,"patchVersion":1,"patchTotal":5},"messages":[{"id":"139797","messageId":"1271552047-sup-9523@kytes","threadId":"23507","inReplyTo":null,"subject":"[PATCH 2/5] Documentation/urls: Rewrite to accomodate ::
","fromName":"Ramkumar Ramachandra","fromEmail":"artagnon@gmail.com","sentAt":"2010-04-18T00:54:18Z","receivedAt":"2010-04-18T00:54:18Z","isPatch":true,"sender":{"key":"r@artagnon.com","avatar":"https://avatars.githubusercontent.com/u/37226?v=4"},"body":"Rewrite the first part of the document to explicitly show differences\nbetween the URLs that can be used with different transport\nprotocols. Mention ::
format to explicitly invoke\na remote helper.\n\nSigned-off-by: Ramkumar Ramachandra \n---\n Documentation/urls.txt | 58 +++++++++++++++++++++++++++--------------------\n 1 files changed, 33 insertions(+), 25 deletions(-)\n\ndiff --git a/Documentation/urls.txt b/Documentation/urls.txt\nindex 459a394..a473da6 100644\n--- a/Documentation/urls.txt\n+++ b/Documentation/urls.txt\n@@ -1,44 +1,52 @@\n GIT URLS[[URLS]]\n ----------------\n \n-One of the following notations can be used\n-to name the remote repository:\n+In general, URLs contain information about the transport protocol, the\n+address of the remote server, and the path to the repository.\n+Depending on the transport protocol, some of this information may be\n+absent.\n+\n+Git natively supports ssh, git, rsync, http, https, ftp, and ftps\n+protocols. The following syntaxes may be used with them:\n \n-- rsync://host.xz/path/to/repo.git/\n-- http://host.xz{startsb}:port{endsb}/path/to/repo.git/\n-- https://host.xz{startsb}:port{endsb}/path/to/repo.git/\n-- git://host.xz{startsb}:port{endsb}/path/to/repo.git/\n-- git://host.xz{startsb}:port{endsb}/~user/path/to/repo.git/\n - ssh://{startsb}user@{endsb}host.xz{startsb}:port{endsb}/path/to/repo.git/\n-- ssh://{startsb}user@{endsb}host.xz/path/to/repo.git/\n-- ssh://{startsb}user@{endsb}host.xz/~user/path/to/repo.git/\n-- ssh://{startsb}user@{endsb}host.xz/~/path/to/repo.git\n+- git://host.xz{startsb}:port{endsb}/path/to/repo.git/\n+- rsync://host.xz/path/to/repo.git/\n+- http{startsb}s{endsb}://host.xz{startsb}:port{endsb}/path/to/repo.git/\n+- ftp{startsb}s{endsb}://host.xz{startsb}:port{endsb}/path/to/repo.git/\n \n-SSH is the default transport protocol over the network. You can\n-optionally specify which user to log-in as, and an alternate,\n-scp-like syntax is also supported. Both syntaxes support\n-username expansion, as does the native git protocol, but\n-only the former supports port specification. The following\n-three are identical to the last three above, respectively:\n+An alternative scp-like syntax may also be used with the ssh protocol:\n \n-- {startsb}user@{endsb}host.xz:/path/to/repo.git/\n-- {startsb}user@{endsb}host.xz:~user/path/to/repo.git/\n-- {startsb}user@{endsb}host.xz:path/to/repo.git\n+- {startsb}user@{endsb}host.xz:path/to/repo.git/\n \n-To sync with a local directory, you can use:\n+The ssh and git protocols additionally support ~username expansion:\n+\n+- ssh://{startsb}user@{endsb}host.xz{startsb}:port{endsb}/~{startsb}user{endsb}/path/to/repo.git/\n+- git://host.xz{startsb}:port{endsb}/~{startsb}user{endsb}/path/to/repo.git/\n+- {startsb}user@{endsb}host.xz:/~{startsb}user{endsb}/path/to/repo.git/\n+\n+For local respositories, also supported by git natively, the following\n+syntaxes may be used:\n \n - /path/to/repo.git/\n - file:///path/to/repo.git/\n \n ifndef::git-clone[]\n-They are mostly equivalent, except when cloning. See\n-linkgit:git-clone[1] for details.\n+These two syntaxes are mostly equivalent, except when cloning, when\n+the former implies --local option. See linkgit:git-clone[1] for\n+details.\n endif::git-clone[]\n \n-ifdef::git-clone[]\n-They are equivalent, except the former implies --local option.\n-endif::git-clone[]\n+When git doesn't know how to handle a certain transport protocol, it\n+attempts to use the 'remote-' remote helper, if one\n+exists. To explicitly request a remote helper, the following syntax\n+may be used:\n+\n+- ::
\n \n+where
may be a path, a server and path, or an arbitrary\n+URL-like string recognized by the specific remote helper being\n+invoked. See linkgit:git-remote-helpers[1] for details.\n \n If there are a large number of similarly-named remote repositories and\n you want to use a different format for them (such that the URLs you\n-- \n1.7.0.4\n"},{"id":"139806","messageId":"20100418025940.GA2249@progeny.tock","threadId":"23507","inReplyTo":"1271552047-sup-9523@kytes","subject":"Re: [PATCH 2/5] Documentation/urls: Rewrite to accomodate ::
","fromName":"Jonathan Nieder","fromEmail":"jrnieder@gmail.com","sentAt":"2010-04-18T02:59:41Z","receivedAt":"2010-04-18T02:59:41Z","isPatch":true,"sender":{"key":"jrnieder@gmail.com","avatar":"https://avatars.githubusercontent.com/u/281595?v=4"},"body":"Ramkumar Ramachandra wrote:\n\n> --- a/Documentation/urls.txt\n> +++ b/Documentation/urls.txt\n> @@ -1,44 +1,52 @@\n> GIT URLS[[URLS]]\n> ----------------\n> \n> -One of the following notations can be used\n> -to name the remote repository:\n> +In general, URLs contain information about the transport protocol, the\n> +address of the remote server, and the path to the repository.\n\nNice.\n\n> +Depending on the transport protocol, some of this information may be\n> +absent.\n> +\n> +Git natively supports ssh, git, rsync, http, https, ftp, and ftps\n> +protocols.\n\nHmm: if it is not built in to the git binary, is it right to call the\nsupport native? I don’t mean to say HTTP support is a second-class\ncitizen (with http-backend on the server side, it isn’t any more), but\nit is possible that remote-http is not installed on a system. Also, it\nis a good example to introduce remote helpers with.\n\nNot a criticism of this patch, just something to think about.\n\n> The following syntaxes may be used with them:\n> \n> -- rsync://host.xz/path/to/repo.git/\n> -- http://host.xz{startsb}:port{endsb}/path/to/repo.git/\n> -- https://host.xz{startsb}:port{endsb}/path/to/repo.git/\n> -- git://host.xz{startsb}:port{endsb}/path/to/repo.git/\n> -- git://host.xz{startsb}:port{endsb}/~user/path/to/repo.git/\n> - ssh://{startsb}user@{endsb}host.xz{startsb}:port{endsb}/path/to/repo.git/\n> -- ssh://{startsb}user@{endsb}host.xz/path/to/repo.git/\n> -- ssh://{startsb}user@{endsb}host.xz/~user/path/to/repo.git/\n> -- ssh://{startsb}user@{endsb}host.xz/~/path/to/repo.git\n> +- git://host.xz{startsb}:port{endsb}/path/to/repo.git/\n> +- rsync://host.xz/path/to/repo.git/\n> +- http{startsb}s{endsb}://host.xz{startsb}:port{endsb}/path/to/repo.git/\n> +- ftp{startsb}s{endsb}://host.xz{startsb}:port{endsb}/path/to/repo.git/\n\nTiny nitpick: I would mention http before rsync.\n\n[...]\n> - /path/to/repo.git/\n> - file:///path/to/repo.git/\n> \n> ifndef::git-clone[]\n> -They are mostly equivalent, except when cloning. See\n> -linkgit:git-clone[1] for details.\n> +These two syntaxes are mostly equivalent, except when cloning, when\n> +the former implies --local option. See linkgit:git-clone[1] for\n> +details.\n> endif::git-clone[]\n> \n> -ifdef::git-clone[]\n> -They are equivalent, except the former implies --local option.\n> -endif::git-clone[]\n\nWhat happened to this passage in git-clone.1?\n\n> +When git doesn't know how to handle a certain transport protocol, it\n> +attempts to use the 'remote-' remote helper, if one\n> +exists. To explicitly request a remote helper, the following syntax\n> +may be used:\n> +\n> +- ::
\n> \n> +where
may be a path, a server and path, or an arbitrary\n> +URL-like string recognized by the specific remote helper being\n> +invoked. See linkgit:git-remote-helpers[1] for details.\n\nExcept for the removal of the ifdef::git-clone[] section,\n\n Reviewed-by: Jonathan Nieder \n\nThanks.\nJonathan\n"},{"id":"139811","messageId":"i2gf3271551004172059sb3abbafcz2feb9eed14e596aa@mail.gmail.com","threadId":"23507","inReplyTo":"20100418025940.GA2249@progeny.tock","subject":"Re: [PATCH 2/5] Documentation/urls: Rewrite to accomodate ::
","fromName":"Ramkumar Ramachandra","fromEmail":"artagnon@gmail.com","sentAt":"2010-04-18T03:59:54Z","receivedAt":"2010-04-18T03:59:54Z","isPatch":true,"sender":{"key":"r@artagnon.com","avatar":"https://avatars.githubusercontent.com/u/37226?v=4"},"body":"Hi,\n\n> Hmm: if it is not built in to the git binary, is it right to call the\n> support native?  I don’t mean to say HTTP support is a second-class\n> citizen (with http-backend on the server side, it isn’t any more), but\n> it is possible that remote-http is not installed on a system.  Also, it\n> is a good example to introduce remote helpers with.\n\nBy \"native\", I meant that no special remote helpers are necessary (it\nwould be with svn, for example). Does the technical detail of whether\nthe support is built into the main Git binary need to enter the\ndocumentation?\n\nThanks for the review! :) I'll post a fixup patch soon.\n\n-- Ram\n"},{"id":"139813","messageId":"1271566767-sup-7167@kytes","threadId":"23507","inReplyTo":"20100418025940.GA2249@progeny.tock","subject":"Re: [PATCH 2/5] Documentation/urls: Rewrite to accomodate ::
","fromName":"Ramkumar Ramachandra","fromEmail":"artagnon@gmail.com","sentAt":"2010-04-18T05:03:16Z","receivedAt":"2010-04-18T05:03:16Z","isPatch":true,"sender":{"key":"r@artagnon.com","avatar":"https://avatars.githubusercontent.com/u/37226?v=4"},"body":"Here's the fixup.\n\ndiff --git a/Documentation/urls.txt b/Documentation/urls.txt\nindex a473da6..8f442ba 100644\n--- a/Documentation/urls.txt\n+++ b/Documentation/urls.txt\n@@ -11,8 +11,8 @@ protocols. The following syntaxes may be used with them:\n \n - ssh://{startsb}user@{endsb}host.xz{startsb}:port{endsb}/path/to/repo.git/\n - git://host.xz{startsb}:port{endsb}/path/to/repo.git/\n-- rsync://host.xz/path/to/repo.git/\n - http{startsb}s{endsb}://host.xz{startsb}:port{endsb}/path/to/repo.git/\n+- rsync://host.xz/path/to/repo.git/\n - ftp{startsb}s{endsb}://host.xz{startsb}:port{endsb}/path/to/repo.git/\n \n An alternative scp-like syntax may also be used with the ssh protocol:\n@@ -37,6 +37,11 @@ the former implies --local option. See linkgit:git-clone[1] for\n details.\n endif::git-clone[]\n \n+ifdef::git-clone[]\n+These two syntaxes are mostly equivalent, except the former implies\n+--local option.\n+endif::git-clone[]\n+\n When git doesn't know how to handle a certain transport protocol, it\n attempts to use the 'remote-' remote helper, if one\n exists. To explicitly request a remote helper, the following syntax\n"},{"id":"139833","messageId":"7vk4s4wqo5.fsf@alter.siamese.dyndns.org","threadId":"23507","inReplyTo":"1271566767-sup-7167@kytes","subject":"Re: [PATCH 2/5] Documentation/urls: Rewrite to accomodate ::
","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2010-04-18T18:39:54Z","receivedAt":"2010-04-18T18:39:54Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Thanks, both. Will squash into the second patch.\n\nExcept that I'll further move rsync:// lower so that the curl family can\nstay together, both in the list (that appeared in your \"fixup\") and in the\nintroductory text.\n"},{"id":"139836","messageId":"m2pfabb9a1e1004181144x10f6a0f1x2e55bc025aba781b@mail.gmail.com","threadId":"23507","inReplyTo":"7vk4s4wqo5.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH 2/5] Documentation/urls: Rewrite to accomodate ::
","fromName":"Sverre Rabbelier","fromEmail":"srabbelier@gmail.com","sentAt":"2010-04-18T18:44:13Z","receivedAt":"2010-04-18T18:44:13Z","isPatch":true,"sender":{"key":"srabbelier@gmail.com","avatar":"https://avatars.githubusercontent.com/u/3098?v=4"},"body":"Heya,\n\nOn Sun, Apr 18, 2010 at 20:39, Junio C Hamano wrote:\n> Except that I'll further move rsync:// lower so that the curl family can\n> stay together, both in the list (that appeared in your \"fixup\") and in the\n> introductory text.\n\nWith that I think the entire series can be:\n\nAcked-by: Sverre Rabbelier \n\n-- \nCheers,\n\nSverre Rabbelier\n"},{"id":"139847","messageId":"7vzl10v5mo.fsf@alter.siamese.dyndns.org","threadId":"23507","inReplyTo":"m2pfabb9a1e1004181144x10f6a0f1x2e55bc025aba781b@mail.gmail.com","subject":"Re: [PATCH 2/5] Documentation/urls: Rewrite to accomodate ::
","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2010-04-18T20:59:43Z","receivedAt":"2010-04-18T20:59:43Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Sverre Rabbelier writes:\n\n> With that I think the entire series can be:\n>\n> Acked-by: Sverre Rabbelier \n\nThanks. The end results look good to me, too.\n"},{"id":"139848","messageId":"w2gfabb9a1e1004181410ia09cc99o1ba807b53744f3ee@mail.gmail.com","threadId":"23507","inReplyTo":"7vzl10v5mo.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH 2/5] Documentation/urls: Rewrite to accomodate ::
","fromName":"Sverre Rabbelier","fromEmail":"srabbelier@gmail.com","sentAt":"2010-04-18T21:10:16Z","receivedAt":"2010-04-18T21:10:16Z","isPatch":true,"sender":{"key":"srabbelier@gmail.com","avatar":"https://avatars.githubusercontent.com/u/3098?v=4"},"body":"Heya,\n\nOn Sun, Apr 18, 2010 at 22:59, Junio C Hamano wrote:\n> Thanks.  The end results look good to me, too.\n\nHmm. Do we want to get this into 1.7.1? With how much debate this\nseries has had I don't see much added benefit from having it cooking\nin next. I noticed another doc patch got merged into master in the\nlatest cooking, so I'm sort-of assuming there's special rules for\ndocumentation, please correct me if that's nonsense.\n\n-- \nCheers,\n\nSverre Rabbelier\n"},{"id":"139852","messageId":"7vvdbov2zc.fsf@alter.siamese.dyndns.org","threadId":"23507","inReplyTo":"w2gfabb9a1e1004181410ia09cc99o1ba807b53744f3ee@mail.gmail.com","subject":"Re: [PATCH 2/5] Documentation/urls: Rewrite to accomodate ::
","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2010-04-18T21:56:55Z","receivedAt":"2010-04-18T21:56:55Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Sverre Rabbelier writes:\n\n> On Sun, Apr 18, 2010 at 22:59, Junio C Hamano wrote:\n>> Thanks. The end results look good to me, too.\n>\n> Hmm. Do we want to get this into 1.7.1? With how much debate this\n> series has had I don't see much added benefit from having it cooking\n> in next.\n\nI don't think this is that _urgent_, but on the other hand I don't think\nthis is with heavy-enough impact that needs to be out of the coming\nrelease either. As long as there aren't any glaring factual errors or\nformatting breakages, I think it is safe to merge it and get it over\nwith ;-).\n"}]}