{"thread":{"id":"15724","subject":"[PATCH] Documentation: remove '\\' from front of options in githooks doc","startedAt":"2008-09-30T13:37:11Z","lastAt":"2008-09-30T17:27:10Z","messageCount":4,"participants":["SZEDER Gábor","Thomas Rast"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"91947","messageId":"1222781831-6620-1-git-send-email-szeder@ira.uka.de","threadId":"15724","inReplyTo":null,"subject":"[PATCH] Documentation: remove '\\' from front of options in githooks doc","fromName":"SZEDER Gábor","fromEmail":"szeder@ira.uka.de","sentAt":"2008-09-30T13:37:11Z","receivedAt":"2008-09-30T13:37:11Z","isPatch":true,"sender":{"key":"szeder.dev@gmail.com","avatar":"https://avatars.githubusercontent.com/u/116324?v=4"},"body":"... because they show up in the man and html outputs.\n\nSigned-off-by: SZEDER Gábor <szeder@ira.uka.de>\n---\n Documentation/githooks.txt |   12 ++++++------\n 1 files changed, 6 insertions(+), 6 deletions(-)\n\ndiff --git a/Documentation/githooks.txt b/Documentation/githooks.txt\nindex 046a2a7..5bb7b33 100644\n--- a/Documentation/githooks.txt\n+++ b/Documentation/githooks.txt\n@@ -66,7 +66,7 @@ pre-commit\n ----------\n \n This hook is invoked by 'git-commit', and can be bypassed\n-with `\\--no-verify` option.  It takes no parameter, and is\n+with `--no-verify` option.  It takes no parameter, and is\n invoked before obtaining the proposed commit log message and\n making a commit.  Exiting with non-zero status from this script\n causes the 'git-commit' to abort.\n@@ -87,17 +87,17 @@ default log message, and before the editor is started.\n \n It takes one to three parameters.  The first is the name of the file\n that the commit log message.  The second is the source of the commit\n-message, and can be: `message` (if a `\\-m` or `\\-F` option was\n-given); `template` (if a `\\-t` option was given or the\n+message, and can be: `message` (if a `-m` or `-F` option was\n+given); `template` (if a `-t` option was given or the\n configuration option `commit.template` is set); `merge` (if the\n commit is a merge or a `.git/MERGE_MSG` file exists); `squash`\n (if a `.git/SQUASH_MSG` file exists); or `commit`, followed by\n-a commit SHA1 (if a `\\-c`, `\\-C` or `\\--amend` option was given).\n+a commit SHA1 (if a `-c`, `-C` or `--amend` option was given).\n \n If the exit status is non-zero, 'git-commit' will abort.\n \n The purpose of the hook is to edit the message file in place, and\n-it is not suppressed by the `\\--no-verify` option.  A non-zero exit\n+it is not suppressed by the `--no-verify` option.  A non-zero exit\n means a failure of the hook and aborts the commit.  It should not\n be used as replacement for pre-commit hook.\n \n@@ -108,7 +108,7 @@ commit-msg\n ----------\n \n This hook is invoked by 'git-commit', and can be bypassed\n-with `\\--no-verify` option.  It takes a single parameter, the\n+with `--no-verify` option.  It takes a single parameter, the\n name of the file that holds the proposed commit log message.\n Exiting with non-zero status causes the 'git-commit' to\n abort.\n-- \n1.6.0.2.448.g0c4e5\n"},{"id":"91957","messageId":"200809301756.41518.trast@student.ethz.ch","threadId":"15724","inReplyTo":"1222781831-6620-1-git-send-email-szeder@ira.uka.de","subject":"Re: [PATCH] Documentation: remove '\\' from front of options in githooks doc","fromName":"Thomas Rast","fromEmail":"trast@student.ethz.ch","sentAt":"2008-09-30T15:56:32Z","receivedAt":"2008-09-30T15:56:32Z","isPatch":true,"sender":{"key":"tr@thomasrast.ch","avatar":"https://avatars.githubusercontent.com/u/153510?v=4"},"body":"SZEDER Gábor wrote:\n> ... because they show up in the man and html outputs.\n...\n> diff --git a/Documentation/githooks.txt b/Documentation/githooks.txt\n...\n> @@ -66,7 +66,7 @@ pre-commit\n>  ----------\n>  \n>  This hook is invoked by 'git-commit', and can be bypassed\n> -with `\\--no-verify` option.  It takes no parameter, and is\n> +with `--no-verify` option.  It takes no parameter, and is\n>  invoked before obtaining the proposed commit log message and\n>  making a commit.  Exiting with non-zero status from this script\n>  causes the 'git-commit' to abort.\n\nThis must be checked against older asciidocs to verify that it does\nnot turn the unescaped sequence -- into an em-dash.  The \\-- traces\nback to e1ccf53 ([PATCH] Escape asciidoc's built-in em-dash\nreplacement, 2005-09-12), which indicates that it _was_ a problem in\nthe past.  (The asciidoc 8.2.5 on my system does not use an em-dash in\neither case.)\n\nACK on all single-dash cases though, which are indeed wrong and were\napparently introduced later in 8089c85 (git-commit: add a\nprepare-commit-msg hook, 2008-02-05).\n\n- Thomas\n\n-- \nThomas Rast\ntrast@student.ethz.ch\n\n\n"},{"id":"91961","messageId":"20080930172155.GJ6941@neumann","threadId":"15724","inReplyTo":"200809301756.41518.trast@student.ethz.ch","subject":"Re: [PATCH] Documentation: remove '\\' from front of options in githooks doc","fromName":"SZEDER Gábor","fromEmail":"szeder@ira.uka.de","sentAt":"2008-09-30T17:21:55Z","receivedAt":"2008-09-30T17:21:55Z","isPatch":true,"sender":{"key":"szeder.dev@gmail.com","avatar":"https://avatars.githubusercontent.com/u/116324?v=4"},"body":"Hi Thomas,\n\nOn Tue, Sep 30, 2008 at 05:56:32PM +0200, Thomas Rast wrote:\n> This must be checked against older asciidocs to verify that it does\n> not turn the unescaped sequence -- into an em-dash.  The \\-- traces\n> back to e1ccf53 ([PATCH] Escape asciidoc's built-in em-dash\n> replacement, 2005-09-12), which indicates that it _was_ a problem in\n> the past.  (The asciidoc 8.2.5 on my system does not use an em-dash in\n> either case.)\nIndeed, now that I look more closely, I had those '\\' characters only\nin front of short options in the outputs.\n\nHowever, it worth noting that there are plenty other places where\nthe double dash in long options are not escaped with a '\\'.\n\n> ACK on all single-dash cases though, which are indeed wrong and were\n> apparently introduced later in 8089c85 (git-commit: add a\n> prepare-commit-msg hook, 2008-02-05).\nI will send a corrected patch shortly.\n\nThanks,\nGábor\n"},{"id":"91962","messageId":"20080930172710.GK6941@neumann","threadId":"15724","inReplyTo":"20080930172155.GJ6941@neumann","subject":"[PATCH] Documentation: remove '\\' in front of short options","fromName":"SZEDER Gábor","fromEmail":"szeder@ira.uka.de","sentAt":"2008-09-30T17:27:10Z","receivedAt":"2008-09-30T17:27:10Z","isPatch":true,"sender":{"key":"szeder.dev@gmail.com","avatar":"https://avatars.githubusercontent.com/u/116324?v=4"},"body":"... because they show up in the man and html outputs.\n\nThis escaping is only needed for double dashes to be compatible with\nolder asciidoc versions;  see commit e1ccf53 ([PATCH] Escape asciidoc's\nbuilt-in em-dash replacement, 2005-09-12).\n\nSigned-off-by: SZEDER Gábor <szeder@ira.uka.de>\n---\n Documentation/git-rev-list.txt |    6 +++---\n Documentation/githooks.txt     |    6 +++---\n 2 files changed, 6 insertions(+), 6 deletions(-)\n\ndiff --git a/Documentation/git-rev-list.txt b/Documentation/git-rev-list.txt\nindex fd1de92..1c9cc28 100644\n--- a/Documentation/git-rev-list.txt\n+++ b/Documentation/git-rev-list.txt\n@@ -32,9 +32,9 @@ SYNOPSIS\n \t     [ \\--cherry-pick ]\n \t     [ \\--encoding[=<encoding>] ]\n \t     [ \\--(author|committer|grep)=<pattern> ]\n-\t     [ \\--regexp-ignore-case | \\-i ]\n-\t     [ \\--extended-regexp | \\-E ]\n-\t     [ \\--fixed-strings | \\-F ]\n+\t     [ \\--regexp-ignore-case | -i ]\n+\t     [ \\--extended-regexp | -E ]\n+\t     [ \\--fixed-strings | -F ]\n \t     [ \\--date={local|relative|default|iso|rfc|short} ]\n \t     [ [\\--objects | \\--objects-edge] [ \\--unpacked ] ]\n \t     [ \\--pretty | \\--header ]\ndiff --git a/Documentation/githooks.txt b/Documentation/githooks.txt\nindex 046a2a7..7fefdb1 100644\n--- a/Documentation/githooks.txt\n+++ b/Documentation/githooks.txt\n@@ -87,12 +87,12 @@ default log message, and before the editor is started.\n \n It takes one to three parameters.  The first is the name of the file\n that the commit log message.  The second is the source of the commit\n-message, and can be: `message` (if a `\\-m` or `\\-F` option was\n-given); `template` (if a `\\-t` option was given or the\n+message, and can be: `message` (if a `-m` or `-F` option was\n+given); `template` (if a `-t` option was given or the\n configuration option `commit.template` is set); `merge` (if the\n commit is a merge or a `.git/MERGE_MSG` file exists); `squash`\n (if a `.git/SQUASH_MSG` file exists); or `commit`, followed by\n-a commit SHA1 (if a `\\-c`, `\\-C` or `\\--amend` option was given).\n+a commit SHA1 (if a `-c`, `-C` or `\\--amend` option was given).\n \n If the exit status is non-zero, 'git-commit' will abort.\n \n-- \n1.6.0.2.448.g0c4e5\n"}]}