{"thread":{"id":"4883","subject":"[PATCH] Revert \"Documentation: Fix ssh://[user@]host.xz URL\"","startedAt":"2006-07-14T10:24:46Z","lastAt":"2006-07-15T07:44:25Z","messageCount":12,"participants":["Alp Toker","Junio C Hamano","Jonas Fonseca"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"23810","messageId":"11528726881431-git-send-email-alp@atoker.com","threadId":"4883","inReplyTo":null,"subject":"[PATCH] Revert \"Documentation: Fix ssh://[user@]host.xz URL\"","fromName":"Alp Toker","fromEmail":"alp@atoker.com","sentAt":"2006-07-14T10:24:46Z","receivedAt":"2006-07-14T10:24:46Z","isPatch":true,"sender":{"key":"alp@atoker.com","avatar":"https://gravatar.com/avatar/5367b73f1bb2598314e084847e4f45c303514c7fd192cfa9aa3c3437511567cd?d=mp&s=160"},"body":"This reverts commit 8d5e26848d0ddd5063f078b0c34c6c48b68d0d94.\n\nThe patch did not eliminate warnings caused by bad asciidoc markup.\nNor did it correct the generated output, which remains unchanged.\n\nSigned-off-by: Alp Toker <alp@atoker.com>\n---\n Documentation/urls.txt |    6 +++---\n 1 files changed, 3 insertions(+), 3 deletions(-)\n\ndiff --git a/Documentation/urls.txt b/Documentation/urls.txt\nindex 93378d2..9abec80 100644\n--- a/Documentation/urls.txt\n+++ b/Documentation/urls.txt\n@@ -10,9 +10,9 @@ to name the remote repository:\n - https://host.xz/path/to/repo.git/\n - git://host.xz/path/to/repo.git/\n - git://host.xz/~user/path/to/repo.git/\n-- ssh://+++[user@+++]host.xz/path/to/repo.git/\n-- ssh://+++[user@+++]host.xz/~user/path/to/repo.git/\n-- ssh://+++[user@+++]host.xz/~/path/to/repo.git\n+- ssh://[user@]host.xz/path/to/repo.git/\n+- ssh://[user@]host.xz/~user/path/to/repo.git/\n+- ssh://[user@]host.xz/~/path/to/repo.git\n ===============================================================\n \n SSH Is the default transport protocol and also supports an\n-- \n1.4.1.g28ec\n"},{"id":"23811","messageId":"1152872688966-git-send-email-alp@atoker.com","threadId":"4883","inReplyTo":"11528726881431-git-send-email-alp@atoker.com","subject":"[PATCH] Documentation: Fix ssh:// URLs in generated documentation","fromName":"Alp Toker","fromEmail":"alp@atoker.com","sentAt":"2006-07-14T10:24:47Z","receivedAt":"2006-07-14T10:24:47Z","isPatch":true,"sender":{"key":"alp@atoker.com","avatar":"https://gravatar.com/avatar/5367b73f1bb2598314e084847e4f45c303514c7fd192cfa9aa3c3437511567cd?d=mp&s=160"},"body":"Commit c3f17061be95de3498449a548e93883aebff23d6 was causing warnings\nduring doc generation due to bad asciidoc markup.\n\nThis resulted in \"ssh://[user@]host.xz/path/to/repo.git/\" being rendered\nas \"host.xz/path/to/repo.git/\" in the man pages and html output.\n\nThis patch converts sections listing URL formats to verbatim.\n\nSigned-off-by: Alp Toker <alp@atoker.com>\n---\n Documentation/urls.txt |   30 ++++++++++++------------------\n 1 files changed, 12 insertions(+), 18 deletions(-)\n\ndiff --git a/Documentation/urls.txt b/Documentation/urls.txt\nindex 9abec80..a071915 100644\n--- a/Documentation/urls.txt\n+++ b/Documentation/urls.txt\n@@ -4,33 +4,27 @@ GIT URLS[[URLS]]\n One of the following notations can be used\n to name the remote repository:\n \n-===============================================================\n-- rsync://host.xz/path/to/repo.git/\n-- http://host.xz/path/to/repo.git/\n-- https://host.xz/path/to/repo.git/\n-- git://host.xz/path/to/repo.git/\n-- git://host.xz/~user/path/to/repo.git/\n-- ssh://[user@]host.xz/path/to/repo.git/\n-- ssh://[user@]host.xz/~user/path/to/repo.git/\n-- ssh://[user@]host.xz/~/path/to/repo.git\n-===============================================================\n+\trsync://host.xz/path/to/repo.git/\n+\thttp://host.xz/path/to/repo.git/\n+\thttps://host.xz/path/to/repo.git/\n+\tgit://host.xz/path/to/repo.git/\n+\tgit://host.xz/~user/path/to/repo.git/\n+\tssh://[user@]host.xz/path/to/repo.git/\n+\tssh://[user@]host.xz/~user/path/to/repo.git/\n+\tssh://[user@]host.xz/~/path/to/repo.git\n \n SSH Is the default transport protocol and also supports an\n scp-like syntax.  Both syntaxes support username expansion,\n as does the native git protocol. The following three are\n identical to the last three above, respectively:\n \n-===============================================================\n-- host.xz:/path/to/repo.git/\n-- host.xz:~user/path/to/repo.git/\n-- host.xz:path/to/repo.git\n-===============================================================\n+\thost.xz:/path/to/repo.git/\n+\thost.xz:~user/path/to/repo.git/\n+\thost.xz:path/to/repo.git\n \n To sync with a local directory, use:\n \n-===============================================================\n-- /path/to/repo.git/\n-===============================================================\n+\t/path/to/repo.git/\n \n REMOTES\n -------\n-- \n1.4.1.g28ec\n"},{"id":"23812","messageId":"11528726882895-git-send-email-alp@atoker.com","threadId":"4883","inReplyTo":"1152872688966-git-send-email-alp@atoker.com","subject":"[PATCH] Documentation: Minor capitalization typofix","fromName":"Alp Toker","fromEmail":"alp@atoker.com","sentAt":"2006-07-14T10:24:48Z","receivedAt":"2006-07-14T10:24:48Z","isPatch":true,"sender":{"key":"alp@atoker.com","avatar":"https://gravatar.com/avatar/5367b73f1bb2598314e084847e4f45c303514c7fd192cfa9aa3c3437511567cd?d=mp&s=160"},"body":"This ended up appearing in several man pages by inclusion.\n\nSigned-off-by: Alp Toker <alp@atoker.com>\n---\n Documentation/urls.txt |    2 +-\n 1 files changed, 1 insertions(+), 1 deletions(-)\n\ndiff --git a/Documentation/urls.txt b/Documentation/urls.txt\nindex a071915..2b5a7f8 100644\n--- a/Documentation/urls.txt\n+++ b/Documentation/urls.txt\n@@ -13,7 +13,7 @@ to name the remote repository:\n \tssh://[user@]host.xz/~user/path/to/repo.git/\n \tssh://[user@]host.xz/~/path/to/repo.git\n \n-SSH Is the default transport protocol and also supports an\n+SSH is the default transport protocol and also supports an\n scp-like syntax.  Both syntaxes support username expansion,\n as does the native git protocol. The following three are\n identical to the last three above, respectively:\n-- \n1.4.1.g28ec\n"},{"id":"23838","messageId":"7vk66foo5u.fsf@assigned-by-dhcp.cox.net","threadId":"4883","inReplyTo":"1152872688966-git-send-email-alp@atoker.com","subject":"Re: [PATCH] Documentation: Fix ssh:// URLs in generated documentation","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2006-07-14T21:46:21Z","receivedAt":"2006-07-14T21:46:21Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Alp Toker <alp@atoker.com> writes:\n\n> Commit c3f17061be95de3498449a548e93883aebff23d6 was causing warnings\n> during doc generation due to bad asciidoc markup.\n\nIt seems that the '+++' quoting does not seem to work with older asciidoc.\n\nHowever, I just noticed that what the text talks about is wrong\nto begin with (not your fault):\n\n> +\trsync://host.xz/path/to/repo.git/\n> +\thttp://host.xz/path/to/repo.git/\n> +\thttps://host.xz/path/to/repo.git/\n> +\tgit://host.xz/path/to/repo.git/\n> +\tgit://host.xz/~user/path/to/repo.git/\n> +\tssh://[user@]host.xz/path/to/repo.git/\n> +\tssh://[user@]host.xz/~user/path/to/repo.git/\n> +\tssh://[user@]host.xz/~/path/to/repo.git\n>  \n>  SSH Is the default transport protocol and also supports an\n>  scp-like syntax.  Both syntaxes support username expansion,\n>  as does the native git protocol. The following three are\n>  identical to the last three above, respectively:\n>  \n> +\thost.xz:/path/to/repo.git/\n> +\thost.xz:~user/path/to/repo.git/\n> +\thost.xz:path/to/repo.git\n\nSo how about something like this instead, until people have up\nto date asciidoc?\n\ndiff --git a/Documentation/urls.txt b/Documentation/urls.txt\nindex 93378d2..a5519e5 100644\n--- a/Documentation/urls.txt\n+++ b/Documentation/urls.txt\n@@ -10,12 +10,12 @@ to name the remote repository:\n - https://host.xz/path/to/repo.git/\n - git://host.xz/path/to/repo.git/\n - git://host.xz/~user/path/to/repo.git/\n-- ssh://+++[user@+++]host.xz/path/to/repo.git/\n-- ssh://+++[user@+++]host.xz/~user/path/to/repo.git/\n-- ssh://+++[user@+++]host.xz/~/path/to/repo.git\n+- ssh://host.xz/path/to/repo.git/\n+- ssh://host.xz/~user/path/to/repo.git/\n+- ssh://host.xz/~/path/to/repo.git\n ===============================================================\n \n-SSH Is the default transport protocol and also supports an\n+SSH is the default transport protocol and also supports an\n scp-like syntax.  Both syntaxes support username expansion,\n as does the native git protocol. The following three are\n identical to the last three above, respectively:\n@@ -26,6 +26,10 @@ identical to the last three above, respe\n - host.xz:path/to/repo.git\n ===============================================================\n \n+In addition, you can specify the name of the user to log in as\n+by saying `ssh://user@host.xz/\\...` and `user@host.xz:\\...` for\n+the SSH protocol.\n+\n To sync with a local directory, use:\n \n ===============================================================\n"},{"id":"23839","messageId":"20060714215039.GA21994@diku.dk","threadId":"4883","inReplyTo":"11528726881431-git-send-email-alp@atoker.com","subject":"[PATCH] urls.txt: Use substitution to escape square brackets","fromName":"Jonas Fonseca","fromEmail":"fonseca@diku.dk","sentAt":"2006-07-14T21:50:39Z","receivedAt":"2006-07-14T21:50:39Z","isPatch":true,"sender":{"key":"fonseca@diku.dk","avatar":"https://gravatar.com/avatar/f82f3ad698717c51873b020c750a92438c820a24056dc39fe4d07baa10a92264?d=mp&s=160"},"body":"This changes \"[user@]\" to use {startsb} and {endsb} to insert [ and ],\nsimilar to how {caret} is used in git-rev-parse.txt.\n\nSigned-off-by: Jonas Fonseca <fonseca@diku.dk>\n\n---\n\nAlternate \"fix\".\n\n---\n\ndiff --git a/Documentation/asciidoc.conf b/Documentation/asciidoc.conf\nindex 7ce7151..41754d5 100644\n--- a/Documentation/asciidoc.conf\n+++ b/Documentation/asciidoc.conf\n@@ -9,6 +9,8 @@ # the command.\n \n [attributes]\n caret=^\n+startsb=&#91; # [\n+endsb=&#93;   # ]\n \n ifdef::backend-docbook[]\n [gitlink-inlinemacro]\ndiff --git a/Documentation/urls.txt b/Documentation/urls.txt\nindex 93378d2..2950ac9 100644\n--- a/Documentation/urls.txt\n+++ b/Documentation/urls.txt\n@@ -10,9 +10,9 @@ to name the remote repository:\n - https://host.xz/path/to/repo.git/\n - git://host.xz/path/to/repo.git/\n - git://host.xz/~user/path/to/repo.git/\n-- ssh://+++[user@+++]host.xz/path/to/repo.git/\n-- ssh://+++[user@+++]host.xz/~user/path/to/repo.git/\n-- ssh://+++[user@+++]host.xz/~/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 ===============================================================\n \n SSH Is the default transport protocol and also supports an\n\n-- \nJonas Fonseca\n"},{"id":"23840","messageId":"7vfyh3on6w.fsf@assigned-by-dhcp.cox.net","threadId":"4883","inReplyTo":"20060714215039.GA21994@diku.dk","subject":"Re: [PATCH] urls.txt: Use substitution to escape square brackets","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2006-07-14T22:07:19Z","receivedAt":"2006-07-14T22:07:19Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jonas Fonseca <fonseca@diku.dk> writes:\n\n> This changes \"[user@]\" to use {startsb} and {endsb} to insert [ and ],\n> similar to how {caret} is used in git-rev-parse.txt.\n>\n> Signed-off-by: Jonas Fonseca <fonseca@diku.dk>\n\nLooks nicer.  Alp?  Does this work with your version of\nasciidoc?\n"},{"id":"23841","messageId":"44B82243.20604@atoker.com","threadId":"4883","inReplyTo":"7vfyh3on6w.fsf@assigned-by-dhcp.cox.net","subject":"Re: [PATCH] urls.txt: Use substitution to escape square brackets","fromName":"Alp Toker","fromEmail":"alp@atoker.com","sentAt":"2006-07-14T23:01:23Z","receivedAt":"2006-07-14T23:01:23Z","isPatch":true,"sender":{"key":"alp@atoker.com","avatar":"https://gravatar.com/avatar/5367b73f1bb2598314e084847e4f45c303514c7fd192cfa9aa3c3437511567cd?d=mp&s=160"},"body":"Junio C Hamano wrote:\n> Jonas Fonseca <fonseca@diku.dk> writes:\n> \n>> This changes \"[user@]\" to use {startsb} and {endsb} to insert [ and ],\n>> similar to how {caret} is used in git-rev-parse.txt.\n>>\n>> Signed-off-by: Jonas Fonseca <fonseca@diku.dk>\n> \n> Looks nicer.  Alp?  Does this work with your version of\n> asciidoc?\n\nI'm using asciidoc 7.0.2, which came with Ubuntu Dapper.\n\nThe man page output now looks like this:\n\n        ·  ssh://[ # [user@] # ]host.xz/path/to/repo.git/\n        ·  ssh://[ # [user@] # ]host.xz/~user/path/to/repo.git/\n        ·  ssh://[ # [user@] # ]host.xz/~/path/to/repo.git\n\nStill not correct.\n\nOut of the three patches,\n\n   (at) \"ssh://[user@]host.xz/path/to/repo.git/\"\n   (jc) \"ssh://+++[user@+++]host.xz/path/to/repo.git/\"\n   (jf) \"ssh://{startsb}user@{endsb}host.xz/path/to/repo.git/\"\n\nI'd say (at) is still most readable, not requiring +++ markup (jc) or \nhacks to asciidoc.conf (jf). And it's the only fix that's compatible \nwith older versions.\n\nThat's unless I've missed out on some detail that makes (at) unworkable. \n  If it's the dozen or so modified lines that are a problem, we can work \non a wdiff merge strategy or something in the time we save looking for \nthe asciidoc escape sequence that works for everyone. Junio?\n"},{"id":"23842","messageId":"7v8xmvok49.fsf@assigned-by-dhcp.cox.net","threadId":"4883","inReplyTo":"44B82243.20604@atoker.com","subject":"Re: [PATCH] urls.txt: Use substitution to escape square brackets","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2006-07-14T23:13:42Z","receivedAt":"2006-07-14T23:13:42Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Alp Toker <alp@atoker.com> writes:\n\n> Junio C Hamano wrote:\n>>\n>> Looks nicer.  Alp?  Does this work with your version of\n>> asciidoc?\n>\n> I'm using asciidoc 7.0.2, which came with Ubuntu Dapper.\n>\n> The man page output now looks like this:\n>\n>        ·  ssh://[ # [user@] # ]host.xz/path/to/repo.git/\n>        ·  ssh://[ # [user@] # ]host.xz/~user/path/to/repo.git/\n>        ·  ssh://[ # [user@] # ]host.xz/~/path/to/repo.git\n\nI got a bit inventive and removed the comments after #, which\nwere in Jonas's version, when I tried it out.  Maybe doing the\nsame might help you, since I suspect the above '#' are coming\nfrom the comment part.\n\n> Out of the three patches,\n>\n>   (at) \"ssh://[user@]host.xz/path/to/repo.git/\"\n>   (jc) \"ssh://+++[user@+++]host.xz/path/to/repo.git/\"\n>   (jf) \"ssh://{startsb}user@{endsb}host.xz/path/to/repo.git/\"\n>\n> I'd say (at) is still most readable, not requiring +++ markup (jc) or\n> hacks to asciidoc.conf (jf).\n\nI do not think defining [attribute] is a hack; it is a\ndocumented feature.\n"},{"id":"23844","messageId":"7v3bd3ois4.fsf@assigned-by-dhcp.cox.net","threadId":"4883","inReplyTo":"20060714215039.GA21994@diku.dk","subject":"[PATCH] urls.txt: Use substitution to escape square brackets","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2006-07-14T23:42:35Z","receivedAt":"2006-07-14T23:42:35Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Alp is right -- the comments in [attributes] section after the\ndefinition do appear in the output, even with asciidoc 7.1.2, so\nhere is a replacement proposal from me.\n\nThe one by Alp is an easy work-around, but I do not want to\nworry about potential, unintended, breakages that might be\ncaused by changing delimited blocks to literal paragraphs (and\nit changes the resulting rendering from the original).\n\n-- >8 --\nFrom: Jonas Fonseca <fonseca@diku.dk>\n\nThis changes \"[user@]\" to use {startsb} and {endsb} to insert [ and ],\nsimilar to how {caret} is used in git-rev-parse.txt.\n\n[jc: Removed a well-intentioned comment that broke the final\n formatting from the original patch.  While we are at it,\n updated the paragraph that claims to be equivalent to the\n section that was updated earlier without making matching\n changes.]\n\nSigned-off-by: Jonas Fonseca <fonseca@diku.dk>\nSigned-off-by: Junio C Hamano <junkio@cox.net>\n---\n Documentation/asciidoc.conf |    2 ++\n Documentation/urls.txt      |   17 +++++++++--------\n 2 files changed, 11 insertions(+), 8 deletions(-)\n\ndiff --git a/Documentation/asciidoc.conf b/Documentation/asciidoc.conf\nindex 7ce7151..8196d78 100644\n--- a/Documentation/asciidoc.conf\n+++ b/Documentation/asciidoc.conf\n@@ -9,6 +9,8 @@ # the command.\n \n [attributes]\n caret=^\n+startsb=&#91;\n+endsb=&#93;\n \n ifdef::backend-docbook[]\n [gitlink-inlinemacro]\ndiff --git a/Documentation/urls.txt b/Documentation/urls.txt\nindex 9d2ad46..26ecba5 100644\n--- a/Documentation/urls.txt\n+++ b/Documentation/urls.txt\n@@ -10,20 +10,21 @@ to name the remote repository:\n - https://host.xz/path/to/repo.git/\n - git://host.xz/path/to/repo.git/\n - git://host.xz/~user/path/to/repo.git/\n-- ssh://+++[user@+++]host.xz/path/to/repo.git/\n-- ssh://+++[user@+++]host.xz/~user/path/to/repo.git/\n-- ssh://+++[user@+++]host.xz/~/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 ===============================================================\n \n-SSH Is the default transport protocol and also supports an\n-scp-like syntax.  Both syntaxes support username expansion,\n+SSH is the default transport protocol.  You can optionally specify\n+which user to log-in as, and an alternate, scp-like syntax is also\n+supported.  Both syntaxes support username expansion,\n as does the native git protocol. The following three are\n identical to the last three above, respectively:\n \n ===============================================================\n-- host.xz:/path/to/repo.git/\n-- host.xz:~user/path/to/repo.git/\n-- host.xz:path/to/repo.git\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 ===============================================================\n \n To sync with a local directory, use:\n-- \n1.4.1.g28ec\n"},{"id":"23845","messageId":"44B82F50.7050106@atoker.com","threadId":"4883","inReplyTo":"7v8xmvok49.fsf@assigned-by-dhcp.cox.net","subject":"Re: [PATCH] urls.txt: Use substitution to escape square brackets","fromName":"Alp Toker","fromEmail":"alp@atoker.com","sentAt":"2006-07-14T23:57:04Z","receivedAt":"2006-07-14T23:57:04Z","isPatch":true,"sender":{"key":"alp@atoker.com","avatar":"https://gravatar.com/avatar/5367b73f1bb2598314e084847e4f45c303514c7fd192cfa9aa3c3437511567cd?d=mp&s=160"},"body":"Junio C Hamano wrote:\n> Alp Toker <alp@atoker.com> writes:\n> \n>> Junio C Hamano wrote:\n>>> Looks nicer.  Alp?  Does this work with your version of\n>>> asciidoc?\n>> I'm using asciidoc 7.0.2, which came with Ubuntu Dapper.\n>>\n>> The man page output now looks like this:\n>>\n>>        ·  ssh://[ # [user@] # ]host.xz/path/to/repo.git/\n>>        ·  ssh://[ # [user@] # ]host.xz/~user/path/to/repo.git/\n>>        ·  ssh://[ # [user@] # ]host.xz/~/path/to/repo.git\n> \n> I got a bit inventive and removed the comments after #, which\n> were in Jonas's version, when I tried it out.  Maybe doing the\n> same might help you, since I suspect the above '#' are coming\n> from the comment part.\n\nNo. Removing the comments doesn't help -- the output remains broken.\n\n>> Out of the three patches,\n>>\n>>   (at) \"ssh://[user@]host.xz/path/to/repo.git/\"\n>>   (jc) \"ssh://+++[user@+++]host.xz/path/to/repo.git/\"\n>>   (jf) \"ssh://{startsb}user@{endsb}host.xz/path/to/repo.git/\"\n>>\n>> I'd say (at) is still most readable, not requiring +++ markup (jc) or\n>> hacks to asciidoc.conf (jf).\n> \n> I do not think defining [attribute] is a hack; it is a\n> documented feature.\n\n\"You write an AsciiDoc document the same way you would write a normal \ntext document, there are no markup tags or weird format notations.\"\n\nasciidoc solves this problem by design and without markup, with the \nprescribed syntax being to indent and quote the text verbatim, just as \nit was meant to be read.\n\nWhy try to fit a square peg in a round hole by defining attributes? It's \na good indicator that there's a hack going down when three competent \nprogrammers can't escape a square bracket after three attempts spanning \n  two days ;-)\n"},{"id":"23846","messageId":"44B838EE.5020609@atoker.com","threadId":"4883","inReplyTo":"7v3bd3ois4.fsf@assigned-by-dhcp.cox.net","subject":"Re: [PATCH] urls.txt: Use substitution to escape square brackets","fromName":"Alp Toker","fromEmail":"alp@atoker.com","sentAt":"2006-07-15T00:38:06Z","receivedAt":"2006-07-15T00:38:06Z","isPatch":true,"sender":{"key":"alp@atoker.com","avatar":"https://gravatar.com/avatar/5367b73f1bb2598314e084847e4f45c303514c7fd192cfa9aa3c3437511567cd?d=mp&s=160"},"body":"Junio C Hamano wrote:\n> Alp is right -- the comments in [attributes] section after the\n> definition do appear in the output, even with asciidoc 7.1.2, so\n> here is a replacement proposal from me.\n> \n> The one by Alp is an easy work-around, but I do not want to\n> worry about potential, unintended, breakages that might be\n> caused by changing delimited blocks to literal paragraphs (and\n> it changes the resulting rendering from the original).\n> \n> -- >8 --\n> From: Jonas Fonseca <fonseca@diku.dk>\n> \n> This changes \"[user@]\" to use {startsb} and {endsb} to insert [ and ],\n> similar to how {caret} is used in git-rev-parse.txt.\n> \n> [jc: Removed a well-intentioned comment that broke the final\n>  formatting from the original patch.  While we are at it,\n>  updated the paragraph that claims to be equivalent to the\n>  section that was updated earlier without making matching\n>  changes.]\n> \n> Signed-off-by: Jonas Fonseca <fonseca@diku.dk>\n> Signed-off-by: Junio C Hamano <junkio@cox.net>\n\nSigned-off-by: Alp Toker <alp@atoker.com>\n"},{"id":"23853","messageId":"20060715074425.GA2661@diku.dk","threadId":"4883","inReplyTo":"7v3bd3ois4.fsf@assigned-by-dhcp.cox.net","subject":"[PATCH] Make asciidoc related rules depend on asciidoc.conf","fromName":"Jonas Fonseca","fromEmail":"fonseca@diku.dk","sentAt":"2006-07-15T07:44:25Z","receivedAt":"2006-07-15T07:44:25Z","isPatch":true,"sender":{"key":"fonseca@diku.dk","avatar":"https://gravatar.com/avatar/f82f3ad698717c51873b020c750a92438c820a24056dc39fe4d07baa10a92264?d=mp&s=160"},"body":"Any rule running asciidoc, whether using -f asciidoc.conf or not,\nwill read asciidoc.conf. Make such rules depend on asciidoc.conf.\n\nSigned-off-by: Jonas Fonseca <fonseca@diku.dk>\n---\n\nJunio C Hamano <junkio@cox.net> wrote Fri, Jul 14, 2006:\n> Alp is right -- the comments in [attributes] section after the\n> definition do appear in the output, even with asciidoc 7.1.2, so\n> here is a replacement proposal from me.\n\nAm using the same setup as Alp (AsciiDoc 7.0.2 on Ubuntu Dapper), so I\nwas a bit curious that I could not see that. So I guess I compiled the\ndocs first and later decided to add the comments which then didn't cause\na recompilation.\n\nI don't know if there should be a dependency on asciidoc.conf in every\nrule calling asciidoc. Could be annoying if you are only making a minor\nchange.\n\n---\n\ndiff --git a/Documentation/Makefile b/Documentation/Makefile\nindex cc83610..ce6c6cf 100644\n--- a/Documentation/Makefile\n+++ b/Documentation/Makefile\n@@ -76,18 +76,18 @@ README: ../README\n clean:\n \trm -f *.xml *.html *.1 *.7 howto-index.txt howto/*.html doc.dep README\n \n-%.html : %.txt\n+%.html : %.txt asciidoc.conf\n \tasciidoc -b xhtml11 -d manpage -f asciidoc.conf $<\n \n %.1 %.7 : %.xml\n \txmlto -m callouts.xsl man $<\n \n-%.xml : %.txt\n+%.xml : %.txt asciidoc.conf\n \tasciidoc -b docbook -d manpage -f asciidoc.conf $<\n \n git.html: git.txt README\n \n-glossary.html : glossary.txt sort_glossary.pl\n+glossary.html : glossary.txt sort_glossary.pl asciidoc.conf\n \tcat $< | \\\n \tperl sort_glossary.pl | \\\n \tasciidoc -b xhtml11 - > glossary.html\n@@ -97,12 +97,12 @@ howto-index.txt: howto-index.sh $(wildca\n \tsh ./howto-index.sh $(wildcard howto/*.txt) >$@+\n \tmv $@+ $@\n \n-$(patsubst %,%.html,$(ARTICLES)) : %.html : %.txt\n+$(patsubst %,%.html,$(ARTICLES)) : %.html : %.txt asciidoc.conf\n \tasciidoc -b xhtml11 $*.txt\n \n WEBDOC_DEST = /pub/software/scm/git/docs\n \n-$(patsubst %.txt,%.html,$(wildcard howto/*.txt)): %.html : %.txt\n+$(patsubst %.txt,%.html,$(wildcard howto/*.txt)): %.html : %.txt asciidoc.conf\n \trm -f $@+ $@\n \tsed -e '1,/^$$/d' $? | asciidoc -b xhtml11 - >$@+\n \tmv $@+ $@\n-- \nJonas Fonseca\n"}]}