From owner-freebsd-stable Thu Mar 13 23: 0:24 2003 Delivered-To: freebsd-stable@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 70CB037B404; Thu, 13 Mar 2003 23:00:20 -0800 (PST) Received: from david.siemens.de (david.siemens.de [192.35.17.14]) by mx1.FreeBSD.org (Postfix) with ESMTP id 1A14343FD7; Thu, 13 Mar 2003 23:00:16 -0800 (PST) (envelope-from andre.albsmeier@siemens.com) Received: from mail2.siemens.de (mail2.siemens.de [139.25.208.11]) by david.siemens.de (8.11.6/8.11.6) with ESMTP id h2E6xrw24233; Fri, 14 Mar 2003 07:59:53 +0100 (MET) Received: from mars.cert.siemens.de (ust.mchp.siemens.de [139.23.201.17]) by mail2.siemens.de (8.11.6/8.11.6) with ESMTP id h2E6xr607510; Fri, 14 Mar 2003 07:59:53 +0100 (MET) Received: from curry.mchp.siemens.de (curry.mchp.siemens.de [139.25.42.7]) by mars.cert.siemens.de (8.12.8/8.12.8/$SiemensCERT: mail/cert.mc,v 1.42 2003/02/21 12:06:56 ust Exp $) with ESMTP id h2E6xqZM061132; Fri, 14 Mar 2003 07:59:52 +0100 (CET) Received: (from localhost) by curry.mchp.siemens.de (8.12.8/8.12.8) id h2E6xqP4078009; Date: Fri, 14 Mar 2003 07:59:50 +0100 From: Andre Albsmeier To: Erik Trulsson Cc: Andre Albsmeier , Eric Anholt , Garance A Drosihn , ports@FreeBSD.ORG, stable@FreeBSD.ORG Subject: Re: HEADSUP: XFree86 4.3.0 update Message-ID: <20030314075950.A5777@curry.mchp.siemens.de> References: <1047429424.8471.40.camel@leguin> <1047515632.614.163.camel@leguin> <20030313114036.A81385@curry.mchp.siemens.de> <20030313120807.GA90326@falcon.midgard.homeip.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5.1i In-Reply-To: <20030313120807.GA90326@falcon.midgard.homeip.net>; from ertr1013@student.uu.se on Thu, Mar 13, 2003 at 01:08:08PM +0100 X-Echelon: Pretoria, NSA, Clandestine, PGP, strike X-Advice: Drop that crappy M$-Outlook, I'm tired of your viruses! Sender: owner-freebsd-stable@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Thu, 13-Mar-2003 at 13:08:08 +0100, Erik Trulsson wrote: > On Thu, Mar 13, 2003 at 11:40:36AM +0100, Andre Albsmeier wrote: > > On Wed, 12-Mar-2003 at 16:33:52 -0800, Eric Anholt wrote: > > > On Wed, 2003-03-12 at 11:21, Garance A Drosihn wrote: > > > > At 4:37 PM -0800 3/11/03, Eric Anholt wrote: > > > > >I've committed the update of XFree86 to 4.3.0 to ports. I think > > > > >I've cleaned up after my mess at this point, but there may still > > > > >be issues. Please report to me if you have any problems with the > > > > >new ports or any issues with XFree86 that you didn't have in 4.2.0. > > > > > > > > A minor observation: > > > > > > > > As I watch this all compiling, I notice a lot of compiles > > > > are generating: > > > > > > > > /usr/include/sys/cdefs.h:273: warning: `_POSIX_C_SOURCE' is not defined > > > > /usr/include/sys/cdefs.h:279: warning: `_POSIX_C_SOURCE' is not defined > > > > > > These ones annoy me, too (-current). > > > > > > /usr/include/sys/time.h: In function `timespec2bintime': > > > /usr/include/sys/time.h:125: warning: ISO C89 forbids long long integer constants > > > /usr/include/sys/time.h: In function `timeval2bintime': > > > /usr/include/sys/time.h:142: warning: ISO C89 forbids long long integer constants > > > > > > If anyone proposes the right way to clean up these messages, I would > > > love to hear. XFree86 uses -ansi -pedantic in its compiles, and I'm > > > wondering if we could just drop that and make things look prettier, at > > > least for this second message. > > Dropping -pedantic should get rid of the warnings about 'long long' and > shouldn't hurt anything. (People developing XFree86 will probably want > to keep it, but for those of us who are just compiling it from ports > there is not much point in using -pedantic.) > Personally I got annoyed enough by warnings about 'long long' that I > modified the source for gcc such that -pedantic no longer warns about > 'long long'. This is almost certainly not the correct solution, though > it stops the warnings. > The correct solution is probably to fix the header files so they don't > use 'long long' when compiled in strict ANSI C mode. > > > > > > I am using this patch now (which might be complete nonsense but it > > stops the warnings): > > I have been using an identical patch for some time now (after being > overly annoyed by the warnings) and I believe it is the correct way of > getting rid of the warnings about _POSIX_C_SOURCE being undefined. > Since this patch > a) is perfectly legal C > b) does not change the semantics of the header file > c) does get rid of the warnings > d) prevents the code from using the value of a preprocessor macro > without defining it first which, while legal, is hardly good > practice (which is why the warning is there in the first place) > I wouldn't call the patch complete nonsense but rather call it a good idea. Well, Garrett Wollman thinks differently about this: He said one should not use -Wundef during compiling. Needless to say, I don't agree with him :-). -Andre > > > > > > > --- sys/cdefs.h.ORI Wed Sep 18 16:52:23 2002 > > +++ sys/cdefs.h Thu Mar 13 10:20:54 2003 > > @@ -269,6 +269,8 @@ > > * Our macros begin with two underscores to avoid namespace screwage. > > */ > > > > +#ifdef _POSIX_C_SOURCE > > + > > /* Deal with IEEE Std. 1003.1-1990, in which _POSIX_C_SOURCE == 1. */ > > #if _POSIX_C_SOURCE == 1 > > #undef _POSIX_C_SOURCE /* Probably illegal, but beyond caring now. */ > > @@ -280,6 +282,8 @@ > > #undef _POSIX_C_SOURCE > > #define _POSIX_C_SOURCE 199209 > > #endif > > + > > +#endif > > > > /* Deal with various X/Open Portability Guides and Single UNIX Spec. */ > > #ifdef _XOPEN_SOURCE > > > > > -- > > Erik Trulsson > ertr1013@student.uu.se > > To Unsubscribe: send mail to majordomo@FreeBSD.org > with "unsubscribe freebsd-stable" in the body of the message -- "I think there is a world market for maybe five computers." - Thomas Watson, chairman of IBM, 1943 To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-stable" in the body of the message