[illumos-Advocates] RTI: richard.elling at richardelling.com

[Gordon Ross]

webrev is here:
  http://www.yalms.org/cr/snoop/final/

mail_msg.txt attached.
I did not update the copyright in snoop_rport.c
where I only removed code (which moved over to
snoop_display.c).

$ hg outgoing -v
comparing with /tank/ws/illumos-clone
searching for changes

changeset:   13340:da0cf3ba8101
tag:         tip
user:        Gordon Ross <gwr at nexenta.com>
date:        Sat Apr 16 12:38:55 2011 -0400

description:
	895 snoop lies about SMB errors
	Reviewed by: Richard Lowe <richlowe at richlowe.net>
	Reviewed by: richard.elling at richardelling.com
	Approved by: ?

modified:
   usr/src/cmd/cmd-inet/usr.sbin/snoop/snoop.h
   usr/src/cmd/cmd-inet/usr.sbin/snoop/snoop_display.c
   usr/src/cmd/cmd-inet/usr.sbin/snoop/snoop_rport.c
   usr/src/cmd/cmd-inet/usr.sbin/snoop/snoop_smb.c

$hg pbchk
Copyright check:
usr/src/cmd/cmd-inet/usr.sbin/snoop/snoop_rport.c: no copyright claim
for current year found

C style check:

Header format check:

Java style check:

Mapfile comment check:

File permission check:

Keywords check:

Comments check:

Checking for new tags:

Checking for multiple heads (or branches):

Checking for branch changes:

Checking for uncommitted changes:

Checking for merges:


-------------- next part --------------

==== Nightly distributed build started:   Sat Apr 16 00:10:32 EDT 2011 ====
==== Nightly distributed build completed: Sat Apr 16 08:17:24 EDT 2011 ====

==== Total build time ====

real    8:06:52

==== Build environment ====

/usr/bin/uname
SunOS sun40 5.11 oi_148b i86pc i386 i86pc

/opt/SUNWspro/bin/dmake
dmake: Sun Distributed Make 7.8 SunOS_i386 Patch 126504-01 2007/07/19
number of concurrent jobs = 4

32-bit compiler
/opt/onbld/bin/i386/cw -_cc
cw version 1.29
primary: /opt/onspro/SS12/bin/cc
cc: Sun C 5.9 SunOS_i386 Patch 124868-07 2008/10/07
shadow: /usr/sfw/bin/gcc
gcc (GCC) 3.4.3 (csl-sol210-3_4-20050802)

64-bit compiler
/opt/onbld/bin/i386/cw -_cc
cw version 1.29
primary: /opt/onspro/SS12/bin/cc
cc: Sun C 5.9 SunOS_i386 Patch 124868-07 2008/10/07
shadow: /usr/sfw/bin/gcc
gcc (GCC) 3.4.3 (csl-sol210-3_4-20050802)

/usr/java/bin/javac
java full version "1.6.0_21-b06"

/usr/ccs/bin/as
as: Sun Compiler Common 12 SunOS_i386 snv_121 08/03/2009

/usr/ccs/bin/ld
ld: Software Generation Utilities - Solaris Link Editors: 5.11-1.1726

Build project:  group.staff
Build taskid:   333

==== Nightly argument issues ====


==== Build version ====

snoop

==== Tools build errors ====


==== Build errors (non-DEBUG) ====


==== Build warnings (non-DEBUG) ====


==== Elapsed build time (non-DEBUG) ====

real  2:44:17.6
user  5:53:37.8
sys   1:06:02.0

==== package build errors (non-DEBUG) ====


==== Build errors (DEBUG) ====


==== Build warnings (DEBUG) ====


==== Elapsed build time (DEBUG) ====

real  2:36:16.1
user  5:03:00.9
sys   1:00:00.5

==== package build errors (DEBUG) ====


==== Validating manifests against proto area ====


==== Check ELF runtime attributes ====


==== Diff ELF runtime attributes (since last build) ====


==== 'dmake lint' of src ERRORS ====


==== Elapsed time of 'dmake lint' of src ====

real  2:02:25.4
user  1:48:13.5
sys     20:20.4

==== lint warnings src ====

lint: Warning: library -lkrb5 not found

==== lint noise differences src ====


==== cstyle/hdrchk errors ====


==== Find core files ====


==== Check lists of files ====


==== Impact on file permissions ====