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

Re: [RFC PATCH v1 00/13][GSoC] doc: (monospace) apply CodingGuidelines on a large-scale

From
Jean-Noël Avila <avila.jn@gmail.com>
Date
Apr 19, 2021, 09:04 UTC
Message-ID
<ec10d2ae-f546-90d1-9b7a-96f25893405f@gmail.com>
In-Reply-To
<87h7k9hq75.fsf@Inspiron.i-did-not-set--mail-host-address--so-tickle-me>
Le 13/04/2021 à 22:42, Firmin Martin a écrit :
Show 36 quoted lines
> Hi Jean-Noël,
>
> Jean-Noël Avila <avila.jn@gmail.com> writes:
>
>> Le 09/04/2021 à 06:02, Firmin Martin a écrit :
>>> This patch series aims to make the documentation fully compliant to the
>>> CodingGuidelines regarding monospace font. After it, new contributors
>>> who just want to change a little portion of the documention could just
>>> look around how it has been done without being confused by the
>>> inconsistency between the documentation and the CodingGuidelines.
>>
>> Thank you for tackling the task of formating the docu and directing to
>> CodingGuidelines for some markup rules. It appears that the last rule
>> about backticks is wrong with late Asciidoctor, for which backticks are
> Thanks. As a new Git contributor, I used to think that we don't use asciidoctor,
> until I see in Documentation/Makefile:
>
>     ifdef USE_ASCIIDOCTOR
>     ASCIIDOC = asciidoctor
>     ...
>     ASCIIDOC_EXTRA += -acompat-mode -atabsize=8
>     ...
>     endif
>
> So, we actually use both depending the variable USE_ASCIIDOCTOR. 
>
>> only a font switcher and no longer hold any semantic meaning. This means
>> that double-hyphens may need escaping (see:
>> https://asciidoctor.org/docs/migration/#migration-cheatsheet) when
>> switching style and tools.
> In the helpful link that you provide, it says that the "setext-style
> (i.e., two-line) section title" enables compatibility mode.  As far as I
> can tell, most man pages (git.*.txt) fall under this category, except
> the most important one: user-manual.txt.  But this is in fact not
> relevant, because the snippet above of the Makefile suggests that we
> actually explicitly running asciidoctor in compatibility mode.

If we are to continue using asciidoctor, I guess this compatibility mode will be removed at some point in the future. I don't have all the required inputs for how asciidoc and asciidoctor will evolve and may split in their behavior, but here is how I see it:

There's now a working group whose interest is in standardizing asciidoc and making the format evolve (https://www.eclipse.org/org/workinggroups/asciidoc-charter.php). It is led by the authors of asciidoctor, so I guess most of the path forward for asciidoc format is asciidoctor and leaving compatibility mode behind as a legacy format.

Show 17 quoted lines
>
>> One other rule worth adding would be that tabs are banned from asciidoc
>> because they cannot always be matched with correct indentation.
> I'm an absolute novice regarding AsciiDoc vs. Asciidoctor. But from the
> user guide of AsciiDoc (https://asciidoc.org/userguide.html#_tabs), it says
>
>     By default tab characters input files will translated to 8
>     spaces. Tab expansion is set with the tabsize entry in the
>     configuration file [miscellaneous] section and can be overridden in
>     included files by setting a tabsize attribute in the include macro’s
>     attribute list. For example:
>
>     include::addendum.txt[tabsize=2] The tab size can also be set using
>     the attribute command-line option, for example --attribute tabsize=4
>
> ... and we also explicitly set it to 8 spaces (see above). Could you
> elaborate a bit on the matter ?

Tab management in asciidoc and asciidoctor was the source of a number of bugs, because the markup relies on "visual" alignment instead of semantic alignment. So you have to define tabsize in order for tabs to be correctly interpreted. Instead of having to add specification, my approach would be to just get rid of the source and the solution of the issue at the same time.

Previous: Firmin MartinNext: Jean-Noël Avila
Message 19 of 20 in “[GSoC] doc: (monospace) apply CodingGuidelines on a large-scale”
  1. 00/13 [GSoC] doc: (monospace) apply CodingGuidelines on a large-scaleFirmin Martin, Apr 9, 2021
  2. 01/13 doc: typeset command-line options in monospaceFirmin Martin, Apr 9, 2021
  3. Bagas SanjayaApr 9, 2021
  4. Firmin MartinApr 10, 2021
  5. 03/13 doc: typeset configuration options in monospaceFirmin Martin, Apr 9, 2021
  6. 02/13 doc: typeset branches and remotes in monospaceFirmin Martin, Apr 9, 2021
  7. 05/13 doc: typeset git-svn subcommands in monospaceFirmin Martin, Apr 9, 2021
  8. 06/13 doc: typeset dummy URLs and protocols in monospaceFirmin Martin, Apr 9, 2021
  9. 07/13 doc: typeset git dotfiles in monospaceFirmin Martin, Apr 9, 2021
  10. 04/13 doc: typeset git-related commands in monospaceFirmin Martin, Apr 9, 2021
  11. 11/13 doc: typeset config option entries in monospaceFirmin Martin, Apr 9, 2021
  12. 08/13 doc: typeset filepath and $variables in monospaceFirmin Martin, Apr 9, 2021
  13. 10/13 doc: typeset more command entries in monospaceFirmin Martin, Apr 9, 2021
  14. 09/13 doc: typeset command/option/value entries in monospaceFirmin Martin, Apr 9, 2021
  15. 13/13 doc: typeset common programs in monospaceFirmin Martin, Apr 9, 2021
  16. 12/13 doc: typeset environment vars without $ in monospaceFirmin Martin, Apr 9, 2021
  17. Jean-Noël AvilaApr 12, 2021
  18. Firmin MartinApr 13, 2021
  19. Jean-Noël AvilaApr 19, 2021
  20. Jean-Noël AvilaApr 19, 2021

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.