Ethan,
> Tim, > > One more comment in beadm.1m.txt > > 41 - Do we really need this line here? Its not a special individual No. It's nuked. Thanks Tim > operation, so why are we highlighting it? When 1833 gets fixed and > we have something like a "beadm description ... " or whatever > subcommand, that would be something we'd add to this list. > > > thanks, > -ethan > > > Tim Knitter wrote: >> Ethan Quach wrote: >> >>> Ethan Quach wrote: >>> >>>> Tim Knitter wrote: >>>> >>>> >>>>> Ethan Quach wrote: >>>>> >>>>>> Tim, >>>>>> >>>>>> The code changes look fine. My only comment is on the man page. >>>>>> Ideally, I'd rather us not mention the use of 'description' as >>>>>> being used >>>>>> as the title in the grub menu at all if possible. ~But~ if this >>>>>> *is* something >>>>>> we do want to publicize, then it should be stated as a side note >>>>>> rather than >>>>>> the only use of 'description'. i.e. something like: >>>>>> >>>>>> -d description Create a new BE with a description >>>>>> associated with it. This description >>>>>> is also used as the title for this BE's >>>>>> entry in the GRUB menu. >>>>>> >>>>>> >>>>> I like this better than the example I passed back to Barbara >>>>> earlier with the addition of the last sentence I added. >>>>> >>>>> "If a description is not supplied, beName will be used as the entry >>>>> in the GRUB menu." >>>>> >>>>> >>>> Hmm, could we instead move the last two sentences out into the main >>>> beadm create paragraph above? >>>> >>> .. the reason for this request is that with this additional sentence, >>> seems like >>> we're starting to describe behavior outside of what's just inherent >>> for the option. >>> >>> >> >> Agreed. Webrev updated. >> >> Thanks >> Tim >> >> >>> -ethan >>> >>> >>>> thanks, >>>> -ethan >>>> >>>> >>>> >>>> >>>>> >>>>>> Also, could you add an Example to show how to quote the description >>>>>> text on the command line. >>>>>> >>>>>> >>>>> Done. >>>>> See updated webrev. >>>>> >>>>> Thanks >>>>> Tim >>>>> >>>>> >>>>>> thanks, >>>>>> -ethan >>>>>> >>>>>> >>>>>> Tim Knitter wrote: >>>>>> >>>>>>> In order to test the functionality that this code exposes, beadm >>>>>>> has also been >>>>>>> modified to make use of the new interface in libbe.c and is being >>>>>>> addressed by >>>>>>> this bug. A [-d description] opt & arg is beaing added to the >>>>>>> beadm usage for create as follows: >>>>>>> >>>>>>> beadm create [-a] [-d description] [-e non-activeBeName | >>>>>>> beName at snapshot] >>>>>>> [-o property=value] ... [-p zpool] beName >>>>>>> >>>>>>> Please see the updated webrev that includes the changes to make >>>>>>> the above happen. >>>>>>> >>>>>>> These changes have been successfully tested on bld99. >>>>>>> >>>>>>> Thanks >>>>>>> Tim >>>>>>> >>>>>>> Tim Knitter wrote: >>>>>>> >>>>>>> >>>>>>>> Caimaniacs, >>>>>>>> >>>>>>>> Please review the following simple change: >>>>>>>> >>>>>>>> 4054 beCopy() needs to take parameter for BE description >>>>>>>> >>>>>>>> WR: http://cr.opensolaris.org/~tsk/4054/ >>>>>>>> B: http://defect.opensolaris.org/bz/show_bug.cgi?id=4054 >>>>>>>> >>>>>>>> This has been tested on bld99 with pkg(1) and beadm(1M). >>>>>>>> >>>>>>>> Thanks >>>>>>>> Tim >>>>>>>> _______________________________________________ >>>>>>>> caiman-discuss mailing list >>>>>>>> caiman-discuss at opensolaris.org >>>>>>>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss >>>>>>>> >>>>>>> _______________________________________________ >>>>>>> caiman-discuss mailing list >>>>>>> caiman-discuss at opensolaris.org >>>>>>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss >>>>>>> >>>> _______________________________________________ >>>> caiman-discuss mailing list >>>> caiman-discuss at opensolaris.org >>>> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss >>>> >> _______________________________________________ >> caiman-discuss mailing list >> caiman-discuss at opensolaris.org >> http://mail.opensolaris.org/mailman/listinfo/caiman-discuss >>
