[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually
From: |
Dmitry V. Levin |
Subject: |
Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually |
Date: |
Fri, 2 Nov 2012 00:08:54 +0400 |
On Thu, Nov 01, 2012 at 06:22:32AM -0600, Eric Blake wrote:
> On 10/31/2012 05:28 PM, Dmitry V. Levin wrote:
> > On Tue, Sep 04, 2012 at 01:09:24PM +0200, Jim Meyering wrote:
> > [...]
> >> Here's the revised commit:
> >>
> >> Subject: [PATCH] fts: reduce two or more trailing slashes to just one,
> >> usually
> >>
> >> * lib/fts.c (fts_open): Upon initialization, if a name ends in two
> >> or more slashes, trim all but the final one. But if a name consists
> >> solely of two slashes, don't modify it. If it consists solely of
> >> three or more slashes, strip all but one.
>
> > JFYI, a side effect of this commit is that find(1) output changed:
> > before$ find /// -prune
> > ///
> > after$ find /// -prune
> > /
>
> POSIX requires find to output the user's slashes unchanged, so we have
> caused a regression in find, and need to update find to counteract the
> change.
Thanks. I'm not sure it could be done easily in find without any gnulib
assistance. For example, this new fts_open behavior could be made
controllable by some new FTS_ flag.
--
ldv
pgprjCUVkEJrv.pgp
Description: PGP signature
- Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually, Eric Blake, 2012/11/01
- Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually,
Dmitry V. Levin <=
- Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually, James Youngman, 2012/11/16
- Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually, Dmitry V. Levin, 2012/11/16
- Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually, Paul Eggert, 2012/11/16
- Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually, James Youngman, 2012/11/16
- Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually, Jim Meyering, 2012/11/16
- Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually, Jim Meyering, 2012/11/16
- [PATCH] fts: introduce FTS_VERBATIM, Dmitry V. Levin, 2012/11/17
- Re: [PATCH] fts: introduce FTS_VERBATIM, Jim Meyering, 2012/11/17
- Re: [PATCH] fts: introduce FTS_VERBATIM, Dmitry V. Levin, 2012/11/17