Brian Overstreet wrote: > On Friday October 9 2009 06:25:40 Petr Sumbera wrote: >> Hi, >> >> can somebody please review following changes to Solaris Nevada Tomcat? >> >> 6864292 [nvb119] some JSP and SERVLET functional tests are failed >> (strict_servlet compliance) >> 6878859 Tomcat should create pid file for monitoring by other applications >> 6890019 Tomcat packaging should better support custom ROOT web >> application deployment >> 6889510 tomcat 6 is built with wrong version info >> >> http://cr.opensolaris.org/~xsumbe00/6890019-tomcat/ > The changes look good to me. > +1
Thanks for looking at this! I had to update changes. Can you please look at updated webrev? http://cr.opensolaris.org/~xsumbe00/6890019-tomcat-2/ Changes: Pid file location moved from /var/run/tomcat6/pid to /var/tomcat6/logs/pid. The reason is that /var/run directory is cleaned after every reboot and Tomcat which is running with 'webservd' user credentials is not able to write there. The second change is CR 6893192 (new package). Thanks, Petr
