On Mon 22 Feb 2010 at 10:52AM, Jerry Jelinek wrote:
> On 02/22/10 09:40, Frank Batschulat (Home) wrote:
> >May I have 2 code reviewers for the following minor changes for:
> >
> >PSARC/2010/008 Remove zoneadm install sub-option "-x nodataset"
> >6880288 retire zoneadm install -x nodataset option
> >6890415 zoneadm install fails but returns 0
> >
> >http://cr.opensolaris.org/~batschul/nodataset/
> 
> Frank,
> 
> This looks good to me.

To me as well, although I would be tempted to call temp_err
something like cleanup_err.

        -dp

-- 
Daniel Price, Solaris Kernel Engineering    http://blogs.sun.com/dp
_______________________________________________
zones-discuss mailing list
zones-discuss@opensolaris.org

Reply via email to