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

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

which contains fixes for the following

    8031 IPS changes to address trusted zone installation breakage
    8332 Wrong permissions on git perl include files in SUNWgit package
    8404 webstack ui package incorrectly depends on SUNWphp52d
    8435 nmap package missing dependency on SUNWpysqlite
    8467 SUNWapch22 is missing a dependency on SUNWapu13-ldap => crash
    8546 Korean characters are garbaged on java
    8626 resync repository to snv_111b
    8643 upgrading causes usr/sfw/man to disappear

Ric/Jerry, please feel free to review the whole webrev but I put the
Trusted files at the top of the webrev for easier reviewing.

David,

I reviewed the files related to trusted and they look
ok to me except for the
src/util/distro-import/111a/SUNWtsoldef.xml file.
This is the first time I think I've seen this file
and it doesn't seem right.  It has a bunch of
inherited-pkg-dir entries in it, so thats not going
to work with a zone managed by IPS.  Can you clarify
whats going on with this zone definition?  Maybe I
just don't understand what is supposed to happen to
the labeled-branded zones after you image-update the
system and then try to update those branded zones.

Thanks,
Jerry


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

Reply via email to