[email protected] wrote:
I'd appreciate a review of the following modified webrev

    http://cr.opensolaris.org/~comay/webrev-11817/

which covers fixes for the following issues

    11817 VSU: localedef generates a file that is missing 2 header files
    12014 incorrect package dependency for SUNWapch22m-php5
    12032 SUNWgtk2-print-papi needs to be added to slim_install for b126
    12153 resync repository to snv_126
    12155 driver action for amd_iommu is delivered in the wrong package
    12156 /etc/inet/{hosts,services} are delivered with the wrong mode

* 125/common/SUNWcs
 I personally prefer the "add file" tokens on multiple lines.
 It's easier to quickly see what's what. Hey, but whatever.

* 126/common/SUNWgtk2-print-cups
 Shouldn't this depend upon the CUPS package(s)?

* 126/common/SUNWgtk2-print-papi
 Shouldn't this depend upon the PAPI package(s)?

* 126/common/SUNWhea
 2 is not "several". :-)

_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to