Mainly looks good to me, but see comments below ... Paul
Vivek Titarmare wrote: > I have posted a webrev for package "xstream" which I am porting to > OpenSolaris and would like to request a code review. > > Please see below link > http://cr.opensolaris.org/~vivekrt/6816355-xstream/ 1. pkg version Why are you integrating version 1.3 when there is 1.3.1 ? 2. usr/src/lib/xstream/install-sfw Should there be a symbolic link for xstream-benchmark.jar ? 3. usr/src/pkgdefs/SUNWxstream/copyright Maybe it should say what licence it is END -- ---------------------------------------------------------------------- Paul Cunningham Software Engineer Tadpole Business Unit
