[EMAIL PROTECTED] wrote: >> I'm looking for a review of the fix for bug #2198: >> >> http://defect.opensolaris.org/bz/show_bug.cgi?id=2198 >> Impractical packaging for OpenOffice >> >> Webrev is at: >> >> http://cr.opensolaris.org/~richb/pkg-2198-v1/ > > Look great, Rich although I'm not sure about including the definition > of openoffice-en-US. I assume you have it for consistency but it seems > awkward as it actually has a dependency on "openoffice" while the > others don't. I think leaving it out makes sense unless there was an > explicit request by the OO team to include it.
Sorry, yes. Bart and I discussed this a little while ago and the openoffice-en-US shouldn't be there. I'll make sure I remove it before I check the fix in. Thanks. _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
