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

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 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

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 egg...@cs.ucla.edu 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

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 Jim Meyering
James Youngman wrote: On Fri, Nov 16, 2012 at 10:33 PM, Paul Eggert egg...@cs.ucla.edu 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

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 ends in two

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, usually

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 meyer...@redhat.com Date: Tue, 4 Sep 2012 11:36:38 +0200 Subject: [PATCH] fts: reduce two or more trailing

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

2012-09-04 Thread Jim Meyering
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 //, not to /. This is part of the solution

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/

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 meyer...@redhat.com Date: Tue, 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

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 trimming /// to

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, that is correct.

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, that is correct.

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 it to //, not

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, that is correct.