First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 124993
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Gentoo Games <games@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Patrizio Bassi <hetfield666@gmail.com>
Add CC:
CC:
Remove selected CCs
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
0.5.5-gcc4.patch 0.5.5-gcc4.patch patch Charles Pence 2006-03-28 12:23 0000 10.66 KB Details | Diff
0.5.5-gcc4.patch 0.5.5-gcc4.patch (v2) patch Charles Pence 2006-03-29 17:01 0000 11.61 KB Details | Diff
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 124993 depends on: Show dependency tree
Show dependency graph
Bug 124993 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)







View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug


Description:   Opened: 2006-03-04 11:05 0000
there are a lot of errors i manually fixed due to static/non-static.

ok...easy to fix...but at the end that's:

if /bin/sh ../../libtool --mode=compile i686-pc-linux-gnu-gcc -DHAVE_CONFIG_H
-I. -I. -I../../include -I../../include   -prefer-pic -O3 -mtune=pentium3
-march=pentium3 -pipe -fomit-frame-pointer -mmmx -msse -ffast-math -pipe -Wall
-fno-common -MT gib_classes.lo -MD -MP -MF ".deps/gib_classes.Tpo" -c -o
gib_classes.lo gib_classes.c; \
then mv -f ".deps/gib_classes.Tpo" ".deps/gib_classes.Plo"; else rm -f
".deps/gib_classes.Tpo"; exit 1; fi
 i686-pc-linux-gnu-gcc -DHAVE_CONFIG_H -I. -I. -I../../include -I../../include
-O3 -mtune=pentium3 -march=pentium3 -pipe -fomit-frame-pointer -mmmx -msse
-ffast-math -pipe -Wall -fno-common -MT gib_classes.lo -MD -MP -MF
.deps/gib_classes.Tpo -c gib_classes.c  -fPIC -DPIC -o .libs/gib_classes.o
gib_classes.c: In function 'Object_Class_Children_f':
gib_classes.c:197: error: invalid storage class for function 'iterator'
gib_classes.c: In function 'GIB_Classes_Build_Scripted':
gib_classes.c:615: error: invalid storage class for function 'mtabfree'
gib_classes.c:621: error: invalid storage class for function 'fname'
make[2]: *** [gib_classes.lo] Error 1
make[2]: Leaving directory
`/var/tmp/portage/quakeforge-0.5.5/work/quakeforge-0.5.5/libs/gib'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory
`/var/tmp/portage/quakeforge-0.5.5/work/quakeforge-0.5.5/libs'
make: *** [all-recursive] Error 1


i've no idea about that.

------- Comment #1 From Charles Pence 2006-03-28 12:01:08 0000 -------
I'm getting the same issue, and will provide a patch momentarily.

------- Comment #2 From Charles Pence 2006-03-28 12:23:29 0000 -------
This patch allows it to build with GCC4.  I'm currently in the middle of a
full-system re-emerge with GCC4 (hence how I discovered the build), so the
patch is as-of-yet untested.  I'll be able to test it this evening, and will
report back my results.

------- Comment #3 From Charles Pence 2006-03-28 12:23:59 0000 -------
Created an attachment (id=83322) [edit]
0.5.5-gcc4.patch

------- Comment #4 From Charles Pence 2006-03-28 20:09:48 0000 -------
That is confirmed, the patch produces a functional quakeforge on gcc 4.0.3.

------- Comment #5 From Patrizio Bassi 2006-03-29 00:44:12 0000 -------
charles seems not completly fixed here:

i686-pc-linux-gnu-gcc -DHAVE_CONFIG_H -I. -I. -I../../../include
-I../../../include -O3 -mtune=pentium3 -march=pentium3 -pipe
-fomit-frame-pointer -mmmx -msse -ffast-math -pipe -Wall -fno-common -MT
libvid_x11_la-context_x11.lo -MD -MP -MF .deps/libvid_x11_la-context_x11.Tpo -c
context_x11.c -o .libs/libvid_x11_la-context_x11.o
context_x11.c:100: error: static declaration of 'x_gamma' follows non-static
declaration
../../../include/context_x11.h:54: error: previous declaration of 'x_gamma' was
here
make[3]: *** [libvid_x11_la-context_x11.lo] Error 1
make[3]: Leaving directory
`/var/tmp/portage/quakeforge-0.5.5/work/quakeforge-0.5.5/libs/video/targets'
make[2]: *** [all-recursive] Error 1

....

vid_x11.c:89: error: static declaration of 'doShm' follows non-static
declaration
../../../include/context_x11.h:59: error: previous declaration of 'doShm' was
here


i'm running gcc 4.1.0. can you update patch to 4.1.0 compatibility?

------- Comment #6 From Charles Pence 2006-03-29 06:36:12 0000 -------
My mistake!  The patch I submitted will compile the SDL, SDL32, and SGL
clients---but apparently not the X11 clients.  I'll give it another go tonight
and enable building all the clients.

------- Comment #7 From Patrizio Bassi 2006-03-29 06:56:13 0000 -------
my use flags are:
USE="X alsa dga fbcon ncurses opengl oss sdl svga vorbis xmms xv zlib -3dfx
-cdinstall -debug -ipv6" 0 kB


however your patch seems just to remove static statement.
may be ok...or may not.

check a note in bug http://bugs.gentoo.org/show_bug.cgi?id=123616

ok...that's a static/extern change...but....worth a look.

------- Comment #8 From Charles Pence 2006-03-29 07:17:38 0000 -------
I don't think that's going to be a problem here.  What appears to be happening
in this case is that the QuakeForge developers had a lot of variables that were
supposedly global API vars, and included in header files, but then declared
static upon declaration in the C files.  Not a good way to fly---and gcc4
recognizes that they're confused.  The "static" implies they can't be accessed
outside the individual C file, but then they're marked global...

I'll try to get a build with all USE flags enabled for QF tonight (egh, means I
have to install svgalib...).

------- Comment #9 From Charles Pence 2006-03-29 07:20:35 0000 -------
Oh, and I forgot to add---if you just want to play quake, you can

USE="alsa ncurses opengl sdl vorbis mms zlib -3dfx -X -cdinstall -debug -dga
-fbcon -ipv6 -oss -svga -xv" emerge quakeforge

and it should build and run the sgl, sdl, and sdl32 clients.

------- Comment #10 From Charles Pence 2006-03-29 17:01:21 0000 -------
Got it, here's an updated patch.

------- Comment #11 From Charles Pence 2006-03-29 17:01:46 0000 -------
Created an attachment (id=83406) [edit]
0.5.5-gcc4.patch (v2)

------- Comment #12 From Patrizio Bassi 2006-03-30 00:20:12 0000 -------
works perfectly.

please add to portage and close this bug

------- Comment #13 From Tupone Alfredo 2006-05-06 15:12:54 0000 -------
Patch applied to portage, thanks.

First Last Prev Next    No search results available      Search page      Enter new bug