Looks good to me now Jason. Steve C.
Jason Zhao wrote: > Hi, Steve, >> Jason - >> >> I just finished building snort 2.8.4 for sunfreeware.com and in the >> process I compared the files I got to the ones you have listed >> in your package. I get the same libraries as you except, I get one >> more set that is not in your list. Were these not generated in your >> build or left out for some reason? I cannot find any reference to >> them in the code reviews so far. >> >> >> lib/snort_dynamicpreprocessor: >> >> lrwxrwxrwx 1 steve staff 27 Apr 16 18:36 libsf_dce2_preproc.so -> >> libsf_dce2_preproc.so.0.0.0 >> lrwxrwxrwx 1 steve staff 27 Apr 16 18:36 libsf_dce2_preproc.so.0 >> -> libsf_dce2_preproc.so.0.0.0 >> -rwxr-xr-x 1 steve staff 765108 Apr 16 18:36 libsf_dce2_preproc.so.0.0.0 > Thanks you very much for your supplement. > > For these dce2 libraries, they are new plugin libraries in snort 2.8.4. > On snort 2.8.3, it didn't contain the above libraries. > > I have added it into prototype_<arch> and protype_com so that the > SUNWsnortu will deliver the libraries. > > The webrev has been updated. > http://cr.opensolaris.org/~jxzhao/snort/webrev > > Thanks > Jason >> >> >> >> Steve C. >> >> >> >> >> Jim Walker wrote: >>> Jason Zhao wrote: >>>> >>>> Jason >>>>> Hi, Jim, >>>>> >>>>> The webrev is updated, and please see in line. >>>>>> Jason Zhao wrote: >>>>>>> >>>>>>> The new webrev is updated, please review it again and tell me >>>>>>> your comments. >>>>>>> http://cr.opensolaris.org/~jxzhao/snort/webrev >>> >>> Looks good. >>> >>> Cheers, >>> Jim >>> _______________________________________________ >>> sfwnv-discuss mailing list >>> sfwnv-discuss at opensolaris.org >>> http://mail.opensolaris.org/mailman/listinfo/sfwnv-discuss >>> > >
