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

[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

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 and

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