Re: coreutils 5.3.0: cp --parents broken

[Jim Meyering]

Eric Blake <address@hidden> wrote:

> According to Jim Meyering on 1/25/2005 5:31 AM:
>>      * path-concat.c: Don't include assert.h.
>>      (path_concat): Remove assertion that would have triggered
>>      for ABASE starting with more than one slash.
>>      Reported by Andreas Schwab.
>>
>> Index: path-concat.c
>> -   concatenation.
>> +   concatenation.  However, if ABASE begins with more than one slash,
>> +   set *BASE_IN_RESULT to point to the sole corresponding slash that
>> +   is copied into the result buffer.
>
> Collapsing //file/name to /file/name violates POSIX (some systems use that
> notation for remote machine shared drive access).  XBD 3.2 defines
> absolute path as "A pathname beginning with a single or more than two
> slashes", and XBD 4.11 agrees, so you are only allowed to collapse 3 or
> more leading slashes.

I am well aware of that part of POSIX.
But it doesn't apply here, since we're not talking about a
prefix of the result.

This function concatenates two names, usually a directory and
some other thing (file or directory) and the above ABASE refers
to the latter part.