Comments below .. Paul
Vivek Titarmare wrote: > I have posted a webrev for package "tipc" which I am porting to Neveda > and would like to request a code review. Please see below link > > http://cr.opensolaris.org/~vivekrt/6833850-tipc/ 1. usr/src/lib/tipc/Makefile.sfw Change 'env ' to 'env - ' throughout Do you need ? .. 81 (cd $(CODE_PATH)/$(VER)/usr/src ; \ 82 env "SRC=$(CODE_PATH)/$(VER)/usr/src" \ 83 "MAKEFLAGS=e" \ 84 make clean) when you then do ... 85 rm -rf $(VER) Apply the patches in the '$(VER):' rule rather than in the 'all:' rule I assume the .. 72 make release builds and installs the files; only so the build in the 'all:' rule, then do the install in the 'install:' rules 2. Tops of files (various) Cosmetic: Change so that they conform to those in ... "http://src.opensolaris.org/source/xref/onnv/onnv-gate/usr/src/prototypes/"; ie, mainly add missing line space after CDDL HEADER END 3. usr/src/lib/tipc/install-sfw Delete line as its not used .. 30 TOP=`pwd` Replace with a line-space Add a 'set -o errexit' as in .. "http://src.opensolaris.org/source/xref/sfw/usr/src/cmd/meld/install-sfw"; 4. patches Note, I haven't looked at these 5. usr/src/pkgdefs/Makefile & usr/src/Targetdirs These need resyncing with gate so it doesn't look as though you are trying to change other stuff. 6. usr/src/pkgdefs/SUNWtipc/Makefile Copyright year is wrong; also ensure you check all the other files 7. usr/src/pkgdefs/SUNWtipc/copyright The METADATA file say ... LICENSE: BSD GPL the GPL licence is not in here! 8. usr/src/lib/tipc/METADATA Line ... 9 BUGTRAQ: /solaris/utility/tipc should it have the first '/' ? 9. usr/src/pkgdefs/SUNWtipc/depend Copyright lines are in the wrong place and the year is wrong. 10. usr/src/pkgdefs/SUNWtipc/prototype_com Line is wrong ... 32 ARCH=i386 and there is some other stuff missing I think (in fact this looks like an after 'pkmake' state of file!!) Add pkg version at end of DESC= line ... DESC="............ (n.x.x)" 11. usr/src/pkgdefs/SUNWtipc/prototype_com Don't install file is /usr with the write permissions set! 12. usr/src/pkgdefs/SUNWtipc/prototype_i386 & New Patch Raw usr/src/pkgdefs/SUNWtipc/prototype_sparc Move the list of files to the end of the file as in prototype_com PLEASE DO A SELF REVIEW OF YOUR STUFF BEFORE SENDING OUT THESE WEBREVS so I and other do not waste their time picking I silly errors !!! END -- ---------------------------------------------------------------------- Paul Cunningham Software Engineer Tadpole Business Unit
