Re: svn commit: r216291 - head/cddl/contrib/opensolaris/lib/libzfs/common

2010-12-08 Thread Pawel Jakub Dawidek
On Wed, Dec 08, 2010 at 10:14:13AM +0100, Martin Matuska wrote: > Here are some arguments: > > a) it was removed in OpenSolaris rev. 12865:08987220809a > b) a root pool does not not have to be a boot pool > > So if we need a warning here, it should print only some general > suggestion because > u

Re: svn commit: r216291 - head/cddl/contrib/opensolaris/lib/libzfs/common

2010-12-08 Thread Martin Matuska
Here are some arguments: a) it was removed in OpenSolaris rev. 12865:08987220809a b) a root pool does not not have to be a boot pool So if we need a warning here, it should print only some general suggestion because user may attach any vdev (e.g. a file or a memory disk) to make a mirror from a r

Re: svn commit: r216291 - head/cddl/contrib/opensolaris/lib/libzfs/common

2010-12-08 Thread Pawel Jakub Dawidek
On Wed, Dec 08, 2010 at 08:57:37AM +, Martin Matuska wrote: > Author: mm > Date: Wed Dec 8 08:57:37 2010 > New Revision: 216291 > URL: http://svn.freebsd.org/changeset/base/216291 > > Log: > Do not print OpenSolaris hint to use (non-existing) installgrub(1) command > if creating a mirror

svn commit: r216291 - head/cddl/contrib/opensolaris/lib/libzfs/common

2010-12-08 Thread Martin Matuska
Author: mm Date: Wed Dec 8 08:57:37 2010 New Revision: 216291 URL: http://svn.freebsd.org/changeset/base/216291 Log: Do not print OpenSolaris hint to use (non-existing) installgrub(1) command if creating a mirror by attaching a new vdev to a root pool. Reported by: James R. Van Artsdale