Spoorthy H.S wrote:
> 
> Please review it now. The changes are at 
> http://cr.opensolaris.org/~spoorthy/cryptws/

1. usr/src/pkgdefs/SUNWperl-crypt-cbc/Makefile
>> You should therefore delete the line ...
>>     29 DATAFILES=depend
>> from your SUNWperl-crypt-cbc/Makefile

it's still in the file

2. usr/src/pkgdefs/SUNWperl-crypt-cbc/copyright
>>>>    It might be better to wrap the 'long' lines in this file
>>
>> I still think you should wrap the lines in here instead of having the 
>> very long lines!

3. usr/src/pkgdefs/SUNWperl-crypt-cbc/prototype_com
     + usr/src/lib/perl_crypt_cbc/METADATA
    The lines with "SUNWperlcrypt-cbc" on should be ..
    "SUNWperl-crypt-cbc"

Reply via email to