Re: [patch] nextboot(8) arbitrary kernel environment

2012-01-31 Thread John Baldwin
On Monday, January 30, 2012 2:57:22 pm Ed Maste wrote: > I have a patch to allow nextboot(8) to set arbitrary kernel environment > variables (not just the kernel dir and kernel_options). The usage becomes: > > Usage: nextboot [-e variable=value] [-f] [-k kernel] [-o options] >nextboot -D

Re: [patch] nextboot(8) arbitrary kernel environment

2012-01-30 Thread Ed Maste
On Mon, Jan 30, 2012 at 01:12:21PM -0700, Ian Lepore wrote: > Minor nit: > > -It is not the most thoroughly tested code. > +It is not the most throughly tested code. > > The original spelling is the correct one. Nice catch, and thanks for the review. This came from an accidental revert of

Re: [patch] nextboot(8) arbitrary kernel environment

2012-01-30 Thread Ian Lepore
On Mon, 2012-01-30 at 14:57 -0500, Ed Maste wrote: > I have a patch to allow nextboot(8) to set arbitrary kernel environment > variables (not just the kernel dir and kernel_options). The usage becomes: > > Usage: nextboot [-e variable=value] [-f] [-k kernel] [-o options] >nextboot -D > >

[patch] nextboot(8) arbitrary kernel environment

2012-01-30 Thread Ed Maste
I have a patch to allow nextboot(8) to set arbitrary kernel environment variables (not just the kernel dir and kernel_options). The usage becomes: Usage: nextboot [-e variable=value] [-f] [-k kernel] [-o options] nextboot -D and the new option is documented as: -e variable=value