The current sys-kernel/linux-headers-2.6.3-r1 ebuild includes several patches which modify files in 'include/asm-i386'. Several of the bugs that these fix are also present in the equivilent files in 'include/asm-x86_64'. This is a patch that I've made to apply the fixes from 'linux-headers-2.6.3-appCompat.patch' and 'linux-headers-2.6.3-strict-ansi-fix.patch' to the x86_64 asm files. (I was inspired to do this based upon a particular error that I was receiveing from several packages when compiling them. That error was: /usr/include/linux/byteorder/swab.h:133: error: syntax error before "__u16" /usr/include/linux/byteorder/swab.h:146: error: syntax error before "__u32" This patch fixes this error, and possibly others.) Reproducible: Always Steps to Reproduce:
Created attachment 26555 [details, diff] Adds fixes for x86_64 asm files, based on fixes in linux-headers-2.6.3-strict-ansi-fix.patch and linux-headers-2.6.3-appCompat.patch
Comment on attachment 26555 [details, diff] Adds fixes for x86_64 asm files, based on fixes in linux-headers-2.6.3-strict-ansi-fix.patch and linux-headers-2.6.3-appCompat.patch (obsolete - see updated version)
Created attachment 26559 [details, diff] linux-headers-2.6.3-x86_64-strict-ansi-fix-and-appCompat.patch (updated version of previous patch I submitted; adds fix for another file)
Created attachment 26614 [details, diff] linux-headers-2.6.3-x86_64-strict-ansi-fix-and-appCompat.patch (This should hopefully be the 'final' version of this patch. I forgot to adjust a few paths in the previously posted ones.)
All good; can I just ask whether there is a purpose for the <linux/byteorder/swab.h> patch since it's already in: if you were sourcing your patch from the original patch release that didn't have it, that's understandable; I'll just remove that hunk when I commit it in... http://www.gentoo.org/cgi-bin/viewcvs.cgi/sys-kernel/linux-headers/files/linux-headers-2.6.3-strict-ansi-fix.patch?r1=1.2&r2=1.3
Oops, I was using the earlier version of the "linux-headers-2.6.3-strict-ansi-fix.patch" patch. Just remove the <linux/byteorder/swab.h> diff from my patch. Thanks :-)
Thanks, it's now in CVS [ I just appended your changes minus the <byteorder/swab.h> patch onto the existing patch ]. Should be in Portage within an hour. If there are more things which need adding in please feel welcome to assign patches / compiler errors / suggestions straight on to me.
