fish: When -m option fails, canonicalize device names that are printed.
authorRichard W.M. Jones <rjones@redhat.com>
Mon, 28 Nov 2011 19:12:44 +0000 (19:12 +0000)
committerRichard W.M. Jones <rjones@redhat.com>
Fri, 2 Dec 2011 08:31:45 +0000 (08:31 +0000)
The error message now looks like this:

  guestfish: '/dev/vda6' could not be mounted.  Did you mean one of these?
            /dev/sda1 (ext4)
            /dev/vg_f15x32/lv_root (ext4)
            /dev/vg_f15x32/lv_swap (swap)

Note that '/dev/sda1' has been canonicalized.
(cherry picked from commit f5bfc68fdb3e25b9d75c65e3f5e88983584b25ed)

fish/options.c

index f6cc61f..48c8e1c 100644 (file)
@@ -21,6 +21,8 @@
 #include <stdio.h>
 #include <stdlib.h>
 
+#include "c-ctype.h"
+
 #include "guestfs.h"
 
 #include "options.h"
@@ -98,6 +100,7 @@ add_drives (struct drv *drv, char next_drive)
 }
 
 static void display_mountpoints_on_failure (const char *mp_device);
+static void canonical_device_name (char *dev);
 
 /* List is built in reverse order, so mount them in reverse order. */
 void
@@ -150,6 +153,7 @@ display_mountpoints_on_failure (const char *mp_device)
            program_name, mp_device);
 
   for (i = 0; fses[i] != NULL; i += 2) {
+    canonical_device_name (fses[i]);
     fprintf (stderr, "\t%s (%s)\n", fses[i], fses[i+1]);
     free (fses[i]);
     free (fses[i+1]);
@@ -158,6 +162,17 @@ display_mountpoints_on_failure (const char *mp_device)
   free (fses);
 }
 
+static void
+canonical_device_name (char *dev)
+{
+  if (STRPREFIX (dev, "/dev/") &&
+      (dev[5] == 'h' || dev[5] == 'v') &&
+      dev[6] == 'd' &&
+      c_isalpha (dev[7]) &&
+      (c_isdigit (dev[8]) || dev[8] == '\0'))
+    dev[5] = 's';
+}
+
 void
 free_drives (struct drv *drv)
 {