From mboxrd@z Thu Jan 1 00:00:00 1970 From: John Keeping Subject: Re: [PATCH 6/8] doc: convert \--option to --option Date: Wed, 13 May 2015 10:48:42 +0100 Message-ID: <20150513094841.GA10518@serenity.lan> References: <20150513045650.GA6070@peff.net> <20150513050137.GF6821@peff.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Junio C Hamano , Jean-Noel Avila , git@vger.kernel.org To: Jeff King X-From: git-owner@vger.kernel.org Wed May 13 11:54:34 2015 Return-path: Envelope-to: gcvg-git-2@plane.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1YsTN2-0004r5-RW for gcvg-git-2@plane.gmane.org; Wed, 13 May 2015 11:54:33 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933630AbbEMJy2 (ORCPT ); Wed, 13 May 2015 05:54:28 -0400 Received: from hyena.aluminati.org ([64.22.123.221]:40030 "EHLO hyena.aluminati.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933159AbbEMJyZ (ORCPT ); Wed, 13 May 2015 05:54:25 -0400 X-Greylist: delayed 304 seconds by postgrey-1.27 at vger.kernel.org; Wed, 13 May 2015 05:54:25 EDT Received: from localhost (localhost [127.0.0.1]) by hyena.aluminati.org (Postfix) with ESMTP id 3A4F7201E7; Wed, 13 May 2015 10:49:21 +0100 (BST) X-Quarantine-ID: X-Virus-Scanned: Debian amavisd-new at hyena.aluminati.org X-Spam-Flag: NO X-Spam-Score: -2.9 X-Spam-Level: X-Spam-Status: No, score=-2.9 tagged_above=-9999 required=6.31 tests=[ALL_TRUSTED=-1, BAYES_00=-1.9] autolearn=no Received: from hyena.aluminati.org ([127.0.0.1]) by localhost (hyena.aluminati.org [127.0.0.1]) (amavisd-new, port 10026) with ESMTP id L8EzxoNH1MuG; Wed, 13 May 2015 10:49:18 +0100 (BST) Received: from serenity.lan (banza.aluminati.org [10.0.7.182]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by hyena.aluminati.org (Postfix) with ESMTPSA id AA0E322FDC; Wed, 13 May 2015 10:49:11 +0100 (BST) Content-Disposition: inline In-Reply-To: <20150513050137.GF6821@peff.net> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: On Wed, May 13, 2015 at 01:01:38AM -0400, Jeff King wrote: > Older versions of AsciiDoc would convert the "--" in > "--option" into an emdash. According to 565e135 > (Documentation: quote double-dash for AsciiDoc, 2011-06-29), > this is fixed in AsciiDoc 8.3.0. According to bf17126, we > don't support anything older than 8.4.1 anyway, so we no > longer need to worry about quoting. > > Even though this does not change the output at all, there > are a few good reasons to drop the quoting: > > 1. It makes the source prettier to read. > > 2. We don't quote consistently, which may be confusing when > reading the source. > > 3. Asciidoctor does not like the quoting, and renders a > literal backslash. > > Signed-off-by: Jeff King > --- > So this is the first of the more questionable patches, simply because we > are appeasing Asciidoctor, but not fixing the root problem (i.e., there > are still lots of places where the backslash quoting is messed up, like > HEAD@\{yesterday}). So I suspect we need to figure that problem out, and > then the asciidoctor issues will just go away. However, I think this may > be worth doing just to prettify the source. If we're changing all of these files anyway, it would be nice to wrap literal options in backticks as suggested in Documentation/CodingGuidelines. > Documentation/config.txt | 10 ++-- > Documentation/git-fast-export.txt | 6 +-- > Documentation/git-fast-import.txt | 28 +++++------ > Documentation/git-fetch-pack.txt | 2 +- > Documentation/git-pack-objects.txt | 2 +- > Documentation/git-push.txt | 4 +- > Documentation/git-rev-list.txt | 96 +++++++++++++++++++------------------- > Documentation/git-rev-parse.txt | 2 +- > Documentation/git-send-pack.txt | 2 +- > Documentation/git-show.txt | 2 +- > Documentation/gitdiffcore.txt | 6 +-- > 11 files changed, 80 insertions(+), 80 deletions(-) > > diff --git a/Documentation/config.txt b/Documentation/config.txt > index 948b8b0..e5d2ace 100644 > --- a/Documentation/config.txt > +++ b/Documentation/config.txt > @@ -2279,18 +2279,18 @@ remote..skipFetchAll:: > > remote..receivepack:: > The default program to execute on the remote side when pushing. See > - option \--receive-pack of linkgit:git-push[1]. > + option --receive-pack of linkgit:git-push[1]. > > remote..uploadpack:: > The default program to execute on the remote side when fetching. See > - option \--upload-pack of linkgit:git-fetch-pack[1]. > + option --upload-pack of linkgit:git-fetch-pack[1]. > > remote..tagOpt:: > - Setting this value to \--no-tags disables automatic tag following when > - fetching from remote . Setting it to \--tags will fetch every > + Setting this value to --no-tags disables automatic tag following when > + fetching from remote . Setting it to --tags will fetch every > tag from remote , even if they are not reachable from remote > branch heads. Passing these flags directly to linkgit:git-fetch[1] can > - override this setting. See options \--tags and \--no-tags of > + override this setting. See options --tags and --no-tags of > linkgit:git-fetch[1]. > > remote..vcs:: > diff --git a/Documentation/git-fast-export.txt b/Documentation/git-fast-export.txt > index 929e496..ed57c68 100644 > --- a/Documentation/git-fast-export.txt > +++ b/Documentation/git-fast-export.txt > @@ -67,17 +67,17 @@ produced incorrect results if you gave these options. > have been completed, or to save the marks table across > incremental runs. As is only opened and truncated > at completion, the same path can also be safely given to > - \--import-marks. > + --import-marks. > The file will not be written if no new object has been > marked/exported. > > --import-marks=:: > Before processing any input, load the marks specified in > . The input file must exist, must be readable, and > - must use the same format as produced by \--export-marks. > + must use the same format as produced by --export-marks. > + > Any commits that have already been marked will not be exported again. > -If the backend uses a similar \--import-marks file, this allows for > +If the backend uses a similar --import-marks file, this allows for > incremental bidirectional exporting of the repository by keeping the > marks the same across runs. > > diff --git a/Documentation/git-fast-import.txt b/Documentation/git-fast-import.txt > index bf7fbd4..fd32895 100644 > --- a/Documentation/git-fast-import.txt > +++ b/Documentation/git-fast-import.txt > @@ -42,13 +42,13 @@ OPTIONS > --quiet:: > Disable all non-fatal output, making fast-import silent when it > is successful. This option disables the output shown by > - \--stats. > + --stats. > > --stats:: > Display some basic statistics about the objects fast-import has > created, the packfiles they were stored into, and the > memory used by fast-import during this run. Showing this output > - is currently the default, but can be disabled with \--quiet. > + is currently the default, but can be disabled with --quiet. > > Options for Frontends > ~~~~~~~~~~~~~~~~~~~~~ > @@ -81,12 +81,12 @@ Locations of Marks Files > have been completed, or to save the marks table across > incremental runs. As is only opened and truncated > at checkpoint (or completion) the same path can also be > - safely given to \--import-marks. > + safely given to --import-marks. > > --import-marks=:: > Before processing any input, load the marks specified in > . The input file must exist, must be readable, and > - must use the same format as produced by \--export-marks. > + must use the same format as produced by --export-marks. > Multiple options may be supplied to import more than one > set of marks. If a mark is defined to different values, > the last file wins. > @@ -179,8 +179,8 @@ fast-forward update, fast-import will skip updating that ref and instead > prints a warning message. fast-import will always attempt to update all > branch refs, and does not stop on the first failure. > > -Branch updates can be forced with \--force, but it's recommended that > -this only be used on an otherwise quiet repository. Using \--force > +Branch updates can be forced with --force, but it's recommended that > +this only be used on an otherwise quiet repository. Using --force > is not necessary for an initial import into an empty repository. > > > @@ -231,11 +231,11 @@ Date Formats > ~~~~~~~~~~~~ > The following date formats are supported. A frontend should select > the format it will use for this import by passing the format name > -in the \--date-format= command-line option. > +in the --date-format= command-line option. > > `raw`:: > This is the Git native format and is `