[EMAIL PROTECTED] wrote:
>> I'm looking for a review of the fix for bug #4996:
>>
>>  http://defect.opensolaris.org/bz/show_bug.cgi?id=4996
>>  OpenOffice package contains a few bogus paths
>>
>> Webrev is at:
>>
>>  http://cr.opensolaris.org/~richb/pkg-4996-v1/
>
> Please sort the list of four drops but otherwise looks good.

Done. Rather than generate a new webrev, here's what the diff
looks like now:

diff -r 979f1be5339d src/util/distro-import/unbundleds/OpenOffice
--- a/src/util/distro-import/unbundleds/OpenOffice    Wed Nov 12 
22:40:49 2008 -0800
+++ b/src/util/distro-import/unbundleds/OpenOffice    Thu Nov 13 
09:15:28 2008 -0800
@@ -37,8 +37,10 @@
 import ooobasis30-xsltfilter
 import openofficeorg3
 import openofficeorg30-desktop-int
+drop etc/mailcap
 drop etc/mime.types
-drop etc/mailcap
+drop share/mime
+drop share/mime/packages
 chattr usr/share/applications/openoffice.org3-printeradmin.desktop \
     target=../../../opt/openoffice.org3/share/xdg/printeradmin.desktop
 import openofficeorg3-base

Thanks.

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

Reply via email to