From: Martin Ågren <hidden> Date: 2020-03-29 13:18:27
In the documentation Makefile and a few helper conf and xsl files, we
carry various workarounds to make the docs build at all or to look
better with older docbook-xsl versions. Let's see if we can't just drop
that support. All of this seems to have been useful at one point, but I
doubt that anyone needs this nowadays. This was also suggested in [1].
The final commit in this series bumps the minimum required version of
docbook-xsl to 1.74.0, which is from June 2008. The whole series should
be a no-op unless you're on an older release than that. That said, I've
only tested this with the version that comes with my OS, 1.79.1.
After this series, user-manual.conf still refers to older docbook-xsl
versions. The proper fix there might be to actually be a bit more
aggressive and drop that hunk, making the rendered docs prettier.
There's some history there, including mentions of texinfo, which is
outside my comfort zone. I've got work in progress there, but I'd rather
submit that separately from these "expected no-op" patches.
Martin
[1] https://lore.kernel.org/git/20191030212422.GE29013@sigill.intra.peff.net/
Martin Ågren (6):
Doc: drop support for docbook-xsl before 1.71.1
Doc: drop support for docbook-xsl before 1.72.0
Doc: drop support for docbook-xsl before 1.73.0
manpage-bold-literal.xsl: stop using git.docbook.backslash
manpage-normal.xsl: fold in manpage-base.xsl
INSTALL: drop support for docbook-xsl before 1.74
Documentation/Makefile | 23 -----------------
Documentation/asciidoc.conf | 19 --------------
Documentation/manpage-1.72.xsl | 14 -----------
Documentation/manpage-base.xsl | 35 --------------------------
Documentation/manpage-bold-literal.xsl | 6 ++---
Documentation/manpage-normal.xsl | 25 +++++++++++++-----
Documentation/manpage-suppress-sp.xsl | 21 ----------------
Makefile | 6 -----
INSTALL | 4 +--
9 files changed, 22 insertions(+), 131 deletions(-)
delete mode 100644 Documentation/manpage-1.72.xsl
delete mode 100644 Documentation/manpage-base.xsl
delete mode 100644 Documentation/manpage-suppress-sp.xsl
--
2.26.0
From: Martin Ågren <hidden> Date: 2020-03-29 13:18:33
Drop the DOCBOOK_SUPPRESS_SP mechanism, which needs to be used with
docbook-xsl versions 1.69.1 through 1.71.0.
We probably broke this for Asciidoctor builds in f6461b82b9
("Documentation: fix build with Asciidoctor 2", 2019-09-15). That is, we
should/could fix this similar to 55aca515eb ("manpage-bold-literal.xsl:
match for namespaced "d:literal" in template", 2019-10-31). But rather
than digging out such an old version of docbook-xsl to test that, let's
just use this as an excuse for dropping this decade-old workaround.
DOCBOOK_SUPPRESS_SP was not needed with docbook-xsl 1.69.0 and older.
Maybe such old versions still work fine on our docs, or maybe not. Let's
just refer to everything before 1.71.1 as "not supported". The next
commit will increase the required/assumed version further.
Signed-off-by: Martin Ågren <redacted>
---
Documentation/Makefile | 7 +------
Documentation/manpage-suppress-sp.xsl | 21 ---------------------
INSTALL | 1 +
3 files changed, 2 insertions(+), 27 deletions(-)
delete mode 100644 Documentation/manpage-suppress-sp.xsl
@@ -151,9 +151,7 @@ endif## For docbook-xsl ...-# -1.68.1, no extra settings are needed?-# 1.69.0, set ASCIIDOC_ROFF?-# 1.69.1-1.71.0, set DOCBOOK_SUPPRESS_SP?+# -1.71.0, not supported# 1.71.1, set ASCIIDOC_ROFF?# 1.72.0, set DOCBOOK_XSL_172.# 1.73.0-, no extra settings are needed
@@ -172,9 +170,6 @@ endififndef NO_MAN_BOLD_LITERALXMLTO_EXTRA+=-mmanpage-bold-literal.xslendif-ifdef DOCBOOK_SUPPRESS_SP-XMLTO_EXTRA+=-mmanpage-suppress-sp.xsl-endif# Newer DocBook stylesheet emits warning cruft in the output when# this is not set, and if set it shows an absolute link. Older
@@ -1,21 +0,0 @@-<!-- manpage-suppress-sp.xsl:- special settings for manpages rendered from asciidoc+docbook- handles erroneous, inline .sp in manpage output of some- versions of docbook-xsl -->-<xsl:stylesheet xmlns:xsl="http://www.w3.org/1999/XSL/Transform"- version="1.0">--<!-- attempt to work around spurious .sp at the tail of the line- that some versions of docbook stylesheets seem to add -->-<xsl:template match="simpara">- <xsl:variable name="content">- <xsl:apply-templates/>- </xsl:variable>- <xsl:value-of select="normalize-space($content)"/>- <xsl:if test="not(ancestor::authorblurb) and- not(ancestor::personblurb)">- <xsl:text> </xsl:text>- </xsl:if>-</xsl:template>--</xsl:stylesheet>
@@ -206,6 +206,7 @@ Issues of note: clone two separate git-htmldocs and git-manpages repositories next to the clone of git itself.+ The minimum supported version of docbook-xsl is 1.71.1. It has been reported that docbook-xsl version 1.72 and 1.73 are buggy; 1.72 misformats manual pages for callouts, and 1.73 needs the patch in contrib/patches/docbook-xsl-manpages-charmap.patch
From: Martin Ågren <hidden> Date: 2020-03-29 13:18:33
docbook-xsl 1.72.0 is thirteen years old. Drop the ASCIIDOC_ROFF knob
which was needed to support 1.68.1 - 1.71.1. The next commit will
increase the required/assumed version further.
Signed-off-by: Martin Ågren <redacted>
---
Documentation/Makefile | 10 +---------
Documentation/asciidoc.conf | 19 -------------------
Makefile | 3 ---
INSTALL | 2 +-
4 files changed, 2 insertions(+), 32 deletions(-)
@@ -151,21 +151,13 @@ endif## For docbook-xsl ...-# -1.71.0, not supported-# 1.71.1, set ASCIIDOC_ROFF?+# -1.71.1, not supported# 1.72.0, set DOCBOOK_XSL_172.# 1.73.0-, no extra settings are needed#ifdef DOCBOOK_XSL_172-ASCIIDOC_EXTRA+=-agit-asciidoc-no-roffMANPAGE_XSL=manpage-1.72.xsl-else- ifndef ASCIIDOC_ROFF- # docbook-xsl after 1.72 needs the regular XSL, but will not- # pass-thru raw roff codes from asciidoc.conf, so turn them off.-ASCIIDOC_EXTRA+=-agit-asciidoc-no-roff- endifendififndef NO_MAN_BOLD_LITERALXMLTO_EXTRA+=-mmanpage-bold-literal.xsl
@@ -31,24 +31,6 @@ ifdef::backend-docbook[] endif::backend-docbook[] ifdef::backend-docbook[]-ifndef::git-asciidoc-no-roff[]-# "unbreak" docbook-xsl v1.68 for manpages. v1.69 works with or without this.-# v1.72 breaks with this because it replaces dots not in roff requests.-[listingblock]-<example><title>{title}</title>-<literallayout class="monospaced">-ifdef::doctype-manpage[]- .ft C -endif::doctype-manpage[]-|-ifdef::doctype-manpage[]- .ft -endif::doctype-manpage[]-</literallayout>-{title#}</example>-endif::git-asciidoc-no-roff[]--ifdef::git-asciidoc-no-roff[] ifdef::doctype-manpage[] # The following two small workarounds insert a simple paragraph after screen [listingblock]
@@ -285,9 +285,6 @@ all::# Define DOCBOOK_XSL_172 if you want to format man pages with DocBook XSL v1.72# (not v1.73 or v1.71).#-# Define ASCIIDOC_ROFF if your DocBook XSL does not escape raw roff directives-# (versions 1.68.1 through v1.72).-## Define GNU_ROFF if your target system uses GNU groff. This forces# apostrophes to be ASCII so that cut&pasting examples to the shell# will work.
@@ -206,7 +206,7 @@ Issues of note: clone two separate git-htmldocs and git-manpages repositories next to the clone of git itself.- The minimum supported version of docbook-xsl is 1.71.1.+ The minimum supported version of docbook-xsl is 1.72. It has been reported that docbook-xsl version 1.72 and 1.73 are buggy; 1.72 misformats manual pages for callouts, and 1.73 needs the patch in contrib/patches/docbook-xsl-manpages-charmap.patch
From: Martin Ågren <hidden> Date: 2020-03-29 13:18:35
Drop the DOCBOOK_XSL_172 config knob, which was needed with docbook-xsl
1.72 (but neither 1.71 nor 1.73). Version 1.73.0 is more than twelve
years old.
Together with the last few commits, we are now at a point where we don't
have any Makefile knobs to cater to old/broken versions of docbook-xsl.
Signed-off-by: Martin Ågren <redacted>
---
Documentation/Makefile | 10 ----------
Documentation/manpage-1.72.xsl | 14 --------------
Documentation/manpage-normal.xsl | 3 +--
Makefile | 3 ---
INSTALL | 5 ++---
5 files changed, 3 insertions(+), 32 deletions(-)
delete mode 100644 Documentation/manpage-1.72.xsl
@@ -149,16 +149,6 @@ endif-include ../config.mak.autogen-include ../config.mak-#-# For docbook-xsl ...-# -1.71.1, not supported-# 1.72.0, set DOCBOOK_XSL_172.-# 1.73.0-, no extra settings are needed-#--ifdef DOCBOOK_XSL_172-MANPAGE_XSL=manpage-1.72.xsl-endififndef NO_MAN_BOLD_LITERALXMLTO_EXTRA+=-mmanpage-bold-literal.xslendif
@@ -1,14 +0,0 @@-<!-- manpage-1.72.xsl:- special settings for manpages rendered from asciidoc+docbook- handles peculiarities in docbook-xsl 1.72.0 -->-<xsl:stylesheet xmlns:xsl="http://www.w3.org/1999/XSL/Transform"- version="1.0">--<xsl:import href="manpage-base.xsl"/>--<!-- these are the special values for the roff control characters- needed for docbook-xsl 1.72.0 -->-<xsl:param name="git.docbook.backslash">▓</xsl:param>-<xsl:param name="git.docbook.dot" >⌂</xsl:param>--</xsl:stylesheet>
@@ -282,9 +282,6 @@ all::# Define NO_ST_BLOCKS_IN_STRUCT_STAT if your platform does not have st_blocks# field that counts the on-disk footprint in 512-byte blocks.#-# Define DOCBOOK_XSL_172 if you want to format man pages with DocBook XSL v1.72-# (not v1.73 or v1.71).-## Define GNU_ROFF if your target system uses GNU groff. This forces# apostrophes to be ASCII so that cut&pasting examples to the shell# will work.
@@ -206,9 +206,8 @@ Issues of note: clone two separate git-htmldocs and git-manpages repositories next to the clone of git itself.- The minimum supported version of docbook-xsl is 1.72.- It has been reported that docbook-xsl version 1.72 and 1.73 are- buggy; 1.72 misformats manual pages for callouts, and 1.73 needs+ The minimum supported version of docbook-xsl is 1.73.+ It has been reported that docbook-xsl version 1.73 is buggy; it needs the patch in contrib/patches/docbook-xsl-manpages-charmap.patch Users attempting to build the documentation on Cygwin may need to ensure
From: Martin Ågren <hidden> Date: 2020-03-29 13:18:36
We used to assign git.docbook.backslash one of two different values --
one "normal" and one for working around a problem with docbook-xsl 1.72.
After the previous commit, we don't support that version anymore and
always use the "normal" value, a literal backslash.
Just explicitly use a backslash instead of using git.docbook.backslash.
The next commit will drop the definition of git.docbook.backslash
entirely.
Signed-off-by: Martin Ågren <redacted>
---
Documentation/manpage-bold-literal.xsl | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
From: Martin Ågren <hidden> Date: 2020-03-29 13:18:39
After an earlier commit, we only include manpage-base.xsl from a single
file, manpage-normal.xsl. Fold the former into the latter.
We only ever needed the "base, normal and non-normal" construct to
support a single non-normal case, namely to work around issues with
docbook-xsl 1.72 handling backslashes and dots. If we ever need
something like this again, we can re-introduce manpage-base.xsl and
friends. Whatever issue we'd be trying to work around, it probably
wouldn't involve dots and backslashes like this, anyway.
Signed-off-by: Martin Ågren <redacted>
---
Documentation/manpage-base.xsl | 35 --------------------------------
Documentation/manpage-normal.xsl | 22 ++++++++++++++++----
2 files changed, 18 insertions(+), 39 deletions(-)
delete mode 100644 Documentation/manpage-base.xsl
@@ -1,35 +0,0 @@-<!-- manpage-base.xsl:- special formatting for manpages rendered from asciidoc+docbook -->-<xsl:stylesheet xmlns:xsl="http://www.w3.org/1999/XSL/Transform"- version="1.0">--<!-- these params silence some output from xmlto -->-<xsl:param name="man.output.quietly" select="1"/>-<xsl:param name="refentry.meta.get.quietly" select="1"/>--<!-- convert asciidoc callouts to man page format;- git.docbook.backslash and git.docbook.dot params- must be supplied by another XSL file or other means -->-<xsl:template match="co">- <xsl:value-of select="concat(- $git.docbook.backslash,'fB(',- substring-after(@id,'-'),')',- $git.docbook.backslash,'fR')"/>-</xsl:template>-<xsl:template match="calloutlist">- <xsl:value-of select="$git.docbook.dot"/>- <xsl:text>sp </xsl:text>- <xsl:apply-templates/>- <xsl:text> </xsl:text>-</xsl:template>-<xsl:template match="callout">- <xsl:value-of select="concat(- $git.docbook.backslash,'fB',- substring-after(@arearefs,'-'),- '. ',$git.docbook.backslash,'fR')"/>- <xsl:apply-templates/>- <xsl:value-of select="$git.docbook.dot"/>- <xsl:text>br </xsl:text>-</xsl:template>--</xsl:stylesheet>
@@ -3,10 +3,24 @@<xsl:stylesheetxmlns:xsl="http://www.w3.org/1999/XSL/Transform"version="1.0">-<xsl:importhref="manpage-base.xsl"/>-<!-- these are the normal values for the roff control characters -->-<xsl:paramname="git.docbook.backslash">\</xsl:param>-<xsl:paramname="git.docbook.dot">.</xsl:param>+<!-- these params silence some output from xmlto -->+<xsl:paramname="man.output.quietly"select="1"/>+<xsl:paramname="refentry.meta.get.quietly"select="1"/>++<!-- convert asciidoc callouts to man page format -->+<xsl:templatematch="co">+<xsl:value-ofselect="concat('\fB(',substring-after(@id,'-'),')\fR')"/>+</xsl:template>+<xsl:templatematch="calloutlist">+<xsl:text>.sp </xsl:text>+<xsl:apply-templates/>+<xsl:text> </xsl:text>+</xsl:template>+<xsl:templatematch="callout">+<xsl:value-ofselect="concat('\fB',substring-after(@arearefs,'-'),'. \fR')"/>+<xsl:apply-templates/>+<xsl:text>.br </xsl:text>+</xsl:template></xsl:stylesheet>
From: Martin Ågren <hidden> Date: 2020-03-29 13:18:42
Several of the previous commits have been bumping the minimum supported
version of docbook-xsl and dropping various workarounds. Most recently,
we made the minimum be 1.73.0.
In INSTALL, we claim that with 1.73, one needs a certain patch in
contrib/patches/. There is no such patch. It was added in 2ec39edad9
("INSTALL: add warning on docbook-xsl 1.72 and 1.73", 2007-08-03) and
dropped in 9721ac9010 ("contrib: remove continuous/ and patches/",
2013-06-03).
Rather than resurrecting version 1.73 and the patch and testing them,
just raise our minimum supported docbook-xsl version to 1.74.
Signed-off-by: Martin Ågren <redacted>
---
INSTALL | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
@@ -206,9 +206,7 @@ Issues of note: clone two separate git-htmldocs and git-manpages repositories next to the clone of git itself.- The minimum supported version of docbook-xsl is 1.73.- It has been reported that docbook-xsl version 1.73 is buggy; it needs- the patch in contrib/patches/docbook-xsl-manpages-charmap.patch+ The minimum supported version of docbook-xsl is 1.74. Users attempting to build the documentation on Cygwin may need to ensure that the /etc/xml/catalog file looks something like this:
From: Jeff King <hidden> Date: 2020-03-30 09:26:16
On Sun, Mar 29, 2020 at 03:18:10PM +0200, Martin Ågren wrote:
Several of the previous commits have been bumping the minimum supported
version of docbook-xsl and dropping various workarounds. Most recently,
we made the minimum be 1.73.0.
In INSTALL, we claim that with 1.73, one needs a certain patch in
contrib/patches/. There is no such patch. It was added in 2ec39edad9
("INSTALL: add warning on docbook-xsl 1.72 and 1.73", 2007-08-03) and
dropped in 9721ac9010 ("contrib: remove continuous/ and patches/",
2013-06-03).
Perhaps nobody asking about the missing patch since 2013 should be a
hint that nobody cares about these old versions. :)
Rather than resurrecting version 1.73 and the patch and testing them,
just raise our minimum supported docbook-xsl version to 1.74.
From: Jeff King <hidden> Date: 2020-03-30 09:45:16
On Sun, Mar 29, 2020 at 03:18:04PM +0200, Martin Ågren wrote:
In the documentation Makefile and a few helper conf and xsl files, we
carry various workarounds to make the docs build at all or to look
better with older docbook-xsl versions. Let's see if we can't just drop
that support. All of this seems to have been useful at one point, but I
doubt that anyone needs this nowadays. This was also suggested in [1].
Yay, I'm very happy to see this series. I'd be happy to go even further
if there's some benefit, but I think this removes the last of the
Makefile knobs.
Thanks for splitting each version bump into its own patch. That made it
a lot easier to see what we we're gaining from each one.
I double-checked that there's no difference with doc-diff when using a
modern asciidoc/docbook combo, though I'm sure you did as well.
After this series, user-manual.conf still refers to older docbook-xsl
versions. The proper fix there might be to actually be a bit more
aggressive and drop that hunk, making the rendered docs prettier.
There's some history there, including mentions of texinfo, which is
outside my comfort zone. I've got work in progress there, but I'd rather
submit that separately from these "expected no-op" patches.
Yeah, dropping that bit from user-manual.conf seems reasonable. That
shouldn't show anything in doc-diff because it's not installed with the
manpages. And the HTML build wouldn't use docbook. I installed the
zillion packages needed to build user-manual.pdf. The behavior without
that block looks significantly nicer (the example blocks are actually
shaded).
The very first example block in section 1.1 has some funny indentation,
but that's true before or after.
Anyway, that was just for my own curiosity. If you've got further work
in that area and prefer to do it as a separate series, that's fine by
me.
-Peff
From: brian m. carlson <hidden> Date: 2020-03-30 10:46:15
On 2020-03-29 at 13:18:04, Martin Ågren wrote:
In the documentation Makefile and a few helper conf and xsl files, we
carry various workarounds to make the docs build at all or to look
better with older docbook-xsl versions. Let's see if we can't just drop
that support. All of this seems to have been useful at one point, but I
doubt that anyone needs this nowadays. This was also suggested in [1].
The final commit in this series bumps the minimum required version of
docbook-xsl to 1.74.0, which is from June 2008. The whole series should
be a no-op unless you're on an older release than that. That said, I've
only tested this with the version that comes with my OS, 1.79.1.
After this series, user-manual.conf still refers to older docbook-xsl
versions. The proper fix there might be to actually be a bit more
aggressive and drop that hunk, making the rendered docs prettier.
There's some history there, including mentions of texinfo, which is
outside my comfort zone. I've got work in progress there, but I'd rather
submit that separately from these "expected no-op" patches.
This seems sane to me and I'm all for dropping support for a
twelve-year-old version of the DocBook XSL stylesheets. On the odd
chance that someone did need to download a newer version, this is a
dependency that's easy to set up and build.
--
brian m. carlson: Houston, Texas, US
OpenPGP: https://keybase.io/bk2204
From: Martin Ågren <hidden> Date: 2020-03-31 19:26:19
On Mon, 30 Mar 2020 at 11:45, Jeff King [off-list ref] wrote:
On Sun, Mar 29, 2020 at 03:18:04PM +0200, Martin Ågren wrote:
Yay, I'm very happy to see this series. I'd be happy to go even further
if there's some benefit, but I think this removes the last of the
Makefile knobs.
Yes, I should have been clear about that: This does remove the last
Makefile knob. At least I couldn't find any more with some browsing and
grepping.
quoted
After this series, user-manual.conf still refers to older docbook-xsl
versions. The proper fix there might be to actually be a bit more
aggressive and drop that hunk, making the rendered docs prettier.
There's some history there, including mentions of texinfo, which is
outside my comfort zone. I've got work in progress there, but I'd rather
submit that separately from these "expected no-op" patches.
Yeah, dropping that bit from user-manual.conf seems reasonable. That
shouldn't show anything in doc-diff because it's not installed with the
manpages. And the HTML build wouldn't use docbook. I installed the
zillion packages needed to build user-manual.pdf. The behavior without
that block looks significantly nicer (the example blocks are actually
shaded).
This matches what I've seen.
Anyway, that was just for my own curiosity. If you've got further work
in that area and prefer to do it as a separate series, that's fine by
me.
Not sure about "further work" really. I'm pretty sure you tried out the
exact same diff that I have, and I'm glad you agree it looks prettier /
"significantly nicer". A small part of why I didn't submit it is that
it's user manual vs man pages. Another part is that it's an intentional
change as opposed to an intended no-op.
But most importantly: When I looked into the history, I came upon
c2a7f5d438 ("docs: monospace listings in docbook output", 2012-08-07),
which made me worry about breaking "make info". On second thought, I
might have broken it many times already over the past few years, since
I've never built the info. So maybe worrying about that all of a sudden
is a bit unfounded in a way. :-/
(I tried to build "info" while working on this. It works in the sense
that it doesn't error out, but I don't get anything that looks remotely
useful. I've never used info at all though, to be honest, so could be
missing something fundamental.)
Here's what I have. I suppose it could be framed as a patch 1/7 instead.
Martin
-- >8 --
Subject: [PATCH 7/6?] user-manual.conf: don't specify [listingblock]
This is the config file we use when we build the user manual with
AsciiDoc. The comment at the top of this chunk that we're removing says
the following:
"unbreak" docbook-xsl v1.68 for manpages (sic!). v1.69 works with or
without this.
This comes from d19fbc3c17 ("Documentation: add git user's manual",
2007-01-07), where it looks like this conf file in general and this
snippet in particular was copy-pasted from asciidoc.conf.
This chunk is very similar to something we just got rid of for the
manpages, and because this appears to be aimed at v1.68 -- which we no
longer support for the manpages as of a few commits ago --, it's
tempting to get rid of this. That reveals an interesting aspect of
"works with or without this": it turns out it actually works /better/
without!
Dropping this makes us render code snippets and shell listings using
<screen> rather than <literallayout>, just like Asciidoctor does. In
user-manual.pdf, this puts the contents into dimmed-background,
easy-to-distinguish-from-the-surrounding-text boxes, as opposed to
white-background (transparent) boxes.
Signed-off-by: Martin Ågren <redacted>
---
Documentation/user-manual.conf | 10 ----------
1 file changed, 10 deletions(-)
@@ -9,13 +9,3 @@ tilde=~ [linkgit-inlinemacro] <ulink url="{target}.html">{target}{0?({0})}</ulink>--ifdef::backend-docbook[]-# "unbreak" docbook-xsl v1.68 for manpages. v1.69 works with or without this.-[listingblock]-<example><title>{title}</title>-<literallayout class="monospaced">-|-</literallayout>-{title#}</example>-endif::backend-docbook[]
From: Jeff King <hidden> Date: 2020-04-01 10:17:19
On Tue, Mar 31, 2020 at 09:26:00PM +0200, Martin Ågren wrote:
But most importantly: When I looked into the history, I came upon
c2a7f5d438 ("docs: monospace listings in docbook output", 2012-08-07),
which made me worry about breaking "make info". On second thought, I
might have broken it many times already over the past few years, since
I've never built the info. So maybe worrying about that all of a sudden
is a bit unfounded in a way. :-/
(I tried to build "info" while working on this. It works in the sense
that it doesn't error out, but I don't get anything that looks remotely
useful. I've never used info at all though, to be honest, so could be
missing something fundamental.)
I don't find "info" useful, either. The big thing it offers over "man"
is actual hyperlinks, but linkgit attributes (the only thing we actually
bother annotating) don't seem to actually become links! These days I'd
expect most people interested in hyperlinking to just use HTML. I
wouldn't be surprised if there's some emacs integration with info,
though, so maybe people use it there. I dunno. I'd be happy to see all
of the info bits go away.
I've add brian to the cc, since he was the last person to touch texi
stuff (though he may have just been doing it out of a sense of
cleanup/duty).
The .texi generated for the user-manual is the same before and after
your patch, so I think it's OK to do.
There _is_ an older regression in the user-manual, though. After commit
03920ac69b (asciidoctor: fix user-manual to be built by `asciidoctor`,
2017-01-02), the whole front-matter is missing from the generated
"git.info" file, and there's a funky ": idm4" node. Nobody seems to have
noticed or complained since then (though perhaps people are more likely
to look at the manpages than the user manual).
From: brian m. carlson <hidden> Date: 2020-04-02 00:45:40
On 2020-04-01 at 10:17:16, Jeff King wrote:
On Tue, Mar 31, 2020 at 09:26:00PM +0200, Martin Ågren wrote:
quoted
But most importantly: When I looked into the history, I came upon
c2a7f5d438 ("docs: monospace listings in docbook output", 2012-08-07),
which made me worry about breaking "make info". On second thought, I
might have broken it many times already over the past few years, since
I've never built the info. So maybe worrying about that all of a sudden
is a bit unfounded in a way. :-/
(I tried to build "info" while working on this. It works in the sense
that it doesn't error out, but I don't get anything that looks remotely
useful. I've never used info at all though, to be honest, so could be
missing something fundamental.)
I don't find "info" useful, either. The big thing it offers over "man"
is actual hyperlinks, but linkgit attributes (the only thing we actually
bother annotating) don't seem to actually become links! These days I'd
expect most people interested in hyperlinking to just use HTML. I
wouldn't be surprised if there's some emacs integration with info,
though, so maybe people use it there. I dunno. I'd be happy to see all
of the info bits go away.
I believe Emacs does have good integration with the info format. I
personally prefer documentation in almost any other format and would
also be happy to see it go away, but Emacs users may find it convenient.
Junio, I believe you use Emacs. Do you have an opinion on whether, on
the whole, Emacs users are likely to make good use of the info
documentation?
I've add brian to the cc, since he was the last person to touch texi
stuff (though he may have just been doing it out of a sense of
cleanup/duty).
I clearly see the commit in the history with my name on it and the
explanation seems like something I might have written, but I have no
recollection of touching that code or writing the patch at all. I very
likely only touched it because I felt compelled not to introduce a
regression (although, as you mentioned, I failed).
The .texi generated for the user-manual is the same before and after
your patch, so I think it's OK to do.
There _is_ an older regression in the user-manual, though. After commit
03920ac69b (asciidoctor: fix user-manual to be built by `asciidoctor`,
2017-01-02), the whole front-matter is missing from the generated
"git.info" file, and there's a funky ": idm4" node. Nobody seems to have
noticed or complained since then (though perhaps people are more likely
to look at the manpages than the user manual).
If nobody seems to care very much for the info documentation, we could
drop it. I've put a query out on Twitter to see if anyone I know is
using it.
--
brian m. carlson: Houston, Texas, US
OpenPGP: https://keybase.io/bk2204