Bharath Kumar wrote:
>>>>> I have updated the webrev with changes as per your comments. Please 
>>>>> help review it again.( 
>>>>> http://cr.opensolaris.org/~bkumarm/sblimcimclient/ )
   49 f none usr/share/lib/sblim/sblimCIMClient.jar 0444 root bin
   50 f none usr/share/lib/sblim/sblimSLPClient.jar 0444 root bin

usr/src/pkgdefs/SUNWSblimCimClient/copyright
- the copyrights statements can be reduced to:
   9 (C) Copyright IBM Corp. 2005-2008
- or:
   9 (C) Copyright IBM Corp. 2005, 2006, 2007, 2008


usr/src/pkgdefs/SUNWSblimCimClient/prototype_com
- has this been through ARC review yet?
- why here?
   49 f none usr/share/lib/sblim/sblimCIMClient.jar 0444 root bin
   50 f none usr/share/lib/sblim/sblimSLPClient.jar 0444 root bin
- instead of under?
usr/share/lib/java

Cheers,
Jim

-- 
Jim Walker, http://blogs.sun.com/jwalker
Sun Microsystems, Software, Solaris QE
x77744, 500 Eldorado Blvd, Broomfield CO 80021

Reply via email to