Dan Anderson wrote: > Here's a simple review to change default from cde-login to gdm: > > http://dan.drydog.com/reviews/6923877-smf/ > > This needed because cde-login no longer exists in build snv_133. This > change is already in the Indiana/IPF packages, but not the Nevada/SVR4 > packages and is needed for testing (as there's no graphical login > without the workaround of "svcadm enable gdm").
It'd probably be better to insert it into a section other than "Enable CDE services", as it's not a CDE service. It may actually be helpful to style your changes after those made in Indiana already, as far as location and commentary. -- Rich
