bug#9308: [PATCH] Proposed fix

2013-09-26 Thread Pádraig Brady
On 09/26/2013 05:27 AM, Jim Meyering wrote: On Wed, Sep 25, 2013 at 3:55 AM, Pádraig Brady p...@draigbrady.com wrote: ... The change looks correct, thanks. I've added NEWS, and tests in the attached and will push soon. Thanks to both of you. The complete patch looks fine. One suggested

bug#9308: [PATCH] Proposed fix

2013-09-25 Thread Pádraig Brady
On 09/23/2013 04:37 AM, Sergio Durigan Junior wrote: Hi, This bug is simple enough that it took only a one-liner to fix it. I believe it makes sense, because (as correctly pointed out by Daniel), rm's manpage explicitly mentions that -I and --interactive=once should be the same. Tested

bug#9308: [PATCH] Proposed fix

2013-09-25 Thread Jim Meyering
On Wed, Sep 25, 2013 at 3:55 AM, Pádraig Brady p...@draigbrady.com wrote: ... The change looks correct, thanks. I've added NEWS, and tests in the attached and will push soon. Thanks to both of you. The complete patch looks fine. One suggested tweak: - rm -I will prompt for confirmation when

bug#9308: [PATCH] Proposed fix

2013-09-23 Thread Sergio Durigan Junior
Hi, This bug is simple enough that it took only a one-liner to fix it. I believe it makes sense, because (as correctly pointed out by Daniel), rm's manpage explicitly mentions that -I and --interactive=once should be the same. Tested here, without failures. I already have commit access to the