Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 7 Jun 2015 21:57:21 +0000 (UTC)
From:      Baptiste Daroussin <bapt@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r284139 - head/usr.sbin/pw
Message-ID:  <201506072157.t57LvLww030937@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: bapt
Date: Sun Jun  7 21:57:20 2015
New Revision: 284139
URL: https://svnweb.freebsd.org/changeset/base/284139

Log:
  Remove '-q' support for pw [user|group] next
  
  the intent of -q in this command is to return as exit status the value of the
  next group/user id, which does not make sense given exit status are limited to
  values between 0 and 255.

Modified:
  head/usr.sbin/pw/pw.8
  head/usr.sbin/pw/pw.c
  head/usr.sbin/pw/pw_group.c
  head/usr.sbin/pw/pw_user.c

Modified: head/usr.sbin/pw/pw.8
==============================================================================
--- head/usr.sbin/pw/pw.8	Sun Jun  7 21:14:48 2015	(r284138)
+++ head/usr.sbin/pw/pw.8	Sun Jun  7 21:57:20 2015	(r284139)
@@ -24,7 +24,7 @@
 .\"
 .\" $FreeBSD$
 .\"
-.Dd June 3, 2015
+.Dd June 7, 2015
 .Dt PW 8
 .Os
 .Sh NAME
@@ -128,7 +128,6 @@
 .Op Fl V Ar etcdir
 .Ar usernext
 .Op Fl C Ar config
-.Op Fl q
 .Nm
 .Op Fl R Ar rootdir
 .Op Fl V Ar etcdir
@@ -184,7 +183,6 @@
 .Op Fl V Ar etcdir
 .Ar groupnext
 .Op Fl C Ar config
-.Op Fl q
 .Nm
 .Op Fl R Ar rootdir
 .Op Fl V Ar etcdir

Modified: head/usr.sbin/pw/pw.c
==============================================================================
--- head/usr.sbin/pw/pw.c	Sun Jun  7 21:14:48 2015	(r284138)
+++ head/usr.sbin/pw/pw.c	Sun Jun  7 21:57:20 2015	(r284139)
@@ -113,7 +113,7 @@ main(int argc, char *argv[])
 			"R:V:C:qn:u:rY",
 			"R:V:C:qn:u:c:d:e:p:g:G:mM:l:k:s:w:L:h:H:FNPY",
 			"R:V:C:qn:u:FPa7",
-			"R:V:C:q",
+			"R:V:C",
 			"R:V:C:q",
 			"R:V:C:q"
 		},
@@ -122,7 +122,7 @@ main(int argc, char *argv[])
 			"R:V:C:qn:g:Y",
 			"R:V:C:qn:d:g:l:h:H:FM:m:NPY",
 			"R:V:C:qn:g:FPa",
-			"R:V:C:q"
+			"R:V:C"
 		 }
 	};
 
@@ -469,8 +469,7 @@ cmdhelp(int mode, int which)
 				"usage: pw usernext [switches]\n"
 				"\t-V etcdir      alternate /etc location\n"
 				"\t-R rootir      alternate root directory\n"
-				"\t-C config      configuration file\n"
-				"\t-q             quiet operation\n",
+				"\t-C config      configuration file\n",
 				"usage pw: lock [switches]\n"
 				"\t-V etcdir      alternate /etc locations\n"
 				"\t-C config      configuration file\n"
@@ -524,7 +523,6 @@ cmdhelp(int mode, int which)
 				"\t-V etcdir      alternate /etc location\n"
 				"\t-R rootir      alternate root directory\n"
 				"\t-C config      configuration file\n"
-				"\t-q             quiet operation\n"
 			}
 		};
 

Modified: head/usr.sbin/pw/pw_group.c
==============================================================================
--- head/usr.sbin/pw/pw_group.c	Sun Jun  7 21:14:48 2015	(r284138)
+++ head/usr.sbin/pw/pw_group.c	Sun Jun  7 21:57:20 2015	(r284139)
@@ -73,11 +73,8 @@ pw_group(int mode, char *name, long id, 
 	 * next gid to stdout
 	 */
 	if (mode == M_NEXT) {
-		gid_t next = gr_gidpolicy(cnf, id);
-		if (getarg(args, 'q'))
-			return next;
-		printf("%u\n", next);
-		return EXIT_SUCCESS;
+		printf("%u\n", gr_gidpolicy(cnd, id));
+		return (EXIT_SUCCESS);
 	}
 
 	if (mode == M_PRINT && getarg(args, 'a')) {

Modified: head/usr.sbin/pw/pw_user.c
==============================================================================
--- head/usr.sbin/pw/pw_user.c	Sun Jun  7 21:14:48 2015	(r284138)
+++ head/usr.sbin/pw/pw_user.c	Sun Jun  7 21:57:20 2015	(r284139)
@@ -158,14 +158,10 @@ pw_user(int mode, char *name, long id, s
 	 * With M_NEXT, we only need to return the
 	 * next uid to stdout
 	 */
-	if (mode == M_NEXT)
-	{
-		uid_t next = pw_uidpolicy(cnf, id);
-		if (getarg(args, 'q'))
-			return next;
-		printf("%u:", next);
+	if (mode == M_NEXT) {
+		printf("%u:", pw_uidpolicy(cnf, id));
 		pw_group(mode, name, -1, args);
-		return EXIT_SUCCESS;
+		return (EXIT_SUCCESS);
 	}
 
 	/*



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201506072157.t57LvLww030937>