Hi Stefan, This mainly looks good to me, see below for a few minor comments ...
Paul Stefan Teleman wrote: > http://cr.opensolaris.org/~steleman/6929417/ 1. usr/src/cmd/nmap/install-nmap Line 114, does this need the write permission? 2. Tarball The adding of the new tarball and removal of the old is not showing up in the webrev. 3. usr/src/cmd/nmap/METADATA Observation: 8 SUPPORT: C is that correct (I forget what they should be)? -- Paul Cunningham Software Engineer
