From owner-svn-src-all@FreeBSD.ORG Sat Dec 3 10:31:21 2011 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id C1A08106564A; Sat, 3 Dec 2011 10:31:21 +0000 (UTC) (envelope-from joel@FreeBSD.org) Received: from mail.vnode.se (mail.vnode.se [62.119.52.80]) by mx1.freebsd.org (Postfix) with ESMTP id 406C28FC0A; Sat, 3 Dec 2011 10:31:21 +0000 (UTC) Received: from mail.vnode.se (localhost [127.0.0.1]) by mail.vnode.se (Postfix) with ESMTP id 590F0E3F07A; Sat, 3 Dec 2011 11:31:19 +0100 (CET) X-Virus-Scanned: amavisd-new at vnode.se Received: from mail.vnode.se ([127.0.0.1]) by mail.vnode.se (mail.vnode.se [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id t1FSdxkNEOPt; Sat, 3 Dec 2011 11:31:16 +0100 (CET) Received: from goofy01.vnodelab.local (unknown [212.247.52.12]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mail.vnode.se (Postfix) with ESMTPSA id 9984BE3F079; Sat, 3 Dec 2011 11:31:15 +0100 (CET) Date: Sat, 3 Dec 2011 11:31:13 +0100 From: Joel Dahl To: Ken Smith Message-ID: <20111203103113.GK23987@goofy01.vnodelab.local> References: <201112020038.pB20cmt6068628@svn.freebsd.org> <20111202094411.GJ23987@goofy01.vnodelab.local> <08BC9075-C4E8-4156-BAAF-C097B118B7B4@cederstrand.dk> <1322832618.99806.8.camel@bauer.cse.buffalo.edu> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1322832618.99806.8.camel@bauer.cse.buffalo.edu> User-Agent: Mutt/1.5.21 (2010-09-15) Cc: src-committers@freebsd.org, Erik Cederstrand , svn-src-all@freebsd.org, Robert Watson , Joel Dahl , svn-src-head@freebsd.org Subject: Re: svn commit: r228192 - head/usr.sbin/bsdinstall/scripts X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 03 Dec 2011 10:31:22 -0000 On 02-12-2011 8:30, Ken Smith wrote: > On Fri, 2011-12-02 at 14:18 +0100, Erik Cederstrand wrote: > > Den 02/12/2011 kl. 10.44 skrev Joel Dahl: > > > > > On 02-12-2011 0:38, Ken Smith wrote: > > >> Author: kensmith > > >> Date: Fri Dec 2 00:38:47 2011 > > >> New Revision: 228192 > > >> URL: http://svn.freebsd.org/changeset/base/228192 > > >> > > >> Log: > > >> Add a screen that asks if the user would like to enable crash dumps, > > >> giving them a very brief description of the trade-offs. Whether the > > >> user opts in or out add an entry to what will become /etc/rc.conf > > >> explaining what dumpdev is and how to turn on/off crash dumps. The folks > > >> who handle interacting with users submitting PRs have asked for this. > > > > > > Hmm. Two things I'd like to bring up: > > > > > > * Not specifically aimed at this commit, but my recommendation > > > would be that we keep bsdinstall as simple as possible: installing FreeBSD > > > should require a minimum amount of keystrokes. I realise this is just one > > > more screen, but I hope we don't turn bsdinstall into a configuration > > > utility where you can disable/enable just about anything in rc.conf. > > > > > > * Mentioning future system crashes during installation feels awkward. Is that > > > really what we want? I understand the problem and how this helps us with > > > debugging, but this is like saying to users that what they are installing > > > is unstable and that it'll eventuelly crash and die. I know we discussed > > > ways of making crash dumps smarter in order to not fill up /var, which in > > > turn would allow us to always have it on. Maybe that is the right path? > > > > Maybe do away with the dialog, and then unconditionally enable textdump in > > rc.conf, adding the magic so textdump_enable="YES" sets up textdump with > > sane defaults. > > > > Thanks, > > Erik > > > > This just implements what we seemed to settle on during the latest round > of several discussions we've had about this issue. It comes up each > I've turned off dumpdev as part of converting head into a stable branch, > which is a step that got started way before I started being the one to > do it. I'll be happy to see what I just added go away but the > pre-condition for that is infrastructure that prevents crash dumps > (of any form, including textdumps) from filling up /var if the user > turns out to be clueless about them and doesn't realize they should > pay attention to them. Thanks for the explanation Ken. I'm fine with this being in the release but I was curious about the discussion resulting in this change (because I obviously didn't see it). It sounds like we've got another project to hack on before 10.0. :-) -- Joel