From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 16588 invoked by alias); 29 May 2005 18:19:49 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 16569 invoked by uid 22791); 29 May 2005 18:19:47 -0000 Received: from c-66-30-17-189.hsd1.ma.comcast.net (HELO cgf.cx) (66.30.17.189) by sourceware.org (qpsmtpd/0.30-dev) with ESMTP; Sun, 29 May 2005 18:19:47 +0000 Received: by cgf.cx (Postfix, from userid 201) id BF94213CA7E; Sun, 29 May 2005 14:19:45 -0400 (EDT) Date: Sun, 29 May 2005 18:27:00 -0000 From: Christopher Faylor To: gdb-patches@sourceware.org Subject: Re: RFA: Use lrealpath instead of gdb_realpath Message-ID: <20050529181945.GE11936@trixie.casa.cgf.cx> Mail-Followup-To: gdb-patches@sourceware.org References: <20050528234233.GA3440@nevyn.them.org> <20050529142112.GB23858@nevyn.them.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.8i X-SW-Source: 2005-05/txt/msg00625.txt.bz2 On Sun, May 29, 2005 at 09:09:25PM +0300, Eli Zaretskii wrote: >>From: Daniel Jacobowitz >> >>>. The Windows code in lrealpath converts all forward slashes to >>> backslashes, which I think is a bad idea. By contrast, the current >>> gdb_realpath would leave existing forward slashes alone. That is >>> an incompatible change, and my gut feeling is that we should not >>> risk it. On top of that, I think we always use '/' to generate >>> file names, so with this change we will produce ugly file names >>> like D:\foo\bar/baz.c. >> >>The Windows maintainers disagree with you. So, for the record, do I. > >Disagree with what part? that backslashes might cause trouble or that >D:\foo\bar/baz.c is ugly? Let me clarify what I said before. I *do* think that D:\foo\bar/baz.c is ugly. I haven't seen a case where this type of path was unacceptable, though. However, if there was going to be a problem then it would probably be in the use of '/'. So, I think that lrealpath should probably be changed to convert every slash to a backslash. cgf