Re: svn commit: r307469 - head/etc

2016-10-20 Thread Jilles Tjoelker
On Wed, Oct 19, 2016 at 04:45:21PM -0600, Warner Losh wrote: > What missing quotes? It seemed to work find on the system I brought up... In ${root_rw_mount#[Yy][Ee][Ss]} with root_rw_mount=YES, the substring operation results in an empty string, which is removed because there are no quotes. This m

Re: svn commit: r307469 - head/etc

2016-10-19 Thread Warner Losh
What missing quotes? It seemed to work find on the system I brought up... I didn't do the case statement because it would be three extra big useless syntactical boogers in the code :) I do like the idea of using checkyesno. Warner On Wed, Oct 19, 2016 at 2:26 PM, Jilles Tjoelker wrote: > On Mo

Re: svn commit: r307469 - head/etc

2016-10-19 Thread Jilles Tjoelker
On Mon, Oct 17, 2016 at 04:07:13AM +, Warner Losh wrote: > Author: imp > Date: Mon Oct 17 04:07:13 2016 > New Revision: 307469 > URL: https://svnweb.freebsd.org/changeset/base/307469 > Log: > Allow root_rw_mount to be both lower and upper case. Before, if it was > upper case, you'd wind up

svn commit: r307469 - head/etc

2016-10-16 Thread Warner Losh
Author: imp Date: Mon Oct 17 04:07:13 2016 New Revision: 307469 URL: https://svnweb.freebsd.org/changeset/base/307469 Log: Allow root_rw_mount to be both lower and upper case. Before, if it was upper case, you'd wind up with a read-only filesystem when you should sometimes. PR: 213549