Re: coreutils bug with "ln x d/"

[Jim Meyering]

Paul Eggert <address@hidden> wrote:
> OK, I split the patch into two.  This takes a more conservative
> approach than what I wrote above: the first patch fixes the bug, and
> the second patch (which I'll send in my next message) is the cleanup
> patch.
>
> These two patches supersedes the long patch I sent you Friday
> <http://lists.gnu.org/archive/html/bug-coreutils/2004-06/msg00146.html>
> but it's independent of the patch to "install" that I sent yesterday
> <http://lists.gnu.org/archive/html/bug-coreutils/2004-06/msg00153.html>.
>
> Here's the first (bugfix) patch.
>
> 2004-06-27  Paul Eggert  <address@hidden>
>
>       Fix a bug: formerly, if d/x was a directory and x a file, "ln x
>       d/" incorrectly created a link d/x/x.  It also saves some system
>       calls.
>
>       * NEWS: Document the fix.
>
>       * src/ln.c (main): Don't append basename to dest if this
>       results in an existing directory name.
>       * tests/ln/misc: See whether a trailing slash is followed too far.

I've applied that.
Thanks for separating it.