public inbox for newlib-cvs@sourceware.org
help / color / mirror / Atom feed
From: Corinna Vinschen <corinna@sourceware.org>
To: newlib-cvs@sourceware.org
Subject: [newlib-cygwin] Fix pdf build failure with texinfo 6.1.0
Date: Tue, 25 Oct 2016 15:45:00 -0000	[thread overview]
Message-ID: <20161025154522.33603.qmail@sourceware.org> (raw)

https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;h=cf6e411f17be3bdce2df24911517798df4172859

commit cf6e411f17be3bdce2df24911517798df4172859
Author: Thomas Preudhomme <thomas.preudhomme@foss.arm.com>
Date:   Tue Oct 25 16:11:09 2016 +0100

    Fix pdf build failure with texinfo 6.1.0
    
    Hi,
    
    make pdf on Ubuntu 16.04 fail with:
    
    newlib/libc/libc.texinfo:9: Missing @endcsname inserted.
    
    After a lot of fiddling the reason appears to be the combination of concept
    and function index despite a lack of concept index entries. Arguably texinfo
    should not error in that case but here we are, newlib will fail to build its
    documentation on some systems because of this. Since libc.texinfo only
    contains function index entries this patch simply removes the combination of
    indices. It does the same for libm.texinfo which has concept index entries but
    no function index entries.
    
    Tested by running make pdf, make dvi, make info and make html successfully.
    libc.pdf appears to have only one index as expected.
    
    == Proposed commit message ==
    
    Fix pdf build failure with texinfo 6.1.0 as provided in Ubuntu 16.04. Index
    combination in libc.texinfo and libm.texinfo fails because both file have only
    one type of index entries. Removing index combination is thus harmless and
    solves the problem.
    
    Is this ok for master?
    
    Best regards,
    
    Thomas

Diff:
---
 newlib/libc/libc.texinfo | 1 -
 newlib/libm/libm.texinfo | 1 -
 2 files changed, 2 deletions(-)

diff --git a/newlib/libc/libc.texinfo b/newlib/libc/libc.texinfo
index 6216c99..482b0ed 100644
--- a/newlib/libc/libc.texinfo
+++ b/newlib/libc/libc.texinfo
@@ -1,6 +1,5 @@
 \input texinfo.tex
 @setfilename libc.info
-@syncodeindex fn cp
 
 @ifinfo
 @format
diff --git a/newlib/libm/libm.texinfo b/newlib/libm/libm.texinfo
index 5c28975..1b5be1f 100644
--- a/newlib/libm/libm.texinfo
+++ b/newlib/libm/libm.texinfo
@@ -1,6 +1,5 @@
 \input texinfo.tex
 @setfilename libm.info
-@syncodeindex fn cp
 
 @ifinfo
 @format


                 reply	other threads:[~2016-10-25 15:45 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20161025154522.33603.qmail@sourceware.org \
    --to=corinna@sourceware.org \
    --cc=newlib-cvs@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).