ver6 (Re: [PATCH 1/8] busybox -- SELinux option support for coreutils: ver3

Bernhard Fischer rep.dot.nop at gmail.com
Fri Mar 9 17:06:46 UTC 2007


On Fri, Apr 06, 2007 at 11:28:34PM +0900, Yuichi Nakamura wrote:
>Thanks for review!

A minor cosmetic nit:
@@ -1304,8 +1327,9 @@
 #define id_full_usage \
        "Print information for USERNAME or the current user" \
        "\n\nOptions:\n" \
-       USE_SELINUX( \
-       "       -c      Prints only the security context\n") \
+       USAGE_SELINUX( \
+       "       -Z      prints only the security context\n" \
+       ) \
        "       -g      Prints only the group ID\n" \
        "       -u      Prints only the user ID\n" \
        "       -n      Print a name instead of a number\n" \

Please remove USAGE_SELINUX. Nowadays, there is USE_SELINUX which should
be used everywhere.




More information about the busybox mailing list