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

Re: [PATCH 3/3] git-log.txt,rev-list-options.txt: put option blocks in proper order

From
Junio C Hamano <gitster@pobox.com>
Date
Mar 9, 2011, 23:38 UTC
Message-ID
<7vbp1j985v.fsf@alter.siamese.dyndns.org>
In-Reply-To
<f34ebc6211ead3fa6698ff9b7a92903723715a28.1299572814.git.git@drmicha.warpmail.net>
Michael J Gruber <git@drmicha.warpmail.net> writes:
Show 17 quoted lines
> diff --git a/Documentation/git-log.txt b/Documentation/git-log.txt
> index 48c1715..6ae57dc 100644
> --- a/Documentation/git-log.txt
> +++ b/Documentation/git-log.txt
> @@ -77,12 +77,12 @@ Common diff options
>  ~~~~~~~~~~~~~~~~~~~
>  
>  :git-log: 1
> -include::diff-options.txt[]
> -
>  include::rev-list-options.txt[]
>  
>  include::pretty-formats.txt[]
>  
> +include::diff-options.txt[]
> +
>  include::diff-generate-patch.txt[]

This is wrong. The title "Common diff options", telling the AsciiDoc that we are formatting for git-log manual page with ":git-log: 1" and inclusion of diff-options.txt form a single group. With your patch, the "Common diff options" section will become ampty and makes AsciiDoc barf.

You would need _at least_ something like the attached patch on top, which for now I'll squash in.

 Documentation/git-log.txt |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/Documentation/git-log.txt b/Documentation/git-log.txt
index 6ae57dc..c43aa43 100644
--- a/Documentation/git-log.txt
+++ b/Documentation/git-log.txt
@@ -73,14 +73,14 @@ produced by --stat etc.
 	to be prefixed with "\-- " to separate them from options or
 	refnames.
 
-Common diff options
-~~~~~~~~~~~~~~~~~~~
-
-:git-log: 1
 include::rev-list-options.txt[]
 
 include::pretty-formats.txt[]
 
+Common diff options
+-------------------
+
+:git-log: 1
 include::diff-options.txt[]
 
 include::diff-generate-patch.txt[]
Previous: Michael J GruberNext: Michael J Gruber
Message 13 of 14 in “--max-count useless with git-rev-list's --reverse”
  1. Ævar Arnfjörð BjarmasonMar 7, 2011
  2. Shawn PearceMar 7, 2011
  3. Jay SoffianMar 8, 2011
  4. Michael J GruberMar 8, 2011
  5. Michael J GruberMar 8, 2011
  6. 1/3 rev-list-options.txt: typo fixMichael J Gruber, Mar 8, 2011
  7. Junio C HamanoMar 8, 2011
  8. Michael J GruberMar 9, 2011
  9. 2/3 git-log.txt,rev-list-options.txt: -n/--max-count is commit limitingMichael J Gruber, Mar 8, 2011
  10. Junio C HamanoMar 8, 2011
  11. Michael J GruberMar 9, 2011
  12. 3/3 git-log.txt,rev-list-options.txt: put option blocks in proper orderMichael J Gruber, Mar 8, 2011
  13. Junio C HamanoMar 9, 2011
  14. Michael J GruberMar 10, 2011

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.