Cedega CVS compile errors

trawler · 10-17-2005, 05:47 PM

I've been trying to compile the Cedega CVS, and came across these errors:

Code:

gcc -MMD -c  -I. -I. -I../../include -I../../include  -g -O2 -Wall -mpreferred-s
tack-boundary=2 -fno-keep-static-consts -D__const=const -fno-strict-aliasing -D_
_int8=char -D__int16=short -D__int32=int "-D__int64=long long" -D__WINE__ -D_REE                                            NTRANT -I/usr/X11R6/include -o readres.o readres.c
newstruc.c: In function �handle_ani_list�:
newstruc.c:740: error: invalid lvalue in increment
newstruc.c: In function �new_ani_curico�:
newstruc.c:851: error: invalid lvalue in increment
make[2]: *** [newstruc.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: *** Waiting for unfinished jobs....
make[2]: *** Waiting for unfinished jobs....
make[2]: Leaving directory `/home/karen/.WineCVS/sources/cvscedega/winex/tools/w                                            rc'
make[1]: *** [wrc] Error 2
make[1]: Leaving directory `/home/karen/.WineCVS/sources/cvscedega/winex/tools'
make: *** [tools] Error 2

I've understood it had something to do with the version of flex i currently have installed on my Fedora FC4 machine (flex-2.5.4a-34) and that installing the older version of flex-2.5.4a-7 should solve the problem.

However, it didn't, and i'm still getting these errors.
What do i need to do next?

Thanx

trawler · 10-17-2005, 06:52 PM

These are the lines in newstruc.c, as mentioned in the error:
(newstruc.c: In function 'handle_ani_list':
newstruc.c:740: error: invalid lvalue in incremen)

Code:

static void handle_ani_list(riff_tag_t *lst, enum res_e type, int isswapped)
{
	riff_tag_t *rtp = lst+1;	/* Skip the "LIST" tag */

	while((char *)rtp < (char *)lst + lst->size + sizeof(*lst))
	{
		if(!memcmp(rtp->tag, info, sizeof(info)))
		{
			rtp = (riff_tag_t *)(((char *)rtp) + 4);
		}
		else if(!memcmp(rtp->tag, inam, sizeof(inam)))
		{
			/* Ignore the icon/cursor name; its a string */
			rtp = NEXT_TAG(rtp);
		}
		else if(!memcmp(rtp->tag, iart, sizeof(iart)))
		{
			/* Ignore the author's name; its a string */
			rtp = NEXT_TAG(rtp);
		}
		else if(!memcmp(rtp->tag, fram, sizeof(fram)))
		{
			/* This should be followed by "icon"s, but we
			 * simply ignore this because it is pure
			 * non-information.
			 */
			rtp = (riff_tag_t *)(((char *)rtp) + 4);
		}
		else if(!memcmp(rtp->tag, icon, sizeof(icon)))
		{
			handle_ani_icon(rtp, type, isswapped);
			rtp = NEXT_TAG(rtp);
		}
		else
			internal_error(__FILE__, __LINE__, "Unknown tag \"%c%c%c%c\" in RIFF file",
				       isprint(rtp->tag[0]) ? rtp->tag[0] : '.',
				       isprint(rtp->tag[1]) ? rtp->tag[1] : '.',
				       isprint(rtp->tag[2]) ? rtp->tag[2] : '.',
				       isprint(rtp->tag[3]) ? rtp->tag[3] : '.');

		/* FIXME: This relies in sizeof(DWORD) == sizeof(pointer_type) */
		if((DWORD)rtp & 1)
			((char *)rtp)++; <--- line 740
	}
}

Hope it'll help any of you who might know what the problem is...

brodskie · 10-29-2005, 05:46 AM

I have the exact same problem. Do you have GCC 4.0 installed?
I know the solution, but it involves editing hundreds of lines of code. Some lines need to be re-arranged to conform to stricter guidelines apparently required by GCC 4.0. If you reply and tell me that you're interested I'll summarize every change, but again, it's a lot of work! I'd complain to transgaming anyway.

trawler · 10-29-2005, 06:13 AM

I do have gcc4.. and after TONS of research i realized apparently it was what giving me the hard time...
Eventually i caved... and installed the ready made cedega rpm, which works great

Im doubt if transgaming will do anything about it though...

brodskie · 10-29-2005, 06:24 AM

For anyone else who has this problem. I just came up with a solution for compiling. Delete the CVS source code, install GCC 3.4, then type
CC='gcc-3.4' sh WineCVS.sh

This will force the script to use GCC 3.4, so everything will be fine.

oni9 · 10-31-2005, 07:55 PM

have you try to change line 740

from

Code:

/* FIXME: This relies in sizeof(DWORD) == sizeof(pointer_type) */
if((DWORD)rtp & 1)
  ((char *)rtp)++; // line 740 here

to

Code:

/* FIXME: This relies in sizeof(DWORD) == sizeof(pointer_type) */
if((DWORD)rtp & 1)
  rpt = ((char *)rtp)+1; // line 740 here

brodskie · 11-01-2005, 06:36 AM

yes I have.

Once you do that to the two problematic lines, when you recompile, you are comfronted with a file containing hundreds of these errors. They all can be fixed the same way. Maybe a script can do it?