Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually

2012-11-16 Thread Jim Meyering
James Youngman wrote: > On Fri, Nov 16, 2012 at 10:33 PM, Paul Eggert wrote: >> On 11/16/12 14:10, Dmitry V. Levin wrote: >>> As I said, my proposal is to introduce a new FTS_ flag that would make >>> fts_open behave as before that change, and use this flag in findutils. >>> I haven't heard yet ne

Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually

2012-11-16 Thread Jim Meyering
Dmitry V. Levin wrote: > On Fri, Nov 16, 2012 at 09:52:06PM +, James Youngman wrote: >> Did anybody do any further work on whether it worked best to fix this >> in gnulib or in findutils? > > As I said, my proposal is to introduce a new FTS_ flag that would make > fts_open behave as before that

Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually

2012-11-16 Thread James Youngman
On Fri, Nov 16, 2012 at 10:33 PM, Paul Eggert wrote: > On 11/16/12 14:10, Dmitry V. Levin wrote: >> As I said, my proposal is to introduce a new FTS_ flag that would make >> fts_open behave as before that change, and use this flag in findutils. >> I haven't heard yet neither from Jim nor from othe

Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually

2012-11-16 Thread Paul Eggert
On 11/16/12 14:10, Dmitry V. Levin wrote: > As I said, my proposal is to introduce a new FTS_ flag that would make > fts_open behave as before that change, and use this flag in findutils. > I haven't heard yet neither from Jim nor from other gnulib people whether > it is acceptable or there is a be

Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually

2012-11-16 Thread James Youngman
Did anybody do any further work on whether it worked best to fix this in gnulib or in findutils?

Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually

2012-11-16 Thread Dmitry V. Levin
On Fri, Nov 16, 2012 at 09:52:06PM +, James Youngman wrote: > Did anybody do any further work on whether it worked best to fix this > in gnulib or in findutils? As I said, my proposal is to introduce a new FTS_ flag that would make fts_open behave as before that change, and use this flag in fi

Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually

2012-11-01 Thread Dmitry V. Levin
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, >

Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually

2012-11-01 Thread Eric Blake
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

Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually

2012-10-31 Thread Dmitry V. Levin
On Tue, Sep 04, 2012 at 01:09:24PM +0200, Jim Meyering wrote: [...] > Here's the revised commit: > > >From 3a9002d3cc63da7110f133b1040d2d2b0aad8305 Mon Sep 17 00:00:00 2001 > From: Jim Meyering > Date: Tue, 4 Sep 2012 11:36:38 +0200 > Subject: [PATCH] fts: reduce two or more trailing slashes to j

Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually

2012-09-04 Thread Jim Meyering
Eric Blake wrote: > On 09/04/2012 04:57 AM, Eric Blake wrote: >>> * 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 or more slashes, reduce it to "//", not to "/". >> >> For two slashes, tha

Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually

2012-09-04 Thread Eric Blake
On 09/04/2012 05:09 AM, Jim Meyering wrote: > Eric Blake wrote: >> On 09/04/2012 04:57 AM, Eric Blake wrote: * 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 or more slashes, reduce i

Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually

2012-09-04 Thread Jim Meyering
Eric Blake wrote: > On 09/04/2012 04:57 AM, Eric Blake wrote: >>> * 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 or more slashes, reduce it to "//", not to "/". >>> >> >> For two slashes,

Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually

2012-09-04 Thread Jim Meyering
Eric Blake wrote: > On 09/04/2012 04:57 AM, Eric Blake wrote: >>> * 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 or more slashes, reduce it to "//", not to "/". >>> >> >> For two slashes,

Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually

2012-09-04 Thread Eric Blake
On 09/04/2012 04:57 AM, Eric Blake wrote: >> * 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 or more slashes, reduce it to "//", not to "/". >> > > For two slashes, that is correct. But tri

Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually

2012-09-04 Thread Eric Blake
On 09/04/2012 04:24 AM, Jim Meyering wrote: >>From 99376a06eed07ded17919384cf1d633e43ad1bd0 Mon Sep 17 00:00:00 2001 > From: Jim Meyering > Date: Tue, 4 Sep 2012 11:36:38 +0200 > Subject: [PATCH] fts: reduce two or more trailing spaces to just one, usually > > * lib/f

Re: [PATCH] fts: reduce two or more trailing spaces to just one, usually

2012-09-04 Thread Jim Meyering
Jim Meyering wrote: ... > Subject: [PATCH] fts: reduce two or more trailing spaces to just one, usually Glad I noticed the typo in that summary before pushing: s/spaces/slashes/

[PATCH] fts: reduce two or more trailing spaces to just one, usually

2012-09-04 Thread Jim Meyering
e, 4 Sep 2012 11:36:38 +0200 Subject: [PATCH] fts: reduce two or more trailing spaces 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 or more slashes, reduce it to "//"