|
From: | Eric Blake |
Subject: | bug#6402: [PATCH] rm: added --directory (-d) option |
Date: | Wed, 15 Sep 2010 16:23:33 -0600 |
User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.9) Gecko/20100907 Fedora/3.1.3-1.fc13 Mnenhy/0.8.3 Thunderbird/3.1.3 |
On 08/18/2010 07:05 AM, William Plusnick wrote:
Sorry to take so long to reply, life got busy all of a sudden.
Equally sorry for my delay in turn. Now are we even? :)
I took your advice and instead made a trivial patch (I don't think I will have to sign anything) that instead prints an error message that points them to rmdir and exits.
Thanks for the patch. However, it's harder to review patches that aren't sent as plain/text MIME types. Here's the meat:
@@ -227,10 +227,10 @@ main (int argc, char **argv) switch (c) { case 'd': - /* Ignore this option, for backward compatibility with - coreutils 5.92. FIXME: Some time after 2005, change this - to report an error (or perhaps behave like FreeBSD does) - instead of ignoring the option. */ + /* This is not supported by vote of the maintainers. Print + * an error message rather than ignore it. */ + error (EXIT_FAILURE, 0, "the -d flag is not supported in rm(1)" + ", please see rmdir(1) instead."); break;
Indeed trivial, but missing translation of the string. At any rate, it's certainly after 2005, so according to our own comment, we indeed should do something very similar to what you have proposed!
-- Eric Blake address@hidden +1-801-349-2682 Libvirt virtualization library http://libvirt.org
[Prev in Thread] | Current Thread | [Next in Thread] |