Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 11 Sep 2018 16:46:28 +0000 (UTC)
From:      Edward Tomasz Napierala <trasz@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r338589 - head/sbin/geom/core
Message-ID:  <201809111646.w8BGkSFX033021@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: trasz
Date: Tue Sep 11 16:46:28 2018
New Revision: 338589
URL: https://svnweb.freebsd.org/changeset/base/338589

Log:
  Minor usability improvements to geom(8).
  
  Approved by:	re (kib)
  MFC after:	2 weeks
  Sponsored by:	DARPA, AFRL

Modified:
  head/sbin/geom/core/geom.c

Modified: head/sbin/geom/core/geom.c
==============================================================================
--- head/sbin/geom/core/geom.c	Tue Sep 11 15:56:06 2018	(r338588)
+++ head/sbin/geom/core/geom.c	Tue Sep 11 16:46:28 2018	(r338589)
@@ -145,8 +145,8 @@ usage(void)
 {
 
 	if (class_name == NULL) {
-		errx(EXIT_FAILURE, "usage: %s <class> <command> [options]",
-		    "geom");
+		fprintf(stderr, "usage: geom <class> <command> [options]\n");
+		exit(EXIT_FAILURE);
 	} else {
 		struct g_command *cmd;
 		const char *prefix;
@@ -191,8 +191,7 @@ load_module(void)
 		/* Not present in kernel, try loading it. */
 		if (kldload(name2) < 0 || modfind(name1) < 0) {
 			if (errno != EEXIST) {
-				errx(EXIT_FAILURE,
-				    "%s module not available!", name2);
+				err(EXIT_FAILURE, "cannot load %s", name2);
 			}
 		}
 	}
@@ -460,7 +459,8 @@ run_command(int argc, char *argv[])
 			usage();
 		}
 		if (!std_available(cmd->gc_name)) {
-			warnx("Command '%s' not available.", argv[0]);
+			warnx("Command '%s' not available; "
+			    "try 'load' first.", argv[0]);
 			exit(EXIT_FAILURE);
 		}
 	}
@@ -644,7 +644,7 @@ get_class(int *argc, char ***argv)
 
 	/* If we can't load or list, it's not a class. */
 	if (!std_available("load") && !std_available("list"))
-		errx(EXIT_FAILURE, "Invalid class name.");
+		errx(EXIT_FAILURE, "Invalid class name '%s'.", class_name);
 
 	if (*argc < 1)
 		usage();
@@ -806,7 +806,7 @@ std_list(struct gctl_req *req, unsigned flags __unused
 	classp = find_class(&mesh, gclass_name);
 	if (classp == NULL) {
 		geom_deletetree(&mesh);
-		errx(EXIT_FAILURE, "Class %s not found.", gclass_name);
+		errx(EXIT_FAILURE, "Class '%s' not found.", gclass_name);
 	}
 	nargs = gctl_get_int(req, "nargs");
 	all = gctl_get_int(req, "all");
@@ -814,8 +814,11 @@ std_list(struct gctl_req *req, unsigned flags __unused
 		for (i = 0; i < nargs; i++) {
 			name = gctl_get_ascii(req, "arg%d", i);
 			gp = find_geom(classp, name);
-			if (gp == NULL)
-				errx(EXIT_FAILURE, "No such geom: %s.", name);
+			if (gp == NULL) {
+				errx(EXIT_FAILURE, "Class '%s' does not have "
+				    "an instance named '%s'.",
+				    gclass_name, name);
+			}
 			list_one_geom(gp);
 		}
 	} else {



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