Bug 306761

Summary:	ati-drivers-10.2 does not compile with gentoo-sources-2.6.33
Product:	Gentoo Linux	Reporter:	Sebastian Lüttich <order+gentoo>
Component:	New packages	Assignee:	Gentoo Linux bug wranglers <bug-wranglers>
Status:	VERIFIED DUPLICATE
Severity:	normal	CC:	attilaolah, boltomli, jarausch
Priority:	High
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	patch for ati-drivers-10.2.ebuild needed patch to compile ati-drivers

Description Sebastian Lüttich 2010-02-25 08:31:36 UTC

ati-drivers doesn't compile against 2.6.33. I changed the old patch.

2.6.33 puts autoconf.h and utsrelease.h in include/generated/. To change this in the source of ati-drivers would be to much, therefor I put a symbolic link from include/generated/[autoconf|utsrelease].h to include/linux/ in the kernel source tree.

Reproducible: Always

Comment 1 Sebastian Lüttich 2010-02-25 08:36:06 UTC

Created attachment 221057 [details, diff]
patch for ati-drivers-10.2.ebuild

patch -p0 < ati-drivers-10.2.ebuild.patch

Comment 2 Sebastian Lüttich 2010-02-25 08:36:55 UTC

Created attachment 221059 [details, diff]
needed patch to compile ati-drivers

put this file into $FILESDIR/kernel/

Comment 3 Rémi Cardona (RETIRED) gentoo-dev

2010-02-26 06:36:47 UTC


*** This bug has been marked as a duplicate of bug 306771 ***

Comment 4 Helmut Jarausch 2010-02-26 16:32:33 UTC

(In reply to comment #2)
> Created an attachment (id=221059) [details]
> needed patch to compile ati-drivers
> 
> put this file into $FILESDIR/kernel/
> 

I fear that doesn't suffice here - from the log
Applying 2.6.33-10.2-fix_compilation.patch
...
common/lib/modules/fglrx/build_mod/2.6.x/firegl_public.c:31:28: error: linux/autoconf.h: No such file or directory
...
common/lib/modules/fglrx/build_mod/2.6.x/firegl_public.c:166:30: error: linux/utsrelease.h: No such file or directory

Comment 5 Sebastian Lüttich 2010-02-26 16:51:16 UTC

(In reply to comment #4)
> (In reply to comment #2)
> > Created an attachment (id=221059) [details] [details]
> > needed patch to compile ati-drivers

> I fear that doesn't suffice here - from the log
> Applying 2.6.33-10.2-fix_compilation.patch

Read the source Luke!

|2.6.33 puts autoconf.h and utsrelease.h in include/generated/. To change this
|in the source of ati-drivers would be to much, therefor I put a symbolic link
|from include/generated/[autoconf|utsrelease].h to include/linux/ in the kernel
|source tree.

from my initial post.

Happy gl-ing, 

Sebastian

Comment 6 Attila Oláh 2010-02-26 20:26:07 UTC

For me the attached patch only solves half of the problem. I still get these errors:

/var/tmp/portage/x11-drivers/ati-drivers-10.2/work/common/lib/modules/fglrx/build_mod/2.6.x/firegl_public.c: In function '__ke__cmpxchg':                                             
/var/tmp/portage/x11-drivers/ati-drivers-10.2/work/common/lib/modules/fglrx/build_mod/2.6.x/firegl_public.c:1473: error: variable or field '__ret' declared void                      
/var/tmp/portage/x11-drivers/ati-drivers-10.2/work/common/lib/modules/fglrx/build_mod/2.6.x/firegl_public.c:1473: error: variable or field '__old' declared void                      
/var/tmp/portage/x11-drivers/ati-drivers-10.2/work/common/lib/modules/fglrx/build_mod/2.6.x/firegl_public.c:1473: error: variable or field '__new' declared void

Comment 7 Attila Oláh 2010-02-27 00:00:22 UTC

Correction:

The patch worked great. It looks like I applied it on a wrong copy; silly me...