Date: Sat, 2 Nov 2013 22:18:00 -0400 From: Outback Dingo <outbackdingo@gmail.com> To: Devin Teske <dteske@freebsd.org> Cc: "<freebsd-current@freebsd.org> Current" <freebsd-current@freebsd.org>, "Teske, Devin" <Devin.Teske@fisglobal.com> Subject: Re: [CFT] bsdinstall and zfsboot enhancements Message-ID: <CAKYr3zy4FNYWOkHzJXk1YW7VJW3adb97xwfaMHyngATtY3G8ag@mail.gmail.com> In-Reply-To: <C9783B1F-20EA-4C08-9947-70DF363E8B6A@fisglobal.com> References: <C9783B1F-20EA-4C08-9947-70DF363E8B6A@fisglobal.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, Nov 2, 2013 at 1:59 PM, Teske, Devin <Devin.Teske@fisglobal.com>wrote: > Hi all, > > Another Call For Testing... > This one is for bsdinstall. > > Two patchsets are required for this CFT: > > http://druidbsd.cvs.sf.net/viewvc/druidbsd/bsdinstall_debug/ > http://druidbsd.cvs.sf.net/viewvc/druidbsd/bsdinstall_zfs/ > > The enhancements are: > > + Add a `-D FILE" command-line option for overriding the > path to the bsdinstall log file (BSDINSTALL_LOG env var). > > + Document new `-D FILE' in the man page for bsdinstall. > > + If FILE in `-D FILE' begins with a +, debug output goes to > stdout (interleaved between dialog(1) invocations/output) as > well as to FILE (minus the leading + of course). > > + If BSDINSTALL_LOG cannot be written, then debugging is > disabled (except in the case of a leading + in the pathname, > wherein debug will still be printed to stdout). > > + Update source code format to approximate a future shstyle(9) > > + Fix a dangling participle ("Begun ..." -> "Began ...") > > + Rewrite the docsinstall script (was necessary to abate direct > dependency on BSDINSTALL_LOG (instead, use fault-tolerant > bsdconfig framework which displays appropriate errors for > package management). > > + Add additional debug output for dhclient/rtsol/wpa_cliscan > > + Display script errors in a textbox rather than just on stdout > > + Update many coments. > > + Add new f_show_err() API call (like f_show_msg but changes > the dialog title to "Error")(see bsdconfig's `common.subr'). > > + Add new f_eval_catch() API call for executing a command via > eval but not before logging the command to debug. Several > example cases documented in API header for function in > bsdconfig's `common.subr'. > > + Fix dialog auto-sizing when launched as an rvalue to a pipe > for indirected scripts (previously would default to 80x24 sizing > in this case, now it can autosize to full size even when in a pipe > chain). > > + Fix a bug in f_snprintf wherein if the $format argument began > with a "-" or "--" it would be misinterpreted as a flag to printf. (this > is in bsdcofig's strings.subr) > > + Add accompanying f_sprintf() and f_vsprintf() to go along with > already existing f_snprintf() and f_vsnprintf() (see bsdconfig's > strings.subr). > > + Update bsdinstall's "config" script to adjust ttyu* entries in > /etc/ttys when it is determined that we are in-fact doing an install > over serial (e.g. bhyve). > > + Remove some unnecessary default ZFS datasets from the > automatic "zfsboot" script. Such as: /usr/ports/distfiles > /usr/ports/packages /usr/obj /var/db /var/empty /var/mail and > /var/run (these can all be created as-needed once the system is > installed). > > + Remove setuid=off for /usr/home (as discussed with others from > last round of CFT). > > + Fix some i18n string violations in "zfsboot" > > + Bolster debugging output in "zfsboot" > > + Fix some string quoting issues in "zfsboot" > > + Fix some variable scope issues in "zfsboot" > > + Only display the ZFS vdev type selection menu when running > interactively (duh). > > + Change "Create" to "Install" in "zfsboot" main menu > > + Increase pedantic error checking in "zfsboot" (type-check > arguments and such). > > + Add a call to "graid destroy" to kill automatic metadata (part of > the series of pedantic destructions we do when bootstrapping > a new/naked disk). > > + Make judicious use of new f_eval_catch() in "zfsboot" > > + More comments (zfsboot). > > + Fixup some variable names for consistency (zfsboot). > > nice work, looks good, patched the bootloader also, seems to function well, love the kernel selection variable > -- > Devin > > _____________ > The information contained in this message is proprietary and/or > confidential. If you are not the intended recipient, please: (i) delete the > message and all copies; (ii) do not disclose, distribute or use the message > in any manner; and (iii) notify the sender immediately. In addition, please > be aware that any message addressed to our domain is subject to archiving > and review by persons other than the intended recipient. Thank you. >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAKYr3zy4FNYWOkHzJXk1YW7VJW3adb97xwfaMHyngATtY3G8ag>