Jason,

Changes look good to me. See below for a couple of things I missed first 
time around though ...

Paul

Jason Zhao wrote:
    ... cut ..
> 
> The new webrev is updated, please review it again and tell me your 
> comments.
> http://cr.opensolaris.org/~jxzhao/snort/webrev

1. usr/src/cmd/Makefile
    It looks as though this may need resyncing with the gate
    as it looks as though you may be try to change stuff you
    aren't really, ie.  ...
     -apr-util/apr-util-1.3: apr/apr-1.3 mysql-5-1  ...
     +apr-util/apr-util-1.3: apr/apr-1.3 mysql-5-0  ...
    &
     -php5: apache2 mysql-5-1 postgres/postgresql-8.3
     +php5: apache2 mysql-5-0 postgres/postgresql-8.3

2. usr/src/cmd/snort/Makefile.sfw
    You are building this with gcc, could it have been built
    with the Sun compilers (the preferred compiler) ??

-- 
----------------------------------------------------------------------
Paul Cunningham
Software Engineer
Tadpole Business Unit

Reply via email to