Re: [PATCH] doc: avoid makeinfo warning

[Pádraig Brady]

On 04/03/17 10:26, Bernhard Voelker wrote:
> makeinfo issues the following:
> 
>   doc/coreutils.texi:6568: warning: @sc argument all uppercase,\
>     thus no effect.
> 
> * doc/coreutils.texi (join invocation): Remove the @sc macro around
> the all uppercase "GNU".
> ---
>  doc/coreutils.texi | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/doc/coreutils.texi b/doc/coreutils.texi
> index a649c08..d582c8d 100644
> --- a/doc/coreutils.texi
> +++ b/doc/coreutils.texi
> @@ -6565,7 +6565,7 @@ Charlie  34    NA
>  @end example
>  
>  
> -To sort a file with a header line, use @sc{GNU} @command{sed -u}.
> +To sort a file with a header line, use GNU @command{sed -u}.
>  The following example sort the files but keeps the first line of each
>  file in place:

Nice one. texinfo 6.1 (on Fedora 25) doesn't catch that,
though I see 6.3 in rawhide which is probably the version you're using.

There was another texinfo issue in 6.0 mentioned by Assaf at:
http://lists.gnu.org/archive/html/coreutils/2016-12/msg00016.html
I can confirm that 6.1 addresses that, and that manual will
be updated shortly after the impending release.

I've pushed this follow up patch to bump the texinfo requirement for devs:

commit ddbe4b7a3756356bfb6af3ebd6ee39a5fe592da5
Author: Pádraig Brady <address@hidden>
Date:   Sat Mar 4 14:10:57 2017 -0800

    maint: bump makeinfo --version requirement to 6.1

    * bootstrap.conf: s/4.13/6.1/ as versions previous to that
    generated invalid html with interspersed <span> tags that
    were visible to the user.  Version 6.1 is available for a
    year now, and is available in most distros.

diff --git a/bootstrap.conf b/bootstrap.conf
index 27ac978..dd93c08 100644
--- a/bootstrap.conf
+++ b/bootstrap.conf
@@ -330,7 +330,7 @@ gettext    0.19.2
 git        1.4.4
 gperf      -
 gzip       -
-makeinfo   4.13
+makeinfo   6.1
 patch      -
 perl       5.5
 rsync      -

thanks,
Pádraig