Re: [platform-testers] new snapshot available: sed-4.3.22-3dc4c

[Assaf Gordon]

Hello Jim,

> On Feb 2, 2017, at 19:27, Jim Meyering <address@hidden> wrote:
> 
>  http://meyering.net/sed/sed-4.3.22-3dc4c.tar.xz

I'm sorry to report a small regression in the texinfo file:
On systems with old makeinfo (version 4.8), the build fails with:

=====
MAKEINFO ./doc/sed.info
./doc/sed.texi:3461: @xref expected braces.
./doc/sed.texi:3461: ` {Execution Cycle} and @ref{Addresses ov...' is too long 
for expansion; not expanded.
./doc/sed.texi:3461: First argument to cross-reference may not be empty.
./doc/sed.texi:3257: Next reference to nonexistent node `Examples' (perhaps 
incorrect sectioning?).
./doc/sed.texi:3449: Cross reference to nonexistent node `Line length 
adjustment' (perhaps incorrect sectioning?).
./doc/sed.texi:3448: Cross reference to nonexistent node `Text search across 
multiple lines' (perhaps incorrect sectioning?).
./doc/sed.texi:1502: Cross reference to nonexistent node `N_command_last_line' 
(perhaps incorrect sectioning?).
./doc/sed.texi:330: Cross reference to nonexistent node `Reporting Bugs' 
(perhaps incorrect sectioning?).
./doc/sed.texi:83: Menu reference to nonexistent node `Command and Option 
Index' (perhaps incorrect sectioning?).
./doc/sed.texi:82: Menu reference to nonexistent node `Concept Index' (perhaps 
incorrect sectioning?).
./doc/sed.texi:81: Menu reference to nonexistent node `GNU Free Documentation 
License' (perhaps incorrect sectioning?).
./doc/sed.texi:80: Menu reference to nonexistent node `Reporting Bugs' (perhaps 
incorrect sectioning?).
./doc/sed.texi:79: Menu reference to nonexistent node `Other Resources' 
(perhaps incorrect sectioning?).
./doc/sed.texi:78: Menu reference to nonexistent node `Limitations' (perhaps 
incorrect sectioning?).
./doc/sed.texi:77: Menu reference to nonexistent node `Examples' (perhaps 
incorrect sectioning?).
makeinfo: Removing output file `./doc/sed.info-t' due to errors; use --force to 
preserve.
*** Error 1 in . (Makefile:2844 './doc/sed.info')
*** Error 1 in . (Makefile:3052 'all-recursive')
*** Error 1 in /tmp/sed-4.3.22-3dc4c.SPmVYj/sed-4.3.22-3dc4c (Makefile:2159 
'all')
=====

First,
The attached patch fixes it (only need to remove the space after the @xref).


Second,
The more interesting question is why didn't it fail before (in sed-4.3.20)
since the texinfo file didn't change (and already contained this problem).

The only reason I could find is that in this new tarball,
somehow 'doc/stamp-vti' is newer than 'doc/version.texi' and 'doc/sed.info',
which perhaps causes 'make' to try and regenerate the info file ?
This is a long-shot but I couldn't find any other difference...

====
$ tar -tvf sed-4.3.20-e9ab.tar.xz | grep -E 'doc/.*(stamp|texi|info)'
-rw-r--r-- 0/0             111 2017-01-31 13:29 sed-4.3.20-e9ab/doc/stamp-vti
-rw-r--r-- 0/0             957 2016-12-10 01:40 sed-4.3.20-e9ab/doc/config.texi
-rw-r--r-- 0/0             111 2017-01-31 13:29 sed-4.3.20-e9ab/doc/version.texi
-rw-r--r-- 0/0          196972 2017-01-31 13:29 sed-4.3.20-e9ab/doc/sed.info
-rw-r--r-- 0/0          152955 2017-01-31 12:31 sed-4.3.20-e9ab/doc/sed.texi
-rw-r--r-- 0/0           23432 2017-01-17 00:35 sed-4.3.20-e9ab/doc/fdl.texi

$ tar -tvf sed-4.3.22-3dc4c.tar.xz | grep -E 'doc/.*(stamp|texi|info)'
-rw-r--r-- 0/0             113 2017-02-02 19:15 sed-4.3.22-3dc4c/doc/stamp-vti
-rw-r--r-- 0/0             957 2016-12-10 01:40 sed-4.3.22-3dc4c/doc/config.texi
-rw-r--r-- 0/0             113 2017-02-02 15:01 
sed-4.3.22-3dc4c/doc/version.texi
-rw-r--r-- 0/0          196974 2017-02-02 15:01 sed-4.3.22-3dc4c/doc/sed.info
-rw-r--r-- 0/0          152955 2017-01-31 16:46 sed-4.3.22-3dc4c/doc/sed.texi
-rw-r--r-- 0/0           23432 2017-01-17 00:35 sed-4.3.22-3dc4c/doc/fdl.texi
====




Other than this, there are no new regressions compared to the previews snapshot 
sed-4.3.20 .



Sorry for the doc-related troubles...
 - assaf

0001-doc-fix-broken-xref.patch
Description: Binary data