Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 464168 - dev-lang/nacl-toolchain-newlib-0_p10915 with sys-apps/texinfo-5.1 bfd.texinfo:326: unknown command `colophon' (error repeated with `cygnus' as well)
Summary: dev-lang/nacl-toolchain-newlib-0_p10915 with sys-apps/texinfo-5.1 bfd.texinfo...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Development (show other bugs)
Hardware: AMD64 Linux
: Normal normal (vote)
Assignee: Chromium Project
URL:
Whiteboard:
Keywords:
: 464212 (view as bug list)
Depends on:
Blocks: texinfo-5.1
  Show dependency tree
 
Reported: 2013-04-02 07:48 UTC by Michael Hampicke
Modified: 2013-04-03 18:27 UTC (History)
3 users (show)

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


Attachments
dev-lang/nacl-toolchain-newlib-0_p10915 build log (build.log,95.99 KB, text/plain)
2013-04-02 07:48 UTC, Michael Hampicke
Details
binutils-texinfo-r0.patch (binutils-texinfo-r0.patch,1.84 KB, patch)
2013-04-03 14:04 UTC, Mike Gilbert
Details | Diff
gcc-texinfo-r0.patch (gcc-texinfo-r0.patch,8.48 KB, patch)
2013-04-03 14:05 UTC, Mike Gilbert
Details | Diff
gcc-texinfo-r0.patch.out (gcc-texinfo-r0.patch.out,13.09 KB, text/plain)
2013-04-03 14:06 UTC, Mike Gilbert
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Hampicke 2013-04-02 07:48:02 UTC
dev-lang/nacl-toolchain-newlib-0_p10915 fails to build on my machine. I attached a build log

Reproducible: Always

Steps to Reproduce:
1. emerge dev-lang/nacl-toolchain-newlib-0_p10915
2.
3.
Actual Results:  
failed build

