On Tue, Nov 07, 2017 at 12:38:17AM -0700, Theo de Raadt wrote:
| >On Mon, Nov 06, 2017 at 03:24:30PM -0700, Theo de Raadt wrote:
| >| > Here's a diff that will make it default to the device carrying the
| >| > root filesystem.  Result this:
| >| 
| >| I disagree strongly with this.  installboot is a disruptive operation,
| >| and you should specify the device.
| >
| >Fair enough.  I thought it an easy diff that can't do harm (sane
| >defaults), but I see your point.  Thanks for the feedback.
| 
| You must be smoking something really strong.

Well .. I *am* Dutch, so... ;)

| It is never a good default for any program to open a raw partition and
| write something to it, based upon incomplete information.
| 
| All the installboot uses in the tree already know what disk and
| partition to write to, and I can assure you it is not the root
| partition, it tends to be a vnd0.  They are all explicit.
| 
| You can be explicit also.

And I will be.  My attempt at more laziness failed ;)

Paul

-- 
>++++++++[<++++++++++>-]<+++++++.>+++[<------>-]<.>+++[<+
+++++++++++>-]<.>++[<------------>-]<+.--------------.[-]
                 http://www.weirdnet.nl/                 

Reply via email to