From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 3327 invoked by alias); 11 Dec 2012 10:20:10 -0000 Received: (qmail 3068 invoked by uid 22791); 11 Dec 2012 10:19:28 -0000 X-Spam-Check-By: sourceware.org Received: from aquarius.hirmke.de (HELO calimero.vinschen.de) (217.91.18.234) by sourceware.org (qpsmtpd/0.83/v0.83-20-g38e4449) with ESMTP; Tue, 11 Dec 2012 10:19:16 +0000 Received: by calimero.vinschen.de (Postfix, from userid 500) id D888F6A09B6; Tue, 11 Dec 2012 11:19:13 +0100 (CET) Date: Tue, 11 Dec 2012 10:20:00 -0000 From: Corinna Vinschen To: gdb-patches@sourceware.org Subject: Re: [RFA/mingw32] environment variables are case-insensitive on win32 Message-ID: <20121211101913.GA965@calimero.vinschen.de> Reply-To: gdb-patches@sourceware.org Mail-Followup-To: gdb-patches@sourceware.org References: <20121210110128.GA12570@calimero.vinschen.de> <20121210134935.GL31477@adacore.com> <20121210152408.GB11967@calimero.vinschen.de> <20121210154231.GC17188@adacore.com> <20121210155752.GA16027@calimero.vinschen.de> <83a9tlc2jb.fsf@gnu.org> <20121210161710.GB16027@calimero.vinschen.de> <836249c1tl.fsf@gnu.org> <20121210165727.GD16027@calimero.vinschen.de> <83y5h5acsa.fsf@gnu.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <83y5h5acsa.fsf@gnu.org> User-Agent: Mutt/1.5.21 (2010-09-15) X-IsSubscribed: yes Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2012-12/txt/msg00311.txt.bz2 On Dec 10 22:10, Eli Zaretskii wrote: > > Date: Mon, 10 Dec 2012 17:57:27 +0100 > > From: Corinna Vinschen > > > > On Dec 10 18:24, Eli Zaretskii wrote: > > > > Date: Mon, 10 Dec 2012 17:17:10 +0100 > > > > From: Corinna Vinschen > > > > > > > > > If some versions of Cygwin are case-insensitive in this context, while > > > > > others aren't, then a run-time test of the kind coded by Jerome is > > > > > _exactly_ the Right Thing. > > > > > > > > These old versions are not at all supported anymore. The right thing to > > > > do in this case is update. > > > > > > Why is requiring users to upgrade more reasonable than half a dozen > > > lines of simple code? > > > > Old, unsupported. What you do by adding these tests and special cases > > is to complicate the tests and add potential bugs for no good reason. > > Again, we are talking about short and straightforward code. Nowhere > near any danger of complicating things and adding bugs. Since when do > we fear adding a bunch of string comparison lines? It's just my opinion. You apparently have another one. I'm not in charge of the code, so do as you see fit. Corinna -- Corinna Vinschen Cygwin Project Co-Leader Red Hat