i386/1549: xfree86 y/n?

[james@squish.net]

>Number:         1549
>Category:       i386
>Synopsis:       xfree86 question could perhaps be explained better
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    bugs
>State:          open
>Class:          change-request
>Submitter-Id:   net
>Arrival-Date:   Fri Dec  8 04:40:01 MST 2000
>Last-Modified:
>Originator:     James Ponder
>Organization:
net
>Release:        2.8
>Environment:
	
	System      : OpenBSD 2.7
	Architecture: OpenBSD.i386
	Machine     : i386
>Description:
	
When you install OpenBSD on i386 it asks:

echo -n "Do you expect to run the X Window System? [y] "

If you install OpenBSD every day, or have done it a hundred times, you
probably know what this means.  Newbies are just going to answer the
question, but an experienced unix sysadmin who has not installed
OpenBSD before is going to see this question and want to know what it's
going to do and why it needs to know at install time.

Can I suggest something along these lines:

echo "Do you expect to run the X Window System?"
echo ""
echo "An affirmative response will enable the kernel option "
echo "machdep.allowaperture which is required to run X.  You can always "
echo "do this by hand later, read xf86(4) for more information."
echo ""
echo -n "Enable aperture driver for X? [y] "
>How-To-Repeat:
	
>Fix:
	
--- install-dist.md	Fri Dec  8 11:35:20 2000
+++ install.md	Fri Dec  8 11:37:43 2000
@@ -76,7 +76,13 @@
 
 md_questions() {
 	echo
-	echo -n "Do you expect to run the X Window System? [y] "
+	echo "Do you expect to run the X Window System?"
+	echo
+	echo "An affirmative response will enable the kernel option "
+	echo "machdep.allowaperture which is required to run X.  You can "
+	echo "always do this by hand later, read xf86(4) for more information."
+	echo
+	echo -n "Enable aperture driver for X? [y] "
 	getresp y
 	case "$resp" in
 		y*|Y*)
@@ -95,7 +101,7 @@
 
 	if [ "$xfree86" = y ]; then
 		echo
-		echo "Enabling machdep.allowaperture. Read xf86(4) for more information."
+		echo "Enabling machdep.allowaperture."
 		echo '1,$s/^#machdep\.allowaperture=1/machdep\.allowaperture=1	/
 w
 q' | ed /mnt/etc/sysctl.conf 2> /dev/null

>Audit-Trail:
>Unformatted: