On Mon, Dec 03, 2007 at 01:11:08PM +0100, Karel Zak wrote:
> > + snprintf(mountprog, sizeof(mountprog), "/sbin/mount.%s", types);
> > + if (stat(mountprog, &statbuf) < 0)
> > + error(_(" You might need a helper program: %s"), mountprog);
> This not a good solution. The "types" could be comma-separated list
> of FS types. This solution also produces unexpected suggestions like
> /sbin/mount.ext3...
I knew there was a reason I hadn't applied it to my tree yet... :-(
> What about to use any more generic message like
> "For several filesystems (e.g. nfs, cifs) you might need a
> /sbin/mount.<type> helper program."
I think this would be fine.
lamont
-
To unsubscribe from this list: send the line "unsubscribe util-linux-ng" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
