bug#6402: [PATCH] rm: remove no-op -d option (original) (raw)

[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

From: Jim Meyering
Subject: bug#6402: [PATCH] rm: remove no-op -d option
Date: Fri, 17 Sep 2010 11:13:25 +0200

Eric Blake wrote:

* src/rm.c (longopts, main): Resolve a fixme. * NEWS: Document the change. Based on a report by William Plusnick. ---

Jim, what do you think of this alternative patch, which avoids the issue of a new translation string by instead letting getopt parsing reject -d like any other unknown option?

I like it. Thank you.

...

--- a/NEWS ... + rm -d now issues an error rather than being silently ignored.

How about a slight change in wording to make clear that the entire "rm" command was not being ignored.

rm's -d now evokes an error; before, it was silently ignored.