typo fix in a comment in a testcase. oh well...

[oweals/busybox.git] / coreutils / rmdir.c

diff --git a/coreutils/rmdir.c b/coreutils/rmdir.c
index 7f3253017f40cec6351748d7109adc38c33c2bd5..2450a43a3fe60cdbab2c1276fe216e2bfbae56b0 100644 (file)
--- a/coreutils/rmdir.c
+++ b/coreutils/rmdir.c
@@ -10,21 +10,31 @@
 /* BB_AUDIT SUSv3 compliant */
 /* http://www.opengroup.org/onlinepubs/007904975/utilities/rmdir.html */
 
-#include <libgen.h>
-#include "busybox.h"
+#include "libbb.h"
 
 /* This is a NOFORK applet. Be very careful! */
 
 
-int rmdir_main(int argc, char **argv);
-int rmdir_main(int argc, char **argv)
+#define PARENTS 0x01
+#define IGNORE_NON_EMPTY 0x02
+
+int rmdir_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
+int rmdir_main(int argc UNUSED_PARAM, char **argv)
 {
        int status = EXIT_SUCCESS;
        int flags;
-       int do_dot;
        char *path;
 
-       flags = getopt32(argc, argv, "p");
+#if ENABLE_FEATURE_RMDIR_LONG_OPTIONS
+       static const char rmdir_longopts[] ALIGN1 =
+               "parents\0"                  No_argument "p"
+               /* Debian etch: many packages fail to be purged or installed
+                * because they desperately want this option: */
+               "ignore-fail-on-non-empty\0" No_argument "\xff"
+               ;
+       applet_long_options = rmdir_longopts;
+#endif
+       flags = getopt32(argv, "p");
        argv += optind;
 
        if (!*argv) {
@@ -34,27 +44,26 @@ int rmdir_main(int argc, char **argv)
        do {
                path = *argv;
 
-               /* Record if the first char was a '.' so we can use dirname later. */
-               do_dot = (*path == '.');
-
                while (1) {
                        if (rmdir(path) < 0) {
+#if ENABLE_FEATURE_RMDIR_LONG_OPTIONS
+                               if ((flags & IGNORE_NON_EMPTY) && errno == ENOTEMPTY)
+                                       break;
+#endif
                                bb_perror_msg("'%s'", path);    /* Match gnu rmdir msg. */
                                status = EXIT_FAILURE;
-                       } else if (flags) {
-                               /* Note: path was not empty or null since rmdir succeeded. */
+                       } else if (flags & PARENTS) {
+                               /* Note: path was not "" since rmdir succeeded. */
                                path = dirname(path);
-                               /* Path is now just the parent component.  Note that dirname
-                                * returns "." if there are no parents.  We must distinguish
-                                * this from the case of the original path starting with '.'.
+                               /* Path is now just the parent component.  Dirname
+                                * returns "." if there are no parents.
                                 */
-                               if (do_dot || (*path != '.') || path[1]) {
+                               if (NOT_LONE_CHAR(path, '.')) {
                                        continue;
                                }
                        }
                        break;
                }
-
        } while (*++argv);
 
        return status;