Summary: | sci-mathematics/octave fails to build with gcc-4.6 | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Diego Elio Pettenò (RETIRED) <flameeyes> |
Component: | New packages | Assignee: | Gentoo Science Mathematics related packages <sci-mathematics> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | steffen |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 346809 | ||
Attachments: |
Build log
gcc-4.6 patch |
Description
Diego Elio Pettenò (RETIRED)
2011-04-15 16:01:47 UTC
Seems to be gcc-4.6 related oct-alloc.cc:32:26: error: ‘size_t’ was not declared in this scope Indeed it is, thanks for noticing. Created attachment 274595 [details, diff] gcc-4.6 patch (This patch should work for any gcc.) Fix 1: the aforementioned missing size_t. I hate it when people don't include stddef.h when they should. Fix 2: fixed the obscure abs() template definition (the error message said something about removing the 'inline' keyword in the macro, which I didn't understand, but it was fine to just remove the macro and rely on the template) Here's the short ebuild change: --- /usr/portage/sci-mathematics/octave/octave-3.4.0.ebuild 2011-03-20 15:31:24.000000000 -0500 +++ octave-3.4.0.ebuild 2011-05-25 06:24:48.539735176 -0500 @@ -46,6 +46,10 @@ dev-util/gperf dev-util/pkgconfig" +src_prepare() { + epatch "${FILESDIR}/${P}-gcc46.patch" +} + src_configure() { # hdf5 disabled because not really useful (bug #299876) econf \ Ha, I just noticed this: http://savannah.gnu.org/bugs/?func=detailitem&item_id=32387 Basically the same but I liked my second fix better :) bluh Here's the correct ebuild patch: --- /usr/portage/sci-mathematics/octave/octave-3.4.0.ebuild 2011-03-20 15:31:24.000000000 -0500 +++ octave-3.4.0.ebuild 2011-05-25 07:08:33.821500250 -0500 @@ -3,7 +3,7 @@ # $Header: /var/cvsroot/gentoo-x86/sci-mathematics/octave/octave-3.4.0.ebuild,v 1.3 2011/03/20 20:01:35 jlec Exp $ EAPI="2" -inherit multilib +inherit eutils multilib DESCRIPTION="High-level interactive language for numerical computations" LICENSE="GPL-3" @@ -46,6 +46,10 @@ dev-util/gperf dev-util/pkgconfig" +src_prepare() { + epatch "${FILESDIR}/${P}-gcc46.patch" +} + src_configure() { # hdf5 disabled because not really useful (bug #299876) econf \ I included the ebuild patch before precisely because I hate editing ebuilds. I should mention that when it gets into the doc/interpreter directory in the install phase, it sometimes fails. Maybe a parallel build issue? I have MAKEOPTS=-j5 All fixed. Thanks. |