Subject: Re: CVS commit: src/distrib/utils/sysinst
To: None <xtraeme@NetBSD.org>
From: Izumi Tsutsui <tsutsui@ceres.dti.ne.jp>
List: source-changes
Date: 06/19/2007 21:22:46
xtraeme@NetBSD.org wrote:

> Module Name:	src
> Committed By:	xtraeme
> Date:		Mon Jun 18 16:58:42 UTC 2007
> 
> Modified Files:
> 	src/distrib/utils/sysinst: disks.c
> 
> Log Message:
> Mount a ptyfs on /dev/pts by default. I'm committing it because
> I did not have any answer for 10 days.

I think more verbose log message (and a call for review) is preferred.

I guess it should be:
"Make sysinst also create a default entry of a ptyfs on /dev/pts
 in /etc/fstab on the target disk as well as kernfs and procfs,"
but your log message may imply that INSTALL kernels should have
"file-system PTYFS" option.

> http://mail-index.netbsd.org/tech-install/2007/06/08/0001.html

It might also be better to post diffs to a list directly
rather than via URL (unless it's too large to post) so that
more people would take a look at and comment about it properly.

---
@@ -497,11 +497,13 @@
 				tmp_mfs_size);
 	}
 
-	/* Add /kern and /proc to fstab and make mountpoint. */
+	/* Add /kern, /proc and /dev/pts to fstab and make mountpoint. */
 	scripting_fprintf(f, "kernfs\t\t/kern\tkernfs\trw\n");
+	scripting_fprintf(f, "ptyfs\t\t/dev/pts\tptyfs\trw\n");
 	scripting_fprintf(f, "procfs\t\t/proc\tprocfs\trw,noauto\n");
 	make_target_dir("/kern");
 	make_target_dir("/proc");
+	make_target_dir("/dev/pts");
 
 	scripting_fprintf(NULL, "EOF\n");
 
---
Izumi Tsutsui