Subject: CVS commit: src/distrib/utils/sysinst
To: None <source-changes@NetBSD.org>
From: David Laight <dsl@netbsd.org>
List: source-changes
Date: 07/28/2003 11:32:22
Module Name:	src
Committed By:	dsl
Date:		Mon Jul 28 11:32:22 UTC 2003

Modified Files:
	src/distrib/utils/sysinst: main.c run.c

Log Message:
Remove extra process and pipe that relayed data from the master pty
to sysinst's main process when a command is run (eg the file list from pax).
Since the pty is in 'packet' mode, the boundaries of the pty reads would
need to be preserved in order to detect control records.
If the pipe became full the main process could treat part of a filename
as a control byte and then save the rest of the name as tty settings.
The effect of 'random' tty settings is unknown, but could be responsible
for core dumps while displaying the full file list from pax.
I suspect that the extra pipe was originally used because select wasn't
supported on the master side of a pty (not checked).

Also cd out of /targetroot if set installation is killed with ^C ^C.


To generate a diff of this commit:
cvs rdiff -r1.40 -r1.41 src/distrib/utils/sysinst/main.c
cvs rdiff -r1.47 -r1.48 src/distrib/utils/sysinst/run.c

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