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

Bug 59845

Summary: Hexcurse doesn't compile with gcc 3.4
Product: Gentoo Linux Reporter: Aard Keimpema <a.keimpema>
Component: Current packagesAssignee: SpanKY <vapier>
Status: RESOLVED FIXED    
Severity: normal    
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: updated ebuild
Patch to fix compiltion for gcc 3.4

Description Aard Keimpema 2004-08-09 03:41:26 UTC
Upon emerging hexcurse the following error occurs:

gcc -DPACKAGE_NAME=\"\" -DPACKAGE_TARNAME=\"\" -DPACKAGE_VERSION=\"\" -DPACKAGE_STRING=\"\" -DPACKAGE_BUGREPORT=\"\" -DPACKAGE=\"hexcurse\" -DVERSION=\"1.55\" -DHAVE_LIBNCURSES=1 -DSTDC_HEADERS=1 -DHAVE_SYS_TYPES_H=1 -DHAVE_SYS_STAT_H=1 -DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1 -DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1 -DHAVE_INTTYPES_H=1 -DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -DHAVE_UNISTD_H=1 -DHAVE_STRING_H=1 -DHAVE_STRINGS_H=1 -DRETSIGTYPE=void  -I. -I. -I../include    -march=athlon-xp -O2 -fomit-frame-pointer -fweb -pipe -c `test -f acceptch.c || echo './'`acceptch.c
getopt.c:40: error: conflicting types for 'alloca'
getopt.c:40: error: conflicting types for 'alloca'
make[1]: *** [getopt.o] Error 1


Reproducible: Always
Steps to Reproduce:
Comment 1 Aard Keimpema 2004-08-09 03:51:20 UTC
The problem is caused by this code portion in getopt.c:
--------------------
#ifdef _SGIAPI
#include <alloca.h>
#else
char *alloca ();
#endif
-------------------
By simply removing the redefinition of alloca the problem is solved. I've attached  a patch for this and an updated ebuild. I've tested the ebuild using gcc-3.4.1 and 
gcc-3.3.4
Comment 2 Aard Keimpema 2004-08-09 04:01:09 UTC
Created attachment 37090 [details]
updated ebuild
Comment 3 Aard Keimpema 2004-08-09 04:02:29 UTC
Created attachment 37091 [details, diff]
Patch to fix compiltion for gcc 3.4
Comment 4 SpanKY gentoo-dev 2004-08-09 17:39:28 UTC
awesome, thanks for the patch, added to cvs