getopt32: remove opt_complementary

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

diff --git a/coreutils/chown.c b/coreutils/chown.c
index 2d8e556f06029f2ed5f7e2e117db952d8b290a32..985d18d6f43b5a58d01e39c5a867c581ea2f2024 100644 (file)
--- a/coreutils/chown.c
+++ b/coreutils/chown.c
@@ -4,19 +4,58 @@
  *
  * Copyright (C) 1999-2004 by Erik Andersen <andersen@codepoet.org>
  *
- * Licensed under GPLv2 or later, see file LICENSE in this tarball for details.
+ * Licensed under GPLv2 or later, see file LICENSE in this source tree.
  */
+//config:config CHOWN
+//config:      bool "chown (7.2 kb)"
+//config:      default y
+//config:      help
+//config:      chown is used to change the user and/or group ownership
+//config:      of files.
+//config:
+//config:config FEATURE_CHOWN_LONG_OPTIONS
+//config:      bool "Enable long options"
+//config:      default y
+//config:      depends on CHOWN && LONG_OPTS
+
+//applet:IF_CHOWN(APPLET_NOEXEC(chown, chown, BB_DIR_BIN, BB_SUID_DROP, chown))
+
+//kbuild:lib-$(CONFIG_CHOWN) += chown.o
 
 /* BB_AUDIT SUSv3 defects - none? */
-/* BB_AUDIT GNU defects - unsupported long options. */
 /* http://www.opengroup.org/onlinepubs/007904975/utilities/chown.html */
 
+//usage:#define chown_trivial_usage
+//usage:       "[-Rh"IF_DESKTOP("LHPcvf")"]... USER[:[GRP]] FILE..."
+//usage:#define chown_full_usage "\n\n"
+//usage:       "Change the owner and/or group of each FILE to USER and/or GRP\n"
+//usage:     "\n       -R      Recurse"
+//usage:     "\n       -h      Affect symlinks instead of symlink targets"
+//usage:       IF_DESKTOP(
+//usage:     "\n       -L      Traverse all symlinks to directories"
+//usage:     "\n       -H      Traverse symlinks on command line only"
+//usage:     "\n       -P      Don't traverse symlinks (default)"
+//usage:     "\n       -c      List changed files"
+//usage:     "\n       -v      List all files"
+//usage:     "\n       -f      Hide errors"
+//usage:       )
+//usage:
+//usage:#define chown_example_usage
+//usage:       "$ ls -l /tmp/foo\n"
+//usage:       "-r--r--r--    1 andersen andersen        0 Apr 12 18:25 /tmp/foo\n"
+//usage:       "$ chown root /tmp/foo\n"
+//usage:       "$ ls -l /tmp/foo\n"
+//usage:       "-r--r--r--    1 root     andersen        0 Apr 12 18:25 /tmp/foo\n"
+//usage:       "$ chown root.root /tmp/foo\n"
+//usage:       "ls -l /tmp/foo\n"
+//usage:       "-r--r--r--    1 root     root            0 Apr 12 18:25 /tmp/foo\n"
+
 #include "libbb.h"
 
 /* This is a NOEXEC applet. Be very careful! */
 
 
-#define OPT_STR     ("Rh" IF_DESKTOP("vcfLHP"))
+#define OPT_STR     "Rh" IF_DESKTOP("vcfLHP")
 #define BIT_RECURSE 1
 #define OPT_RECURSE (opt & 1)
 #define OPT_NODEREF (opt & 2)
@@ -37,6 +76,20 @@
 #define BIT_TRAVERSE_TOP (0x20|0x40)
 #define OPT_TRAVERSE_TOP (IF_DESKTOP(opt & BIT_TRAVERSE_TOP) IF_NOT_DESKTOP(0))
 
+#if ENABLE_FEATURE_CHOWN_LONG_OPTIONS
+static const char chown_longopts[] ALIGN1 =
+       "recursive\0"        No_argument   "R"
+       "dereference\0"      No_argument   "\xff"
+       "no-dereference\0"   No_argument   "h"
+# if ENABLE_DESKTOP
+       "changes\0"          No_argument   "c"
+       "silent\0"           No_argument   "f"
+       "quiet\0"            No_argument   "f"
+       "verbose\0"          No_argument   "v"
+# endif
+       ;
+#endif
+
 typedef int (*chown_fptr)(const char *, uid_t, gid_t);
 
 struct param_t {
@@ -49,8 +102,8 @@ static int FAST_FUNC fileAction(const char *fileName, struct stat *statbuf,
 {
 #define param  (*(struct param_t*)vparam)
 #define opt option_mask32
-       uid_t u = (param.ugid.uid == (uid_t)-1) ? statbuf->st_uid : param.ugid.uid;
-       gid_t g = (param.ugid.gid == (gid_t)-1) ? statbuf->st_gid : param.ugid.gid;
+       uid_t u = (param.ugid.uid == (uid_t)-1L) ? statbuf->st_uid : param.ugid.uid;
+       gid_t g = (param.ugid.gid == (gid_t)-1L) ? statbuf->st_gid : param.ugid.gid;
 
        if (param.chown_func(fileName, u, g) == 0) {
                if (OPT_VERBOSE
@@ -62,7 +115,7 @@ static int FAST_FUNC fileAction(const char *fileName, struct stat *statbuf,
                return TRUE;
        }
        if (!OPT_QUIET)
-               bb_simple_perror_msg(fileName); /* A filename can have % in it... */
+               bb_simple_perror_msg(fileName);
        return FALSE;
 #undef opt
 #undef param
@@ -74,18 +127,18 @@ int chown_main(int argc UNUSED_PARAM, char **argv)
        int opt, flags;
        struct param_t param;
 
-       param.ugid.uid = -1;
-       param.ugid.gid = -1;
-       param.chown_func = chown;
-
-       opt_complementary = "-2";
-       opt = getopt32(argv, OPT_STR);
+#if ENABLE_FEATURE_CHOWN_LONG_OPTIONS
+       opt = getopt32long(argv, "^" OPT_STR "\0" "=2", chown_longopts);
+#else
+       opt = getopt32(argv, "^" OPT_STR "\0" "=2");
+#endif
        argv += optind;
 
        /* This matches coreutils behavior (almost - see below) */
+       param.chown_func = chown;
        if (OPT_NODEREF
-           /* || (OPT_RECURSE && !OPT_TRAVERSE_TOP): */
-           IF_DESKTOP( || (opt & (BIT_RECURSE|BIT_TRAVERSE_TOP)) == BIT_RECURSE)
+       /* || (OPT_RECURSE && !OPT_TRAVERSE_TOP): */
+       IF_DESKTOP( || (opt & (BIT_RECURSE|BIT_TRAVERSE_TOP)) == BIT_RECURSE)
        ) {
                param.chown_func = lchown;
        }
@@ -101,8 +154,7 @@ int chown_main(int argc UNUSED_PARAM, char **argv)
        parse_chown_usergroup_or_die(&param.ugid, argv[0]);
 
        /* Ok, ready to do the deed now */
-       argv++;
-       do {
+       while (*++argv) {
                if (!recursive_action(*argv,
                                flags,          /* flags */
                                fileAction,     /* file action */
@@ -112,7 +164,7 @@ int chown_main(int argc UNUSED_PARAM, char **argv)
                ) {
                        retval = EXIT_FAILURE;
                }
-       } while (*++argv);
+       }
 
        return retval;
 }