Hi William, those changes look good.
I have only nit with respect to fix related to support for IPS mirror - could you please file new bug for this issue, since it is not related to 7209 and it might be confusing in future, if somebody would like to track what was fixed in pushed changes. Thanks ! Jan On 03/09/09 18:22, William Schumann wrote: > Jan, > Would you please look at these code changes? > > http://cr.opensolaris.org/~wmsch/bug-7209/ > http://defect.opensolaris.org/bz/show_bug.cgi?id=7209 > > This happened due to losing track of progress through test plan. > > Conditionally compile variable not present in SPARC > > Also found incorrect usage of attribute symbol causing mirror flag to > be omitted from command. > > Tested on SPARC: > - no mirror or alt authority > - primary authority with mirror > - alternate authority w/o mirror > - alternate authority with mirror > > Thank you, > William
