RFA: Use lrealpath instead of gdb_realpath

Christopher Faylor me@cgf.cx
Sun May 29 17:49:00 GMT 2005


On Sun, May 29, 2005 at 10:21:12AM -0400, Daniel Jacobowitz wrote:
>>Let me explain my concerns about using lrealpath in its current shape:
>>
>> . 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.

Yes.  I have to say that I can't recall a situation where slashes and
backslashes make a difference but, if there was going to be a problem,
it seems like the problem would be with the use of slashes.

>It is not worth my time to continue arguing about filename handling;
>consider the patch withdrawn indefinitely.

That is a shame.  Having two similar functions really doesn't make a
lot of sense to me.

cgf



More information about the Gdb-patches mailing list