On Sat, 23 Dec 2000, Adrian Chadd wrote:

> Here's the patch:
> 
> --- fsck.c.orig Sat Dec 23 11:13:30 2000
> +++ fsck.c      Sat Dec 23 11:13:34 2000
> @@ -501,7 +501,7 @@
>                 errx(1, "partition `%s' is not of a legal vfstype",
>                     str);
> -       if ((vfstype = dktypenames[t]) == NULL)
> +       if ((vfstype = fstypenames[t]) == NULL)
>                 errx(1, "vfstype `%s' on partition `%s' is not supported",
>                     fstypenames[t], str);
> 
> 
> So now is a problem which I'm sure the NetBSD people came up against.
> The fstypenames are names like 4.2BSD, vinum, ISO9660, etc. NetBSD fixed
> this by creating a new list 'mountnames[]', which maps the fs type to
> a string.

fs typenames are already strings in FreeBSD (the kernel's vfc_index is an
implementation detail which should not be visible in applications).

> http://cvsweb.netbsd.org/bsdweb.cgi/syssrc/sys/sys/disklabel.h.diff?r1=1.60&r2=1.61
> 
> What do people think about doing this as well? It would certainly make things
> a little tidier, but every time a new fs comes in the magic autodetection code
> will need to be updated (if appropriate, of course.)

This would be a bug.

Bruce



To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message

Reply via email to