Looks good to me. Jean
Willie Walker wrote: > Hey All (especially Jack, Jean, and Karen): > > This is my first code review request for Caiman, so I hope I'm giving you the > right info. Please review the changes to enable accessibility (a11y) for the > live CD. > > Bug 3785 - Add accessibility support to the 2008.11 live CD > http://defect.opensolaris.org/bz/show_bug.cgi?id=3785 > > Webrev: http://cr.opensolaris.org/~ww36193/slim_3785/ > > The description of the changes is in comment #1 of the bug report: > http://defect.opensolaris.org/bz/show_bug.cgi?id=3785#c1 > > TESTING: > > Systems used: VirtualBox on my AMD-64 Opteron desktop > DVD media on my Toshiba Tecra M2 laptop > > Procedures: > 1) Boot with the default option. Make sure no a11y > features are enabled on the system. > 2) Boot with the screen reader option. Make sure a11y > features were enabled and the screen reader started. > Install. Verify that the only change that was copied > to the system was the /lib/inetd/nwamd wrapper. > 3) Same as #2, but with magnifier. > > NOTES: > > JackS is working on http://defect.opensolaris.org/bz/show_bug.cgi?id=3600 to > support adding grub options to the manifest. If 3600 gets done, the > finalizer > from this patch can be removed and replaced with additional grub lines in > slim_cd.xml. > > Since development freeze is Monday, though, I wanted to get this reviewed > soon > so that I could start work on any suggested changes as soon as possible. > > Thanks! > > Will > > _______________________________________________ > caiman-discuss mailing list > caiman-discuss at opensolaris.org > http://mail.opensolaris.org/mailman/listinfo/caiman-discuss >
