usr/src/common/rbac/exec_attr Should there be a postinstall script or something to get these lines added during an upgrade?
usr/src/pkgdefs/SUNWwireshark/depend Why is bash in the dependencies? (just curious:) usr/src/pkgdefs/SUNWwireshark/pkginfo.tmpl I think "usr" type packages normally have: SUNW_PKG_ALLZONES="true" [note 1] usr/src/pkgdefs/SUNWwiresharkr/copyright There is nothing but Sun-owned content in this package, so this should use the standard Sun copyright. (just delete your copyright file in here) usr/src/pkgdefs/SUNWwiresharkr/pkginfo.tmpl I think "root" packages normally have: SUNW_PKG_ALLZONES="true" It might also be appropriate to set: SUNW_PKG_HOLLOW="true" [note 1] note 1: You should get advice from someone who knows the package rules better than I do. Let me know if you have trouble finding someone for that. Otherwise looks good to me. Thanks, Gordon
