CVS commit: src

["David A. Holland" <dholland%netbsd.org@localhost>]

Module Name:    src
Committed By:   dholland
Date:           Sat Jul 12 16:11:27 UTC 2014

Modified Files:
        src/distrib/utils/sysinst: disks.c
        src/sbin/fsck_lfs: fsck_lfs.8 main.c

Log Message:
Revert version 1.111 of sysinst/disks.c, which caused sysinst to
create lfs fstab entries with fsck disabled, and instead patch
fsck_lfs to exit successfully without doing anything when given the -p
(bootup preen) option. If you really want to do fsck_lfs -p, you can
do fsck_lfs -f -p to make it go.

This has been sitting in my todo queue since February 2010 and was
ok'd by the committer at the time. The original commit was based on
this post:
   http://mail-index.netbsd.org/tech-kern/2010/02/09/msg007306.html

and I remain unconvinced that it's the right thing, but we can at
least do it properly and not ship a sysinst with -7 that creates
permanently wrong fstab files.

Note that this may cause problems for anyone who's taken -p out of the
bootup fsck flags; but doing that is wrong, so don't.


To generate a diff of this commit:
cvs rdiff -u -r1.130 -r1.131 src/distrib/utils/sysinst/disks.c
cvs rdiff -u -r1.24 -r1.25 src/sbin/fsck_lfs/fsck_lfs.8
cvs rdiff -u -r1.45 -r1.46 src/sbin/fsck_lfs/main.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.