From owner-cvs-all Mon Apr 8 23:42:48 2002 Delivered-To: cvs-all@freebsd.org Received: from elvis.mu.org (elvis.mu.org [192.203.228.196]) by hub.freebsd.org (Postfix) with ESMTP id A275D37B41A; Mon, 8 Apr 2002 23:42:42 -0700 (PDT) Received: by elvis.mu.org (Postfix, from userid 1192) id 82685AE211; Mon, 8 Apr 2002 23:42:42 -0700 (PDT) Date: Mon, 8 Apr 2002 23:42:42 -0700 From: Alfred Perlstein To: "M. Warner Losh" Cc: jhb@FreeBSD.org, kris@obsecurity.org, obrien@FreeBSD.org, cvs-all@FreeBSD.org, cvs-committers@FreeBSD.org Subject: Re: cvs commit: src/sys/sys ioccom.h Message-ID: <20020409064242.GJ93885@elvis.mu.org> References: <20020408210151.B18407@xor.obsecurity.org> <20020408.222429.22925353.imp@village.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20020408.222429.22925353.imp@village.org> User-Agent: Mutt/1.3.27i Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG * M. Warner Losh [020408 21:24] wrote: > In message: > John Baldwin writes: > : No, I already said that I reverted the ()'s locally and it still failed to > : compile. Yes, I was editing /usr/include/sys/ioccom.h and not > : /sys/sys/ioccom.h. I also tried putting __P() back in, but it didn't build > : then either. I'm wondering if the pollution in screen's osdef.h is new? > > Ah, the screen port. Lemme test this locally.... Hmmm, the new > version of iocomm doesn't work either: > > osdef.h:81: conflicting types for `ioctl' > /usr/include/sys/ioccom.h:75: previous declaration of `ioctl' > > The real problem is at osdef.h:81: > extern int ioctl __P((int, int, char *)); > > Note the two problems with this: > 1) int doesn't match unsgined long > 2) char * doesn't match ... > > screen needs to be fixed. here's a real patch: > > --- osdef.h.in.orig Mon Apr 8 22:22:18 2002 > +++ osdef.h.in Mon Apr 8 22:22:48 2002 > @@ -111,7 +111,9 @@ > extern int setpgid __P((int, int)); > extern int tcsetpgrp __P((int, int)); > #endif > +#ifndef __FreeBSD__ > extern int ioctl __P((int, int, char *)); > +#endif > > extern int kill __P((int, int)); > > I put this as misc/screen/files/patch-aj and screen just built. I don't have the emails offhand, but the screen people were pretty quick to respond to a help message from me. Have you contacted them about bringing in this fix? -- -Alfred Perlstein [alfred@freebsd.org] 'Instead of asking why a piece of software is using "1970s technology," start asking why software is ignoring 30 years of accumulated wisdom.' Tax deductible donations for FreeBSD: http://www.freebsdfoundation.org/ To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message