Return-Path: Delivered-To: apmail-jakarta-tomcat-dev-archive@www.apache.org Received: (qmail 47406 invoked from network); 23 Jun 2005 07:36:06 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 23 Jun 2005 07:36:06 -0000 Received: (qmail 85166 invoked by uid 500); 23 Jun 2005 07:35:58 -0000 Delivered-To: apmail-jakarta-tomcat-dev-archive@jakarta.apache.org Received: (qmail 85050 invoked by uid 500); 23 Jun 2005 07:35:57 -0000 Mailing-List: contact tomcat-dev-help@jakarta.apache.org; run by ezmlm Precedence: bulk List-Unsubscribe: List-Help: List-Post: List-Id: "Tomcat Developers List" Reply-To: "Tomcat Developers List" Delivered-To: mailing list tomcat-dev@jakarta.apache.org Received: (qmail 84995 invoked by uid 99); 23 Jun 2005 07:35:57 -0000 Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 23 Jun 2005 00:35:57 -0700 X-ASF-Spam-Status: No, hits=0.1 required=10.0 tests=FORGED_RCVD_HELO X-Spam-Check-By: apache.org Received-SPF: pass (asf.osuosl.org: local policy) Received: from [207.155.252.72] (HELO thunderer.cnchost.com) (207.155.252.72) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 23 Jun 2005 00:35:57 -0700 Received: from rcsv650.rowe-clan.net (c-24-13-128-132.hsd1.il.comcast.net [24.13.128.132]) by thunderer.cnchost.com id DAA00590; Thu, 23 Jun 2005 03:35:55 -0400 (EDT) [ConcentricHost SMTP Relay 1.17] Errors-To: Message-Id: <6.2.1.2.2.20050623022905.0a343a60@pop3.rowe-clan.net> X-Mailer: QUALCOMM Windows Eudora Version 6.2.1.2 Date: Thu, 23 Jun 2005 02:33:13 -0500 To: "Tomcat Developers List" From: "William A. Rowe, Jr." Subject: Re: cvs commit: jakarta-tomcat-connectors/jni/native configure.in In-Reply-To: <42BA5E53.2060106@apache.org> References: <20050621103141.17636.qmail@minotaur.apache.org> <42BA5E53.2060106@apache.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" X-Virus-Checked: Checked by ClamAV on apache.org X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N At 02:01 AM 6/23/2005, Mladen Turk wrote: >jfclere@apache.org wrote: >>jfclere 2005/06/21 03:31:41 >> Log: >> -Wall is only for gcc. > >That's true. But if your 'cc' doesn't support >the -Wall then make CFLASG with some platform >switch rather then interfering with 99% of others that are using gcc. >The -Wall is very valuable when dealing with compile warnings, >and that was the main reason what the --enable-maintainer-mode >was added at the first place. It makes no sense without. > > >> - CFLAGS="${CFLAGS} -DDEBUG -Wall" >> + CFLAGS="${CFLAGS} -DDEBUG" > >I would prefer that you add some platform case/esac that will >in ReliantUnix case add what ever you wish to the CFLAGS. Conversely, a compiler case/esac would avoid altogether breaking AIX, HP/UX, Reliant and a dozen other oddballs. Forcing users to play cflags is really a symptom of a weak build system. Bill (Who's wasted untold hours fighting with auto**** linux-isms lately and who has no humor left for penguins :) --------------------------------------------------------------------- To unsubscribe, e-mail: tomcat-dev-unsubscribe@jakarta.apache.org For additional commands, e-mail: tomcat-dev-help@jakarta.apache.org