81507 – davfs2 works on amd64 with a small patch

Bug 81507 - davfs2 works on amd64 with a small patch

Summary: davfs2 works on amd64 with a small patch

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	AMD64 Linux

Importance:	High enhancement (vote)
Assignee:	AMD64 Project

URL:
Whiteboard:
Keywords:	Inclusion

Depends on:
Blocks:

Reported:	2005-02-10 08:11 UTC by Jörg Gollnick
Modified:	2005-07-12 08:33 UTC (History)
CC List:	2 users (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
patch to inlude <linux/compiler.h> (include_compiler.patch,655 bytes, patch) 2005-02-10 08:15 UTC, Jörg Gollnick	Details \| Diff
ebuild for amd64 (davfs2-0.2.3-r1.ebuild,1.26 KB, text/plain) 2005-02-10 08:16 UTC, Jörg Gollnick	Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Jörg Gollnick 2005-02-10 08:11:28 UTC

davfs2 failes to compile

x86_64-pc-linux-gnu-gcc -DHAVE_CONFIG_H  -I/var/tmp/portage/davfs2-0.2.3-r1/work/davfs2-0.2.3 -I./lib  -I./src  -O2 -I/usr/include/neon -DNEON_ZLIB -DNEON_SSL -c src/davfsd.c -o src/davfsd.o
x86_64-pc-linux-gnu-gcc -DHAVE_CONFIG_H  -I/var/tmp/portage/davfs2-0.2.3-r1/work/davfs2-0.2.3 -I./lib  -I./src  -O2 -I/usr/include/neon -DNEON_ZLIB -DNEON_SSL -c src/mount.c -o src/mount.o
In file included from src/davfsd.c:41:
/usr/include/linux/coda.h:762: error: variable or field `__user' declared void
/usr/include/linux/coda.h:762: error: parse error before '*' token
/usr/include/linux/coda.h:763: error: parse error before '*' token
/usr/include/linux/coda.h:766: error: parse error before '}' token
/usr/include/linux/coda.h:769: error: parse error before '*' token
/usr/include/linux/coda.h:772: error: parse error before '}' token
/usr/include/linux/coda.h:771: error: storage size of `vi' isn't known
make: *** [src/davfsd.o] Error 1
make: *** Waiting for unfinished jobs....
In file included from src/mount.c:38:
/usr/include/linux/coda.h:762: error: variable or field `__user' declared void
/usr/include/linux/coda.h:762: error: parse error before '*' token
/usr/include/linux/coda.h:763: error: parse error before '*' token
/usr/include/linux/coda.h:766: error: parse error before '}' token
/usr/include/linux/coda.h:769: error: parse error before '*' token
/usr/include/linux/coda.h:772: error: parse error before '}' token
/usr/include/linux/coda.h:771: error: storage size of `vi' isn't known
make: *** [src/mount.o] Error 1

Source of this Problem is different sigcontext.h for x86 and other architectures. On x86 it includes linux/compiler.h as side effect of including <asm/sigcontext.h>.

Comment 1 Jörg Gollnick 2005-02-10 08:15:26 UTC

Created attachment 50921 [details, diff]
patch to inlude <linux/compiler.h>

Comment 2 Jörg Gollnick 2005-02-10 08:16:31 UTC

Created attachment 50922 [details]
ebuild for amd64

Comment 3 Jannick Kuhr 2005-02-26 10:17:20 UTC

The corrected ebuild works fine for me! Please add to portage.

Comment 4 Simon Stelling (RETIRED) gentoo-dev

2005-07-12 08:33:27 UTC

the patch should not be needed!?

asm/sigcontext.h is a wrapper for either asm-x86_64/sigcontext.h or
asm-i386/sigcontext.h, depending on the ABI, but both inlcude <linux/compiler.h>

nevertheless, i'm able to reproduce the error, and this patch seems to fix it..

included patch and marked ~amd64