----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12213/#review22681 -----------------------------------------------------------
Thanks, Venkat. Just a couple of questions. http://svn.apache.org/repos/asf/incubator/streams/trunk/pom.xml <https://reviews.apache.org/r/12213/#comment46369> Do we need Javax.mail? it is not a dependency of the project that I see. http://svn.apache.org/repos/asf/incubator/streams/trunk/streams-web/pom.xml <https://reviews.apache.org/r/12213/#comment46371> I don't think the container needs these dependencies. http://svn.apache.org/repos/asf/incubator/streams/trunk/streams-web/pom.xml <https://reviews.apache.org/r/12213/#comment46373> What about tomcat 7? http://svn.apache.org/repos/asf/incubator/streams/trunk/streams-web/pom.xml <https://reviews.apache.org/r/12213/#comment46374> See prior comments on mail & activation - Matt Franklin On July 1, 2013, 8:25 p.m., Venkat Mahadevan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12213/ > ----------------------------------------------------------- > > (Updated July 1, 2013, 8:25 p.m.) > > > Review request for streams. > > > Repository: streams > > > Description > ------- > > Added cargo to run tomcat locally on port 8000 > > > Diffs > ----- > > http://svn.apache.org/repos/asf/incubator/streams/trunk/pom.xml 1497338 > http://svn.apache.org/repos/asf/incubator/streams/trunk/streams-web/pom.xml > 1497338 > > Diff: https://reviews.apache.org/r/12213/diff/ > > > Testing > ------- > > > Thanks, > > Venkat Mahadevan > >
