From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with archive (Exim 4.43) id 1MTHo9-00052z-Fi for mharc-grub-devel@gnu.org; Tue, 21 Jul 2009 12:03:13 -0400 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MTHo7-00051C-DO for grub-devel@gnu.org; Tue, 21 Jul 2009 12:03:11 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MTHo2-0004vi-8B for grub-devel@gnu.org; Tue, 21 Jul 2009 12:03:10 -0400 Received: from [199.232.76.173] (port=44393 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MTHo1-0004vU-VF for grub-devel@gnu.org; Tue, 21 Jul 2009 12:03:06 -0400 Received: from c60.cesmail.net ([216.154.195.49]:12989) by monty-python.gnu.org with esmtps (TLS-1.0:RSA_ARCFOUR_SHA1:16) (Exim 4.60) (envelope-from ) id 1MTHo1-0003y9-9p for grub-devel@gnu.org; Tue, 21 Jul 2009 12:03:05 -0400 Received: from unknown (HELO smtprelay1.cesmail.net) ([192.168.1.111]) by c60.cesmail.net with ESMTP; 21 Jul 2009 12:03:03 -0400 Received: from [192.168.0.22] (static-72-92-88-10.phlapa.fios.verizon.net [72.92.88.10]) by smtprelay1.cesmail.net (Postfix) with ESMTPSA id CEC8B34C69 for ; Tue, 21 Jul 2009 12:03:01 -0400 (EDT) From: Pavel Roskin To: The development of GRUB 2 In-Reply-To: References: Content-Type: text/plain Date: Tue, 21 Jul 2009 12:02:59 -0400 Message-Id: <1248192179.27068.18.camel@mj> Mime-Version: 1.0 X-Mailer: Evolution 2.26.3 (2.26.3-1.fc11) Content-Transfer-Encoding: 7bit X-detected-operating-system: by monty-python.gnu.org: Genre and OS details not recognized. Subject: Re: [PATCH] Support FreeBSD in osdetect.lua X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: The development of GRUB 2 List-Id: The development of GRUB 2 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 21 Jul 2009 16:03:11 -0000 On Tue, 2009-07-21 at 15:45 +0200, Vladimir 'phcoder' Serbinenko wrote: > Support FreeBSD on both UFS and ZFS. Requires my zfs.mod for zfs and > my ufs uuid patch for UFS > + grub.add_menu (header .. normal .. footer, title) > + grub.add_menu (header .. " single" .. normal .. footer, > + title .. " (single)") > + grub.add_menu (header .. " verbose" .. normal .. footer, > + title .. " (verbose)") > + grub.add_menu (header .. noacpi .. footer, > + title .. " (without ACPI)") > + grub.add_menu (header .. noacpi .. safe .. footer, > + title .. " (safe mode)") I hate to be "Mister No", but I think this change is not suitable for the generic osdetect.lua. Most users don't have problems with FreeBSD not working on their hardware. Getting five menu entries for different options is too much. It may be OK for a FreeBSD installer, but not for the generic script that also detects other OSes. A reasonable middle ground would be to have one entry called "safe mode" to would lead to a menu offering single mode, disabling ACPI, disabling whatever else is potentially problematic. I think all entries under "safe mode" can enable extra verbosity. The implementation could remove the original menu, or it could insert entries immediately below "safe mode". I don't know if it's possible, but it could be useful. Actually, I don't want osdetect.lua to be overengineered and overloaded with intimate knowledge of different OSes. It's an example for others to extend, not a complete solution. -- Regards, Pavel Roskin