[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: coreutils bug with "ln x d/"
From: |
Jim Meyering |
Subject: |
Re: coreutils bug with "ln x d/" |
Date: |
Sun, 27 Jun 2004 11:44:29 +0200 |
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.