Page MenuHomeFreeBSD

D49621.id152981.diff
No OneTemporary

D49621.id152981.diff

diff --git a/bin/ps/ps.c b/bin/ps/ps.c
--- a/bin/ps/ps.c
+++ b/bin/ps/ps.c
@@ -282,16 +282,23 @@
case 'g':
#if 0
/*-
- * XXX - This SUSv3 behavior is still under debate
- * since it conflicts with the (undocumented)
- * `-g' option. So we skip it for now.
+ * XXX - This behavior is still under debate since it
+ * conflicts with the (undocumented) `-g' option
+ * and is non-standard. However, it is the
+ * behavior of most UNIX systems except
+ * SunOS/Solaris/illumos (see next comment; see
+ * also comment for '-s' below).
*/
add_list(&pgrplist, optarg);
xkeep_implied = 1;
nselectors++;
break;
#else
- /* The historical BSD-ish (from SunOS) behavior. */
+ /*
+ * The historical BSD-ish (from SunOS) behavior: Also
+ * display process group leaders (but we do not filter
+ * them out).
+ */
break; /* no-op */
#endif
case 'H':
@@ -365,10 +372,12 @@
#if 0
case 's':
/*-
- * XXX - This non-standard option is still under
- * debate. This *is* supported on Solaris,
- * Linux, and IRIX, but conflicts with `-s'
- * on NetBSD and maybe some older BSD's.
+ * XXX - This non-standard option is still under debate.
+ * It is supported on Solaris, Linux, IRIX, and
+ * OpenBSD but conflicts with '-s' on NetBSD. This
+ * is the same functionality as POSIX option '-g',
+ * but no current UNIX system provides it under
+ * '-g'.
*/
add_list(&sesslist, optarg);
xkeep_implied = 1;
@@ -385,7 +394,12 @@
nselectors++;
break;
case 'U':
- /* This is what SUSv3 defines as the `-u' option. */
+ /*
+ * POSIX says that '-U' should match on real user IDs,
+ * not effective ones as we are doing here, which is
+ * normally the behavior of option '-u' according to the
+ * standard.
+ */
add_list(&uidlist, optarg);
xkeep_implied = 1;
nselectors++;

File Metadata

Mime Type
text/plain
Expires
Wed, Jul 22, 3:06 PM (4 h, 37 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
35368629
Default Alt Text
D49621.id152981.diff (1 KB)

Event Timeline