Hi Takeshi,

> One comment: There are several lines stay as:
> classification "System/Localizations (System)"
> while others have been changed to:
> classification "System/Localizations"

Yup. I missed the "System/Localizations (System)"
lines for the new package clusters in LangSupport,
because they were in the patch for bug #4214 when
I did the previous adjustment.

New webrev at:

  http://cr.opensolaris.org/~richb/pkg-4214-v2/
 
> Regarding other than classification lines, I verified
> that your fix agrees with what I posted as suggested fix.

Good.

> (Thank you for correcting the missing "depend" issue.)

No problem.

Thanks.


> On 2008年11月05日 00:37, Rich Burridge wrote:
>> Hi all,
>>
>> I'm looking for a review of the fix for bug #4214:
>>
>>   http://defect.opensolaris.org/bz/show_bug.cgi?id=4214
>>   language support cluster update as of 2008.11 snv101
>>
>> Webrev is at:
>>
>>   http://cr.opensolaris.org/~richb/pkg-4214-v1/
>>
>> Tested by publishing the various language package clusters into
>> a local repository with:
>>
>> #! /bin/ksh
>> export ROOT=/export/home/richb/pkg/bugs/4214/gate/proto/root_i386
>> export PYTHONPATH=${ROOT}/usr/lib/python2.4/vendor-packages
>> export REPO=http://localhost:24214/
>> export 
>> WOS_PKGS=/export/home/pkg-solaris/WOS_Packages/x/101/Solaris_11/Product/
>>
>> (cd src/util/distro-import ;    \
>>         make -e BUILDID=101 LangSupport.import
>> ) > errs.cluster 2>&1 &
>>
>> Local repository is at:
>>
>>   http://stard.sfbay.sun.com:24214
>>
>> Thanks.
>>
>> _______________________________________________
>> pkg-discuss mailing list
>> [email protected]
>> http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
>

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

Reply via email to