On 04/19/12 04:24 AM, Peter Dennis wrote:
> 
> 
> On 04/18/12 17:06, Peter Dennis wrote:
>> Hi,
>>
>> Hopefully a trivial review for the following CR's:
>>
>> 7161508 add driver/graphics/mga to solaris-large-server
>> 7162452 update large server group package for key-management package
>>
>> Webrev:
>>
>> https://cr.opensolaris.org/action/browse/onnv/petede/group_package_update/webrev/
>>
>>
>> Changes:
>>
>> Add in the line for the mga driver for SPARC
>> Modify the package name for the key-management package
>> (kept the packages in alphabetical order).
> 
> Based upon feedback I have extended the inclusion of the driver to the
> following group packages:
> 
> solaris-small-server
> solaris-large-server
> solaris-auto-install
> multi-user-desktop
> solaris-desktop
> 
> The last two also include the addition of the fbconfig-mga package.
> 
> This mirrors the actions for the other sparc graphics drivers (efb
> and ast).
> 
> Webrev:
> 
> https://cr.opensolaris.org/action/browse/onnv/petede/group_package_update/webrev/

Looks good to me.   (I'm confused about a bunch of the existing crap in the
multi-user-desktop group package, but that's a different issue, not caused
by this change.)


-- 
        -Alan Coopersmith-              [email protected]
         Oracle Solaris Engineering - http://blogs.oracle.com/alanc
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to