Anurag S. Maskey wrote: > > > Alan Maguire wrote: >> Anurag S. Maskey wrote: >>> http://zhadum.east/export/ws/am223141/checkout-area/nwam1-fixes/webrev/ >>> >>> 13893 duplicating NCP does not generate "add" event >>> http://defect.opensolaris.org/bz/show_bug.cgi?id=13893 >>> >> Looks good, but I think we need to destroy the >> to-be-copied-to ncp in error cases? > The new NCP right? If the walk to copy the NCUs fail, then the new NCP > is destroy in line 378. > oops, must have missed that.
Anurag S. Maskey wrote: > http://zhadum.east/export/ws/am223141/temp/nwam1-work/webrev/ > > 13897 spaces in NCP name not recognized in walks > http://defect.opensolaris.org/bz/show_bug.cgi?id=13897 looks good. Alan
