Jack Schwartz wrote: > Hi everyone. > > Please review the fix for blocker: > 8346 AI should work with NWAM > > This fix is basically the design I posted last Friday, but with better > svc checks. The bug report is current, and the comment in the file > should explain the code changes. > > http://cr.opensolaris.org/~schwartz/090427.1/webrev/ > > Tested by enabling and disabling the different combinations of > svc:network/physical:nwam and svc:network/physical:default > and seeing if installadm starts and/or the appropriate messages are > displayed. > > I would like Sundar and one other person to review please. >
129, 138: please use "enabled", "online", or "in use" rather than "not disabled". Double negatives are often mis-read. Dave
