Re: Your Patches

[Trevor Daniels]

James wrote Saturday, November 24, 2012 1:09 PM

>On 22 November 2012 20:43, Trevor Daniels <address@hidden> wrote:
>>
>> It would be great if you could check these patches out with a full make doc. 
>>  I know the texi stuff is OK, because I always run texi2html on them, and I 
>> know the cross-refs are fine, but a full doc build is the real test.  A 
>> format-patch of 4 combined patches is attached.  I left out the most recent 
>> one about footnotes as that needs more work anyway.
>
> It passes make fine, but when I make doc I get an error.
>
> It seems that there is a problem in the nl docs.
>
> The log learning.bigtexi.log in all the other languages have the usual
> 'warnings' but the dutch one has
>
> --snip--
> WARNING: Unable to find node 'Creating titles' in book notation.
> *** Undefined node `Single staff' in @ref (in
> out-www/learning/fundamental.texi l. 3171)
> Max error number exceeded
> --snip--
>
> unfortunately the build scripts don't say *which* log to look at (i.e.
> there are learning.bigtexi.log files in all the languages - most are 0
> bytes).
>
> But it does stop make doc.

Thanks James.  I think this is because Documentation/nl/learning does
not contain the templates.itely file, so make doc defaults to using
that file from Documentation/learning, which has different node
names after this patch is applied.

All the language directories contain templates.itely except nl, so
the fix is probably to copy this file (before this patch) into nl.

Copying to Francisco and devel for comments before I change
anything.

Trevor