Gerald Jelinek wrote: > # zoneadm -z myzone attach -u -b IDR246802-01 -b 123456-08 > > This new option is brand-specific and only applies to zone brands using > SVr4 packaging.
Given the above I think "-b" is too generic an option name, particularly given that SVR4 packing is not the future direction. When I saw the case synopsis the first thing that came to mind was that "-b" was going to mean backup (not really sure what that would mean though). Given that this is brand specific I think we need to make that obvious, maybe something like this: # zoneadm -z myzone attach -u -x patchbackout=IDR246802-01 -x patchbackout=12345608. I also think it probably makes sense that the list of patches to be backed out could be take from a file eg: # zoneadm -z myzone attach -u -x patchbackoutlist=/tmp/rmpatches Don't take the above as actually complying with CLIP or actually fitting in with the zoneadm CLI they are for illustrative purposes only. Basically I'm saying don't use a single letter option because this is brand specific and the second point of patch list in a file. -- Darren J Moffat
