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?


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.

Thanks,
Nick

Reply via email to