Lucia Lai wrote:
> On 01/14/09 13:10, Nicholas Solter wrote:
>> Yue Lai wrote:
>>   
>>> Hi,
>>>
>>> Please review the fix to CR 6792249 at
>>>
>>> http://cr.opensolaris.org/~llai/6792249/
>>>
>>> Thanks,
>>>
>>> - Lucia
>>>     
>>
>> Lucia,
>>
>> usr/src/cmd/scsnapshot/scsnapshot.ksh
>> usr/src/lib/scsnapshot/ScSnapshot/Makefile
>>
>> Can you add some comments explaining why you're changing the way the 
>> include paths are specified, so that in the future we don't make the 
>> same mistake?
>>   
> Will add comments to both files.
> 
>>
>> usr/src/lib/libexpat/xml_parser/Makefile
>>
>> Is there any way we can avoid hard-coding a perl version? Otherwise 
>> we'll face the same problem again next time the version is reved and the 
>> old version is not available.
>>   
> I thought about it, but since it uses $(PERL_DIR)/lib, and there is no 
> generic way to refer to in different versions of Perl. If you have any 
> ideas let me know.

Sorry, I don't have any ideas. For java, /usr/java is a symlink to 
whatever version of java the system is using, but there doesn't seem to 
be something similar for perl, that I know of.

Thanks,
Nick

> 
> Thanks,
> 
> - Lucia
>> Thanks,
>> Nick
>> _______________________________________________
>> ha-clusters-discuss mailing list
>> ha-clusters-discuss at opensolaris.org
>> http://mail.opensolaris.org/mailman/listinfo/ha-clusters-discuss
>>   
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> ha-clusters-discuss mailing list
> ha-clusters-discuss at opensolaris.org
> http://mail.opensolaris.org/mailman/listinfo/ha-clusters-discuss


Reply via email to