Peter Memishian writes:
>  > > The external webrev is here:
>  > > 
>  > >   http://cr.opensolaris.org/~carlsonj/webrev-4783123/
>  > > 
>  > > An equivalent internal one is here:
>  > > 
>  > >   http://zhadum.east/ws/carlsonj/4783123-fix/webrev/
> 
> Sorry for the delay.  I didn't look at the NWAM changes.  Things look
> good; just some minor comments:

Updated webrevs are here:

        http://zhadum.east/ws/carlsonj/4783123-fix/webrev/
        http://cr.opensolaris.org/~carlsonj/webrev-4783123/

Deltas from the previous webrev:

        http://zhadum.east/ws/carlsonj/4783123-fix/webrev-delta/
        http://cr.opensolaris.org/~carlsonj/webrev-4783123-delta/

Copies of the previous webrev:

        http://zhadum.east/ws/carlsonj/4783123-fix/webrev-old/
        http://cr.opensolaris.org/~carlsonj/webrev-4783123-old/

I've retested, and after updating the CRs, filing the related man page
CRs, and getting the PSARC case approved (there's one OpenSolaris
related issue I need to chase down), I'll file the RTI.  (Unless there
are more comments ...)

-- 
James Carlson, Solaris Networking              <james.d.carlson at sun.com>
Sun Microsystems / 35 Network Drive        71.232W   Vox +1 781 442 2084
MS UBUR02-212 / Burlington MA 01803-2757   42.496N   Fax +1 781 442 1677

Reply via email to