From owner-freebsd-emulation Sun Mar 12 18:55: 7 2000 Delivered-To: freebsd-emulation@freebsd.org Received: from c543746-a.valpo1.in.home.com (c543746-a.valpo1.in.home.com [24.17.32.20]) by hub.freebsd.org (Postfix) with ESMTP id E9A5837B789 for ; Sun, 12 Mar 2000 18:55:01 -0800 (PST) (envelope-from dfe@netnitco.net) Received: from netnitco.net (IDENT:dfe@c543746-a [24.17.32.20]) by c543746-a.valpo1.in.home.com (8.9.3/8.9.3) with ESMTP id UAA26925; Sun, 12 Mar 2000 20:50:44 -0600 Message-ID: <38CC5784.111DC5AF@netnitco.net> Date: Sun, 12 Mar 2000 20:50:44 -0600 From: David Elliott X-Mailer: Mozilla 4.72 [en] (X11; U; Linux 2.2.15-2.5.0 i686) X-Accept-Language: en MIME-Version: 1.0 To: Gerald Pfeifer Cc: freebsd-emulation@freebsd.org, wine-devel@winehq.com Subject: Re: I'm afraid you broke Wine on FreeBSD 3.3 References: Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Sender: owner-freebsd-emulation@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.org Gerald Pfeifer wrote: > On Fri, 10 Mar 2000, David Elliott wrote: > > Sorry about that. It is due to the header files having an ifdef linux > > around them. > > Seriously, code like > > #ifdef linux > /* Copy of info from 2.2.x kernel */ > #define SG_MAX_SENSE 16 /* too little, unlikely to change in 2.2.x */ > > really seems quite bad to me! One should never just copy code from system > headers and I really think Alexandre never should have approved something > like that. > > As a matter of fact I have been disappointed several times lately when > Alexandre applied patches that broke Wine on non-Linux platforms due to > typos or "there's nothing apart from Linux" coding. :-( > > What's the purpose of an extremely strict development model if even the > simplest such issues are not caught during patch review? > > Anyway, the patch below brings Wine back into buildworld on FreeBSD 3.4. > > Gerald > Actually, that is not at all Alexandre's fault. While I didn't like the idea of copying kernel headers, that was how it was originally done. Basically all I did was move the Linux headers out of aspi.h and into winescsi.h but then forgot the appropriate ifdef's in the new file source file I created. I'll go ahead and send that patch to wine-patches then. I suppose we could require kernel headers and do a configure test for them and stuff. With a modern distribution you should have kernel headers installed even if you don't have the kernel source. You can do that in RedHat at least, I would assume other distros have similar packaging designs. -Dave To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-emulation" in the body of the message