Expected Results:  
complete merge
Comment 1 Michael Hampicke 2013-04-02 07:48:23 UTC
Created attachment 344046 [details]
dev-lang/nacl-toolchain-newlib-0_p10915 build log
Comment 2 megabaks 2013-04-02 09:14:33 UTC
subj incompatible with  sys-apps/texinfo-5.1
downgrade it to 4.13-r2
/sorry my bad english
Comment 3 funeralismatic 2013-04-02 09:48:45 UTC
before downgrade sys-apps/texinfo to 4.13-r2 all works
Comment 4 funeralismatic 2013-04-02 09:53:48 UTC
(In reply to comment #3)
> before downgrade sys-apps/texinfo to 4.13-r2 all works

s/defore/after/
Comment 5 Tom Wijsman (TomWij) (RETIRED) gentoo-dev 2013-04-02 10:57:38 UTC
Please comment with the output of `emerge --info`, thank you.
Comment 6 megabaks 2013-04-02 13:08:33 UTC
1. emerge --info unneeded
2. just fix deps
3. arch independed
Comment 7 Mike Gilbert gentoo-dev 2013-04-02 14:51:14 UTC
Given that sys-apps/texinfo is not slotted, depending on an old version seems like a bad idea.

Quoting vapier:
> any core package that includes info pages should not be regenerating them
> hence should not need texinfo.  i've applied a fix already for gcc to do just
> that.

We probably need to do something similar here.
Comment 8 Mike Gilbert gentoo-dev 2013-04-02 18:11:16 UTC
*** Bug 464212 has been marked as a duplicate of this bug. ***
Comment 9 Paweł Hajdan, Jr. (RETIRED) gentoo-dev 2013-04-02 22:17:07 UTC
(In reply to comment #7)
> Quoting vapier:
> > any core package that includes info pages should not be regenerating them
> > hence should not need texinfo.  i've applied a fix already for gcc to do just
> > that.
> 
> We probably need to do something similar here.

Sounds good. Do we know where that patch is?
Comment 10 Mike Gilbert gentoo-dev 2013-04-02 23:19:45 UTC
(In reply to comment #9)
> Sounds good. Do we know where that patch is?

Well, in this case it is binutils rather than gcc that is regenerating an info file.

If you look at naclbinutils-2.20.1-r10915.patch, you can see that it is modifying binutils/doc/binutils.texi. If we drop that patch hunk, I think that will resolve the issue here.

The patch documents the --enable-deterministic-archives option.
Comment 11 Mike Gilbert gentoo-dev 2013-04-02 23:22:50 UTC
We may run into a similar issue with gcc: naclgcc-4.4.3-r10915.patch modifies gcc/doc/invoke.texi.
Comment 12 Mike Gilbert gentoo-dev 2013-04-02 23:37:16 UTC
Upon testing, I made an interesting discovery: the info files get regenerated any time you do an out-of-source build, regardless of any patching.

I had hoped to trick make by touching the info files after we apply patches, but that won't work since we build outside the source directory.
Comment 13 Mike Gilbert gentoo-dev 2013-04-03 00:20:59 UTC
My next idea was to copy the pre-generated info files into the build directory before compiling, but the build directory gets removed when you run make all.

I think I'm out of bright ideas for the time being.
Comment 14 Mike Gilbert gentoo-dev 2013-04-03 01:01:16 UTC
We could backport this commit.

http://sourceware.org/git/?p=binutils.git;a=commit;h=935f85422863b42b6fbca30885885e3fa28eea36
Comment 15 Mike Gilbert gentoo-dev 2013-04-03 01:53:30 UTC
Ok, the binutils commit is an easy backport; just remove the Changelog and copyright updates.

After that is fixed, I now get this failure from gcc:

        makeinfo --split-size=5000000 --no-split -I . -I /tmp/portage/dev-lang/nacl-toolchain-newlib-0_p10915/work/SRC/gcc/gcc/doc \
                -I /tmp/portage/dev-lang/nacl-toolchain-newlib-0_p10915/work/SRC/gcc/gcc/doc/include -o doc/cpp.info /tmp/portage/dev-lang/nacl-toolchain-newlib-0_p10915/work/SRC/gcc/gcc/doc/cpp.texi; \
fi
/tmp/portage/dev-lang/nacl-toolchain-newlib-0_p10915/work/SRC/gcc/gcc/doc/cppopts.texi:761: @itemx must follow @item
make[3]: *** [doc/cpp.info] Error 1
make[3]: Leaving directory `/tmp/portage/dev-lang/nacl-toolchain-newlib-0_p10915/work/BUILD/build-pregcc-x86_64-nacl/gcc'
make[2]: *** [all-gcc] Error 2
make[2]: Leaving directory `/tmp/portage/dev-lang/nacl-toolchain-newlib-0_p10915/work/BUILD/build-pregcc-x86_64-nacl'
make[1]: *** [BUILD/stamp-x86_64-nacl-pregcc] Error 2


That is probably fixed by this commit, which will be more difficult to backport.

http://gcc.gnu.org/git/?p=gcc.git;a=commit;h=85ad240e47dcdc1bf20ef6a95e921ac1f766d995
Comment 16 Paweł Hajdan, Jr. (RETIRED) gentoo-dev 2013-04-03 03:03:16 UTC
(In reply to comment #15)
> Ok, the binutils commit is an easy backport; just remove the Changelog and
> copyright updates.
> 
> [...]
> 
> That is probably fixed by this commit, which will be more difficult to
> backport.
> 
> http://gcc.gnu.org/git/?p=gcc.git;a=commit;
> h=85ad240e47dcdc1bf20ef6a95e921ac1f766d995

Wow, good job finding these commits! What's the difficulty with the second one (just trying to understand)?

Another option I was considering was disabling info files processing completely, if that's reasonably easy to do. nacl-toolchain-newlib is only used for chromium build, these info files wouldn't be very useful anyway (of course if that's the only way to fix the build it makes sense).
Comment 17 Mike Gilbert gentoo-dev 2013-04-03 14:02:12 UTC
(In reply to comment #16)
> Wow, good job finding these commits! What's the difficulty with the second
> one (just trying to understand)?

9 of 13 patch hunks fail. It will take some work to check each failure against the gcc-4.4.3 source.

> Another option I was considering was disabling info files processing
> completely, if that's reasonably easy to do. nacl-toolchain-newlib is only
> used for chromium build, these info files wouldn't be very useful anyway (of
> course if that's the only way to fix the build it makes sense).

I think we would need to patch a bunch of Makefiles. I don't see an obvious way to disable them otherwise.
Comment 18 Mike Gilbert gentoo-dev 2013-04-03 14:04:35 UTC
Created attachment 344230 [details, diff]
binutils-texinfo-r0.patch
Comment 19 Mike Gilbert gentoo-dev 2013-04-03 14:05:24 UTC
Created attachment 344232 [details, diff]
gcc-texinfo-r0.patch

This one needs work
Comment 20 Mike Gilbert gentoo-dev 2013-04-03 14:06:15 UTC
Created attachment 344234 [details]
gcc-texinfo-r0.patch.out
Comment 21 Mike Gilbert gentoo-dev 2013-04-03 17:43:47 UTC
I'm working on fixing the gcc patch.
Comment 22 Mike Gilbert gentoo-dev 2013-04-03 18:27:42 UTC
Ok, should be working now.

+  03 Apr 2013; Mike Gilbert <floppym@gentoo.org>
+  +files/binutils-texinfo-r0.patch, +files/gcc-texinfo-r0.patch,
+  nacl-toolchain-newlib-0_p10915.ebuild:
+  Add patches to fix compatibility with texinfo-5.1. Bug 464168.