On 12/06/11 05:53, Petr Cvachoucek wrote:
Hi all,
I'd like to ask for a code review for this CR.

7109218 pkg uninstall application/texinfo-update leads to service going into
maintenance
http://monaco.us.oracle.com/detail.jsf?cr=7109218

Webrev:
http://tbd.cz.oracle.com/~pcvachou/webrevs/7109218-userland-texinfo/

It is just a single-line change in the package manifest file.

While the change seems reasonable, it also seems like we shouldn't have to
manually tag every SMF manifest with a disable_fmri for itself.

Unfortunately, given that pkgdepend currently only outputs the svc fmri as
a set attribute on the package, not the manifest, I think that would require
first modifying pkgdepend, and as long as the pkg team is doing that, they
could just add the disable_fmri tags in there, though that is a bit outside
the normal scope of pkgdepend.

--
        -Alan Coopersmith-        [email protected]
         Oracle Solaris Platform Engineering: X Window System

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

Reply via email to