From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 1228 invoked by alias); 16 Oct 2007 20:59:53 -0000 Received: (qmail 1219 invoked by uid 22791); 16 Oct 2007 20:59:52 -0000 X-Spam-Status: No, hits=-0.9 required=5.0 tests=AWL,BAYES_00,DK_POLICY_SIGNSOME,SPF_FAIL X-Spam-Check-By: sourceware.org Received: from stelecom.gomel.by (HELO stelecom.gomel.by) (82.209.213.61) by sourceware.org (qpsmtpd/0.31) with ESMTP; Tue, 16 Oct 2007 20:59:46 +0000 Received: by stelecom.gomel.by (Postfix, from userid 501) id 62CD6B019BD9; Tue, 16 Oct 2007 23:59:42 +0300 (EEST) Received: from localhost (unknown [86.57.140.191]) by stelecom.gomel.by (Postfix) with ESMTP id AC66EB019856; Tue, 16 Oct 2007 23:59:31 +0300 (EEST) Date: Tue, 16 Oct 2007 20:59:00 -0000 From: Sergei Gavrikov To: Sergei Organov Cc: Gary Thomas , eCos patches list Subject: Re: Pedantic preprocessor Message-ID: <20071016205928.GA6586@ubuntu> References: <20071013214906.GA9874@ubuntu> <47134EFE.9070602@mlbassoc.com> <87sl4bv2nn.fsf@javad.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <87sl4bv2nn.fsf@javad.com> User-Agent: Mutt/1.5.13 (2006-08-11) X-IsSubscribed: yes Mailing-List: contact ecos-patches-help@ecos.sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: ecos-patches-owner@ecos.sourceware.org X-SW-Source: 2007-10/txt/msg00015.txt.bz2 On Tue, Oct 16, 2007 at 01:18:04PM +0400, Sergei Organov wrote: > Gary Thomas writes: > > Sergei Gavrikov wrote: > [...] > >> ecosconfig new sam7ex256 > >> ecosconfig tree > >> make -s > >> U CYGHWR_HAL_ARM_AT91, new inferred value AT91SAM7S > >> U CYGHWR_HAL_ARM_AT91_FIQ, new inferred value 1 > >> U CYGHWR_HAL_ARM_AT91SAM7, new inferred value at91sam7x256 > >> headers finished > >> cc1: warning: command line option "-Woverloaded-virtual" is valid for C++/ObjC++ but not for C > >> cc1: warning: command line option "-fno-rtti" is valid for C++/ObjC++ but not for C > >> cc1: warning: command line option "-Woverloaded-virtual" is valid for C++/ObjC++ but not for C > >> cc1: warning: command line option "-fno-rtti" is valid for C++/ObjC++ but not for C > >> cc1: warning: command line option "-Woverloaded-virtual" is valid for C++/ObjC++ but not for C > >> cc1: warning: command line option "-fno-rtti" is valid for C++/ObjC++ but not for C > >> build finished > >> > [...] > >> I like to see these 2 lines together. And if anybody knows how to fix > >> this issue in platform CDL, please, enlight me. > >> > > Your patch is the correct one - I've applied it. Next time, don't > > forget the ChangeLog entry. > > Is there anybody here who is willing/capable to fix the rest of HALs the > same way, as every HAL in eCos suffers from the same problem (e.g., I > recently happened to do the same thing for SPARC HAL)? I wouldn't risk to patch all arch. CDLs vis-a-vis, sorry. IMO, this is the experts's deal. I uses the toolchains for arm and i386 targets only. And as I said, the CPP from a pretty old, but the stable GCC suite (3.2.1) quite builds eCos for my targets without any warnings. Though, I think since the ecoscentric has offered the GCC 3.4.4 suite for own customers, they have done something like that receipt already. If they would confirm our way, so the someone's deal will be just a s/CFLAGS/ACTUAL_&/g otherwise, a group of the testers are wanted: find $ECOS_REPOSITORY -regex '.*/arch/.*/cdl/hal.*cdl' | wc -l ... So, submit the same patch for SPARC, please, if you've tested it. Sergei