Hi Paul, New webrev is at http://cr.opensolaris.org/~spoorthy/conmanupd1/ and comments inline..
Paul Cunningham wrote: > Spoorthy, > > See below for my comments ... > > Paul > > Spoorthy H.S wrote: >> >> New webrev is at >> http://cr.opensolaris.org/~spoorthy/conmanupd/ > > === Start of Comments === > > 1. usr/src/Targetdirs > Put your new bits in alphabetically ... > /etc/apache2/2.2 \ > /etc/apache2/2.2/conf.d \ > + /etc/logrotate.d \ > + /etc/default \ > /etc/lighttpd \ > /etc/lighttpd/1.4 \ > /etc/lighttpd/1.4/conf.d \ > /etc/openwsman \ > Done > 2. usr/src/cmd/conman/METADATA > Remove the last blank line > > Complete this line ... > 11 BUGTRAQ: > Raised the request. Will update the field. > 3. usr/src/cmd/conman/Makefile.sfw > On line 33 you change ... > --sysconfdir=/etc > to .. > --sysconfdir=$(CFGETC) > > Cosmetic: move the line-space up a line, eg. .. > 19 # CDDL HEADER END > 20 # > .. > 21 # > 23 # Copyright 2009 Sun Microsystems, Inc. All rights reserved. > Done > 4. usr/src/cmd/conman/install-sfw > Cosmetic: add extra line-space after line 45 > > On lines 74 to 83 you could use ${LIB} instead of > ${PREFIX}/lib > Declared variable is used. > 5. Various SUNWconmanr/* and SUNWconmanu/* files > Change the top of these files so they all conform to > that in ... > "http://src.opensolaris.org/source/xref/onnv/onnv-gate/usr/src/prototypes/"; > > > 6. usr/src/pkgdefs/SUNWconmanr/pkginfo.tmpl > As this is a root package you may want to remove the > pkg version number on the DESC= line (that way this > file should not need changing when the src pkg > is up versioned) > Removed the version number. > 7. usr/src/pkgdefs/SUNWconmanr/prototype_com > Do any of the installed /etc files need preserving > over a SUNW pkg upgrade? If so you need to handle that. > I dont think it happens. > 8. pkgdefs/Makefile > & cmd/Makefile > The changes to these are not in the webrev > Done > 9. usr/src/pkgdefs/SUNWconmanu/copyright > Add src pkg owner copyright lines (after sun disclaimer) > extracted from the src files in the uncompressed tarball, > see example at ... > "http://src.opensolaris.org/source/xref/sfw/usr/src/pkgdefs/SUNWmeld/copyright"; > > > The copyright file is "COPYING" in my package. I copied the contents of it to "copyright" file after SUN disclaimer. > 10. usr/src/pkgdefs/SUNWconmanu/depend > Are there any other dependencies? > No other dependencies except Root pakage - SUNWconmanr. Do let me know how can I run the "package dependency checker". Thanks Spoorthy > === End of Comments =====
