Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 554226 - sci-mathematics/octave-4.0.0 fails because of a LaTeX error
Summary: sci-mathematics/octave-4.0.0 fails because of a LaTeX error
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Science Mathematics related packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-07-08 11:09 UTC by Helmut Jarausch
Modified: 2016-05-02 23:03 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
"pre-patch" which has to be applied after failure (octave-4.0.0-backslash.patch,896 bytes, patch)
2015-07-08 11:09 UTC, Helmut Jarausch
Details | Diff
untested patch (octave-4.0.0-texinfo.patch,2.39 KB, patch)
2015-09-05 02:42 UTC, Mark Wright
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Helmut Jarausch 2015-07-08 11:09:43 UTC
Created attachment 406352 [details, diff]
"pre-patch"  which has to be applied after failure

(/var/tmp/portage/sci-mathematics/octave-4.0.0/work/octave-4.0.0/doc/interprete
r/cp-idx.texi (Concept Index) [923] [924] (./octave.cps
./octave.cps:56: Argument of \\ has an extra }.
<inserted text> 
                \par 
<to be read again> 
                   }
\initial ... .5\baselineskip \leftline {\secbf #1}
                                                  \nobreak \vskip .33\baseli...
l.56 \initial {\\}
                  
? 
./octave.cps:56: Emergency stop.

The problem is that \\ is a LaTeX macro.

Unfortunately, I cannot come up a with a patch, but

after emerge has failed

cd /var/tmp/portage/sci-mathematics/octave-4.0.0/work/octave-4.0.0

and apply the attached "pre-patch"

Then an
ebuild /usr/portage/sci-mathematics/octave/octave-4.0.0.build merge
succeeds.
Comment 1 Mark Wright gentoo-dev 2015-07-08 12:41:52 UTC
Can you please let us know the output of:

equery list virtual/latex-base dev-texlive/texlive-genericrecommended dev-texlive/texlive-metapost sys-apps/texinfo

The build.log might give some hints.

Maybe we need to require higher versions of the doc dependencies, as it
builds fine with doc for me with:

# equery list virtual/latex-base dev-texlive/texlive-genericrecommended dev-texlive/texlive-metapost sys-apps/texinfo
 * Searching for latex-base in virtual ...
[IP-] [  ] virtual/latex-base-1.0:0

 * Searching for texlive-genericrecommended in dev-texlive ...
[IP-] [  ] dev-texlive/texlive-genericrecommended-2014:0

 * Searching for texlive-metapost in dev-texlive ...
[IP-] [  ] dev-texlive/texlive-metapost-2012:0

 * Searching for texinfo in sys-apps ...
[IP-] [  ] sys-apps/texinfo-6.0:0
#
Comment 2 Helmut Jarausch 2015-07-08 13:45:37 UTC
(In reply to Mark Wright from comment #1)
> Can you please let us know the output of:
> 
> equery list virtual/latex-base dev-texlive/texlive-genericrecommended
> dev-texlive/texlive-metapost sys-apps/texinfo
> 
> The build.log might give some hints.
> 
> Maybe we need to require higher versions of the doc dependencies, as it
> builds fine with doc for me with:
> 
> # equery list virtual/latex-base dev-texlive/texlive-genericrecommended
> dev-texlive/texlive-metapost sys-apps/texinfo
>  * Searching for latex-base in virtual ...
> [IP-] [  ] virtual/latex-base-1.0:0
> 
>  * Searching for texlive-genericrecommended in dev-texlive ...
> [IP-] [  ] dev-texlive/texlive-genericrecommended-2014:0
> 
>  * Searching for texlive-metapost in dev-texlive ...
> [IP-] [  ] dev-texlive/texlive-metapost-2012:0
> 
>  * Searching for texinfo in sys-apps ...
> [IP-] [  ] sys-apps/texinfo-6.0:0
> #

-----------------
I have these

 * Searching for latex-base in virtual ...
[IP-] [  ] virtual/latex-base-1.0:0

 * Searching for texlive-genericrecommended in dev-texlive ...
[IP-] [  ] dev-texlive/texlive-genericrecommended-2014:0

 * Searching for texlive-metapost in dev-texlive ...
[IP-] [  ] dev-texlive/texlive-metapost-2014:0

 * Searching for texinfo in sys-apps ...
[IP-] [  ] sys-apps/texinfo-6.0:0
se
Comment 3 Patric Schmitz 2015-09-03 12:04:42 UTC
(In reply to Mark Wright from comment #1)
> Can you please let us know the output of:
> 
> equery list virtual/latex-base dev-texlive/texlive-genericrecommended
> dev-texlive/texlive-metapost sys-apps/texinfo

I can confirm the bug on ~amd64, versions:

[IP-] [  ] virtual/latex-base-1.0:0
[IP-] [  ] dev-texlive/texlive-genericrecommended-2015:0
[IP-] [  ] dev-texlive/texlive-metapost-2015:0
[IP-] [  ] sys-apps/texinfo-6.0:0
Comment 4 Patric Schmitz 2015-09-03 13:32:14 UTC
Ok, it's the texinfo version. Building with texinfo-5.2 succeeds. 

It seems the handling of backslashes in texinfo macros is changing between releases. Looking at the ChangeLog of texinfo and some mailing list discussion, they seem to have serious problems handling the use of backslashes in macro definitions, adding a \fixbackslash, \enablebackslashhack, ...

http://svn.savannah.gnu.org/viewvc/*checkout*/trunk/ChangeLog?root=texinfo

Haven't yet figured out how to fix this properly, I'm on it.
Comment 5 Mark Wright gentoo-dev 2015-09-05 02:42:47 UTC
Created attachment 411014 [details, diff]
untested patch

(In reply to Patric Schmitz from comment #4)
> Ok, it's the texinfo version. Building with texinfo-5.2 succeeds. 
> 
> It seems the handling of backslashes in texinfo macros is changing between
> releases. Looking at the ChangeLog of texinfo and some mailing list
> discussion, they seem to have serious problems handling the use of
> backslashes in macro definitions, adding a \fixbackslash,
> \enablebackslashhack, ...
> 
> http://svn.savannah.gnu.org/viewvc/*checkout*/trunk/ChangeLog?root=texinfo
> 
> Haven't yet figured out how to fix this properly, I'm on it.

Thanks, it seems I am doing something wrong since I can't reproduce the problem
with sys-apps/texinfo-6.0.  This thread says the ancient bundled doc/texinfo.tex
should be removed:

https://lists.gnu.org/archive/html/bug-texinfo/2015-07/msg00035.html

In the hg repo I searched for commits like:

hg log -k tex | less

Which finds these 2:

changeset:   20318:45d7be391982
branch:      stable
user:        Mike Miller <mtmiller@octave.org>
date:        Thu Jul 09 21:45:26 2015 -0400
summary:     build: Use texinfo.tex from gnulib instead of manually imported version

changeset:   20317:2ec049e50ed8
branch:      stable
parent:      20314:3300cb76cccd
user:        Mike Miller <mtmiller@octave.org>
date:        Thu Jul 09 21:44:09 2015 -0400
summary:     doc: Fix backslash characters in docstrings

Which are here:

http://hg.savannah.gnu.org/hgweb/octave/rev/2ec049e50ed8
http://hg.savannah.gnu.org/hgweb/octave/rev/45d7be391982

So the attached patch applies these 2 commits, except we rm doc/texinfo.tex
in the ebuild as that shortens the size of the patch:

argus / # diff -ur /usr/portage/sci-mathematics/octave/octave-4.0.0.ebuild /usr/local/portage/sci-mathematics/octave/octave-4.0.0.ebuild 
--- /usr/portage/sci-mathematics/octave/octave-4.0.0.ebuild     2015-08-26 17:01:02.000000000 +1000
+++ /usr/local/portage/sci-mathematics/octave/octave-4.0.0.ebuild       2015-09-05 12:22:47.699513276 +1000
@@ -100,6 +100,11 @@
                -e 's@A-Za-z0-9@[:alnum:]@g' \
                -e 's@A-Za-z@[:alpha:]@g' \
                libinterp/mkbuiltins || die
+
+       # Fix bug 554226 - sci-mathematics/octave-4.0.0 fails because of a LaTeX error
+       epatch "${FILESDIR}"/${PN}-4.0.0-texinfo.patch
+       rm -f "${S}"/doc/texinfo.tex
+
        autotools-utils_src_prepare
 }
 
argus / # 

It would be neat if you could test this please, thanks!
Comment 6 Mark Wright gentoo-dev 2016-05-02 23:03:11 UTC
Fixed in 4.0.0-r1, the fix is already in 4.0.2.

commit 595225ee11a3f1228d45e112b757e6beda119c78
Author: Mark Wright <gienah@gentoo.org>
Date:   Tue May 3 08:59:40 2016 +1000

    sci-mathematics/octave: Fix bug 554226, thanks to Helmut Jarausch for reporting.
    
    Thanks to Patric Schmitz for helping. The patch from upstream is in versions after 4.0.
    
    Gentoo-bug: 554226