Hi Jyri ,

The changes have been incorporated .Could you please review it again at 
the same link http://cr.opensolaris.org/~sunandam/6640026

Thanks Sunanda

Jyri Virkki wrote:
> Sunanda Menon wrote:
>   
>> Hello Jyri ,
>>
>> The code changes have been done with _install P option .Please review 
>> again at http://cr.opensolaris.org/~sunandam/6640026 and let me know 
>> your comments .
>>     
>
> Slightly unusual usage of _install but I guess if there are a ton of
> perl scripts that might be easiest way.
>
> Never use a fixed common path like "/tmp/text" - if multiple builds
> are running at the same time and happen to execute this same code
> section at the same time, there's going to be some very confusing
> results... At least use $$ as part of the name so multiple of these
> build processes can coexist.
>
>
>   


-- 
Sunanda Menon
Database Technology Group
BLR03, x87098/91-80-66937098


Reply via email to