Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 251120

Summary: games-arcade/kamikaze fails to build with gcc 4.3
Product: Gentoo Linux Reporter: Diego Elio Pettenò (RETIRED) <flameeyes>
Component: [OLD] GamesAssignee: Gentoo Games <games>
Status: RESOLVED FIXED    
Severity: normal CC: esigra, gcc-porting
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 198121    
Attachments: Build log
patch for ebuild

Description Diego Elio Pettenò (RETIRED) gentoo-dev 2008-12-16 10:51:33 UTC
Check attached build log.
Comment 1 Diego Elio Pettenò (RETIRED) gentoo-dev 2008-12-16 10:52:02 UTC
Created attachment 175418 [details]
Build log
Comment 2 Tupone Alfredo gentoo-dev 2009-01-21 09:32:34 UTC
I don't find a way to fix it. I need help on that. Maybe I hit a gcc bug?
Comment 3 SpanKY gentoo-dev 2009-01-21 16:48:08 UTC
it fails because someone thought it'd be a great idea to use the -isystem flag when compiling

i think this might fix it (but i dont have a system w/gcc-4.3 and kde-3 to test):
sed -i 's:-isystem:-idirafter:' configure
Comment 4 Holger Ackermann 2009-01-22 18:23:19 UTC
(In reply to comment #3)
> i think this might fix it (but i dont have a system w/gcc-4.3 and kde-3 to
> test):
> sed -i 's:-isystem:-idirafter:' configure
> 

Right! Checked it and works if you add this to the ebuild (see patch, too):

src_unpack() {
        kde_src_unpack
        sed -i 's:-isystem:-idirafter:' "${S}"/configure
}

The package has "sed" as a new dependency, which should be added. Right?
Comment 5 Holger Ackermann 2009-01-22 18:25:22 UTC
Created attachment 179362 [details, diff]
patch for ebuild

As this patch uses sed, sed must be a dependency for this package?
Comment 6 SpanKY gentoo-dev 2009-01-22 21:43:35 UTC
just about everything in the tree uses sed, so no need to list it as a depend
Comment 7 Tupone Alfredo gentoo-dev 2009-01-23 09:24:20 UTC
The fix is now in portage.

What I did before, trying to fix is modifying the source of configure (actually the acinclude.m4) and regenerating the configure with the eautoreconf. That worked during compilation but during the link I obtained something like:

network.o: In function `Network::connect()':
network.cpp:(.text+0x28c): undefined reference to `Network::signalError()'
network.o: In function `Network::~Network()':
network.cpp:(.text+0x2af): undefined reference to `vtable for Network'
network.o: In function `Network::~Network()':
network.cpp:(.text+0x2ef): undefined reference to `vtable for Network'
network.o: In function `Network::~Network()':
network.cpp:(.text+0x32f): undefined reference to `vtable for Network'
network.o: In function `Network::Network()':
network.cpp:(.text+0x384): undefined reference to `vtable for Network'
network.o: In function `Network::Network()':
network.cpp:(.text+0x3d4): undefined reference to `vtable for Network'
network.o: In function `Network::slotDispatch()':
network.cpp:(.text+0xa1): undefined reference to `Network::signalError()'
network.o: In function `Network::slotInput(int)':
network.cpp:(.text+0x185): undefined reference to `Network::signalInput(int)'

Just patching configure works fine.

Thanks for report and solution. :)