Re: Notes from review of BZ 56029 and ELParser patches

2014-01-21 Thread Mark Thomas
On 21/01/2014 04:14, Konstantin Kolinko wrote: Hi! Reviewing the patches for https://issues.apache.org/bugzilla/show_bug.cgi?id=56029 (r1559555, r1559708, r1559820) Thanks for the review. I'm really beginning to dislike this code ;) I'll take a look at the individual issues asap. As an

[Bug 56044] New: Extract EL implementation from embedded Jasper into a separate JAR

2014-01-21 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=56044 Bug ID: 56044 Summary: Extract EL implementation from embedded Jasper into a separate JAR Product: Tomcat 7 Version: unspecified Hardware: PC OS:

[Bug 56045] New: Cannot start embedded container without Jasper

2014-01-21 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=56045 Bug ID: 56045 Summary: Cannot start embedded container without Jasper Product: Tomcat 7 Version: unspecified Hardware: PC OS: Linux Status: NEW

Git

2014-01-21 Thread Mark Thomas
I've been using Git more and more for Tomcat development and was wondering what folks thought about moving Tomcat to git. This isn't a formal proposal or a vote, I'm just trying to gather some views. On the plus side: - it is much easier to have multiple issues in progress at the same time and

Re: Git

2014-01-21 Thread Rémy Maucherat
2014/1/21 Mark Thomas ma...@apache.org On the down side: - there is much more potential to mess things up Yes, I can help with that if you want :) - cleaning up is potentially more complex - the disruption of the move - particularly if we want to move to a single git repo - could be

Re: Git

2014-01-21 Thread Mark Thomas
On 21/01/2014 10:18, Rémy Maucherat wrote: 2014/1/21 Mark Thomas ma...@apache.org Thoughts? Risky, but it can be attempted. I'm not very familiar with the infrastructure used though. git.apache.org is there already, it mirrors svn, and it is then mirrored again in github ? The repo to do

Re: Git

2014-01-21 Thread Violeta Georgieva
2014/1/21 Mark Thomas ma...@apache.org I've been using Git more and more for Tomcat development and was wondering what folks thought about moving Tomcat to git. This isn't a formal proposal or a vote, I'm just trying to gather some views. On the plus side: - it is much easier to have

Re: Git

2014-01-21 Thread Henri Gomez
Hi all +1 for Git About Maven, it will be a very good move also. There is tons of projects inside ASF using Maven as their build tool. There is also so many projects using Tomcat artefacts, hosted on Central, that it will make their life easier if Maven was used has main build tool for Tomcat.

Re: Git

2014-01-21 Thread Martin Grigorov
Hi Rainer, On Tue, Jan 21, 2014 at 1:01 PM, Rainer Jung rainer.j...@kippdata.dewrote: On 21.01.2014 10:58, Mark Thomas wrote: I've been using Git more and more for Tomcat development and was wondering what folks thought about moving Tomcat to git. This isn't a formal proposal or a vote,

Re: Git

2014-01-21 Thread Olivier Lamy
On 21 January 2014 21:18, Rémy Maucherat r...@apache.org wrote: 2014/1/21 Mark Thomas ma...@apache.org On the down side: - there is much more potential to mess things up Yes, I can help with that if you want :) - cleaning up is potentially more complex - the disruption of the move -

Re: Git

2014-01-21 Thread Mark Thomas
On 21/01/2014 12:27, Olivier Lamy wrote: I wonder if this could improve a bit the modularity of the various jars we deploy. Especially for the embeded part. (maybe could prevent issues as: jsp/jasper is mandatory to run embeded mode even if you don't use it) The modularity is not the issue

Re: Git

2014-01-21 Thread Romain Manni-Bucau
Hi just a side note about maven from a tomee dev (and big tomcat user ;): would be great to make it easier to build for anyone but it would mandate to be useful to restructure tomcat sources to be modular as well which would be really great IMO! Romain Manni-Bucau Twitter: @rmannibucau Blog:

Re: Git

2014-01-21 Thread Rémy Maucherat
2014/1/21 Mark Thomas ma...@apache.org Since we're considering big changes, what about Maven too ? I'll spare you the pros and cons. I still see more cons than pros for moving to Maven. It (mostly) works now and allows behaving better with bigger projects. IMO it is unavoidable now.

Re: Git

2014-01-21 Thread Konstantin Kolinko
2014/1/21 Mark Thomas ma...@apache.org: I've been using Git more and more for Tomcat development and was wondering what folks thought about moving Tomcat to git. This isn't a formal proposal or a vote, I'm just trying to gather some views. On the plus side: - it is much easier to have

[Bug 56046] New: org.apache.tomcat.jdbc.pool.XADataSource InitSQL property

2014-01-21 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=56046 Bug ID: 56046 Summary: org.apache.tomcat.jdbc.pool.XADataSource InitSQL property Product: Tomcat 7 Version: 7.0.35 Hardware: All OS: All

[Bug 56046] org.apache.tomcat.jdbc.pool.XADataSource InitSQL property

2014-01-21 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=56046 Mark Thomas ma...@apache.org changed: What|Removed |Added Component|Connectors |jdbc-pool

Re: Git

2014-01-21 Thread Olivier Lamy
On 21 January 2014 23:42, Konstantin Kolinko knst.koli...@gmail.com wrote: 2014/1/21 Mark Thomas ma...@apache.org: I've been using Git more and more for Tomcat development and was wondering what folks thought about moving Tomcat to git. This isn't a formal proposal or a vote, I'm just trying

Re: Git

2014-01-21 Thread Henri Gomez
- Oliver's project to convert Tomcat build to Maven was not a success I started but this change need a bit support so perso I don't want to spend time if lack of interest. it make sense ;( - To unsubscribe, e-mail:

svn commit: r1560017 - in /tomcat/trunk: java/org/apache/jasper/compiler/ELNode.java java/org/apache/jasper/compiler/ELParser.java test/org/apache/el/TestELInJsp.java test/org/apache/jasper/compiler/T

2014-01-21 Thread markt
Author: markt Date: Tue Jan 21 14:08:03 2014 New Revision: 1560017 URL: http://svn.apache.org/r1560017 Log: Further work for https://issues.apache.org/bugzilla/show_bug.cgi?id=56029 Review from kkolinko - Fix bug in parsing functions and whitespace - More efficient resetting of StringBuidler -

svn commit: r1560019 - /tomcat/trunk/test/webapp/bug5nnnn/bug56029.jspx

2014-01-21 Thread markt
Author: markt Date: Tue Jan 21 14:09:04 2014 New Revision: 1560019 URL: http://svn.apache.org/r1560019 Log: Fix line endings Modified: tomcat/trunk/test/webapp/bug5/bug56029.jspx (props changed) Propchange: tomcat/trunk/test/webapp/bug5/bug56029.jspx

svn commit: r1560023 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/jasper/compiler/ test/org/apache/el/ test/org/apache/jasper/compiler/ test/webapp-3.0/bug5nnnn/

2014-01-21 Thread markt
Author: markt Date: Tue Jan 21 14:14:38 2014 New Revision: 1560023 URL: http://svn.apache.org/r1560023 Log: Further work for https://issues.apache.org/bugzilla/show_bug.cgi?id=56029 Review from kkolinko - Fix bug in parsing functions and whitespace - More efficient resetting of StringBuidler -

svn commit: r1560025 - /tomcat/tc6.0.x/trunk/STATUS.txt

2014-01-21 Thread markt
Author: markt Date: Tue Jan 21 14:16:15 2014 New Revision: 1560025 URL: http://svn.apache.org/r1560025 Log: Another proposal update Modified: tomcat/tc6.0.x/trunk/STATUS.txt Modified: tomcat/tc6.0.x/trunk/STATUS.txt URL:

[Cancelled][VOTE] Release Apache Tomcat 6.0.38

2014-01-21 Thread Mark Thomas
This vote has been cancelled due to the regression described in https://issues.apache.org/bugzilla/show_bug.cgi?id=56029#c3 I will tag 6.0.39 just as soon as the fix for BZ 56029 has the necessary votes in the status file. Mark

Re: Git

2014-01-21 Thread Yoav Shapira
Others have mostly captured the pros and cons, so mine is brief... On Tue, Jan 21, 2014 at 4:58 AM, Mark Thomas ma...@apache.org wrote: I've been using Git more and more for Tomcat development and was wondering what folks thought about moving Tomcat to git. This isn't a formal proposal or a

Re: Git

2014-01-21 Thread Henri Gomez
Also +1 to separating the git vs svn discussion from the build system discussion (Maven, Ant, whatever.) (Side note: been using gradle recently, fun.) +1 for split - To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org

[Bug 56039] JmxRemoteLifecycleListener does not work with SSL

2014-01-21 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=56039 esengst...@gmail.com changed: What|Removed |Added CC||esengst...@gmail.com --

Re: Git

2014-01-21 Thread Rémy Maucherat
2014/1/21 Yoav Shapira yo...@apache.org Also +1 to separating the git vs svn discussion from the build system discussion (Maven, Ant, whatever.) (Side note: been using gradle recently, fun.) +1 Never used that Gradle yet though. Rémy

Multi jvmRoute per Tomcat instance

2014-01-21 Thread Rainer Jung
I conducted a little experiment to facilitate testing of load-balancing, stickyness etc. I wanted to use a single Tomcat instance with multiple connectors (ports) each of which using a different jvmRoute when generating sessions for requests coming in via the respective connector. That allows to

svn commit: r1560136 - /tomcat/tc6.0.x/trunk/STATUS.txt

2014-01-21 Thread kkolinko
Author: kkolinko Date: Tue Jan 21 18:43:32 2014 New Revision: 1560136 URL: http://svn.apache.org/r1560136 Log: Add Jeremy's patch. These changes were overwritten by a subsequent commit, but they may be needed to make a clean svn merge. Modified: tomcat/tc6.0.x/trunk/STATUS.txt Modified:

[Bug 56032] WebSocket Endpoint onClose method not called when channel closed by thread interruption

2014-01-21 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=56032 --- Comment #3 from Kenneth Gendron kenneth.gend...@gmail.com --- Thank you very much. -- You are receiving this mail because: You are the assignee for the bug. - To

svn commit: r1560158 - in /tomcat/trunk: java/org/apache/coyote/http11/upgrade/ java/org/apache/tomcat/websocket/ test/org/apache/tomcat/websocket/

2014-01-21 Thread markt
Author: markt Date: Tue Jan 21 19:51:26 2014 New Revision: 1560158 URL: http://svn.apache.org/r1560158 Log: Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=56032 After an IO error sending a WebSocket message, close the connection. Modified:

svn commit: r1560159 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/coyote/http11/upgrade/ java/org/apache/tomcat/websocket/ test/org/apache/tomcat/websocket/ webapps/docs/

2014-01-21 Thread markt
Author: markt Date: Tue Jan 21 19:54:32 2014 New Revision: 1560159 URL: http://svn.apache.org/r1560159 Log: Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=56032 After an IO error sending a WebSocket message, close the connection. Modified: tomcat/tc7.0.x/trunk/ (props changed)

[Bug 56032] WebSocket Endpoint onClose method not called when channel closed by thread interruption

2014-01-21 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=56032 Mark Thomas ma...@apache.org changed: What|Removed |Added Status|NEW |RESOLVED

svn commit: r1560163 - /tomcat/tc6.0.x/trunk/STATUS.txt

2014-01-21 Thread rjung
Author: rjung Date: Tue Jan 21 20:11:49 2014 New Revision: 1560163 URL: http://svn.apache.org/r1560163 Log: Vote, promote. Modified: tomcat/tc6.0.x/trunk/STATUS.txt Modified: tomcat/tc6.0.x/trunk/STATUS.txt URL:

svn propchange: r1560163 - svn:log

2014-01-21 Thread rjung
Author: rjung Revision: 1560163 Modified property: svn:log Modified: svn:log at Tue Jan 21 20:13:02 2014 -- --- svn:log (original) +++ svn:log Tue Jan 21 20:13:02 2014 @@ -1 +1 @@ -Vote, promote. +Vote, comment.

svn commit: r1560165 - /tomcat/trunk/java/org/apache/catalina/core/StandardContext.java

2014-01-21 Thread markt
Author: markt Date: Tue Jan 21 20:15:06 2014 New Revision: 1560165 URL: http://svn.apache.org/r1560165 Log: Simplify bind and unbind on start Modified: tomcat/trunk/java/org/apache/catalina/core/StandardContext.java Modified: tomcat/trunk/java/org/apache/catalina/core/StandardContext.java

svn commit: r1560164 - /tomcat/trunk/java/org/apache/catalina/core/StandardContext.java

2014-01-21 Thread markt
Author: markt Date: Tue Jan 21 20:14:33 2014 New Revision: 1560164 URL: http://svn.apache.org/r1560164 Log: Simplify bind and unbind on stop Modified: tomcat/trunk/java/org/apache/catalina/core/StandardContext.java Modified: tomcat/trunk/java/org/apache/catalina/core/StandardContext.java

svn commit: r1560177 - /tomcat/trunk/java/org/apache/jasper/compiler/ELParser.java

2014-01-21 Thread markt
Author: markt Date: Tue Jan 21 20:56:21 2014 New Revision: 1560177 URL: http://svn.apache.org/r1560177 Log: Remove unnecessary code Modified: tomcat/trunk/java/org/apache/jasper/compiler/ELParser.java Modified: tomcat/trunk/java/org/apache/jasper/compiler/ELParser.java URL:

svn commit: r1560180 - /tomcat/tc6.0.x/trunk/STATUS.txt

2014-01-21 Thread markt
Author: markt Date: Tue Jan 21 20:57:30 2014 New Revision: 1560180 URL: http://svn.apache.org/r1560180 Log: Address review comment and provide a combined patch. Modified: tomcat/tc6.0.x/trunk/STATUS.txt Modified: tomcat/tc6.0.x/trunk/STATUS.txt URL:

buildbot failure in ASF Buildbot on tomcat-7-trunk

2014-01-21 Thread buildbot
The Buildbot has detected a new failure on builder tomcat-7-trunk while building ASF Buildbot. Full details are available at: http://ci.apache.org/builders/tomcat-7-trunk/builds/1701 Buildbot URL: http://ci.apache.org/ Buildslave for this Build: bb-vm_ubuntu Build Reason: scheduler Build

svn commit: r1560212 - /tomcat/trunk/test/org/apache/tomcat/websocket/TestWebSocketFrameClient.java

2014-01-21 Thread markt
Author: markt Date: Tue Jan 21 22:32:24 2014 New Revision: 1560212 URL: http://svn.apache.org/r1560212 Log: Fix order of magnitude error in the wait time Modified: tomcat/trunk/test/org/apache/tomcat/websocket/TestWebSocketFrameClient.java Modified:

svn commit: r1560213 - /tomcat/trunk/test/org/apache/tomcat/websocket/TestWebSocketFrameClient.java

2014-01-21 Thread markt
Author: markt Date: Tue Jan 21 22:44:17 2014 New Revision: 1560213 URL: http://svn.apache.org/r1560213 Log: Writes don't timeout currently with BIO Modified: tomcat/trunk/test/org/apache/tomcat/websocket/TestWebSocketFrameClient.java Modified:

svn commit: r1560214 - in /tomcat/tc7.0.x/trunk: ./ test/org/apache/tomcat/websocket/TestWebSocketFrameClient.java

2014-01-21 Thread markt
Author: markt Date: Tue Jan 21 22:45:35 2014 New Revision: 1560214 URL: http://svn.apache.org/r1560214 Log: Back port unit test fixes / improvements Modified: tomcat/tc7.0.x/trunk/ (props changed) tomcat/tc7.0.x/trunk/test/org/apache/tomcat/websocket/TestWebSocketFrameClient.java

buildbot success in ASF Buildbot on tomcat-trunk

2014-01-21 Thread buildbot
The Buildbot has detected a restored build on builder tomcat-trunk while building ASF Buildbot. Full details are available at: http://ci.apache.org/builders/tomcat-trunk/builds/5425 Buildbot URL: http://ci.apache.org/ Buildslave for this Build: bb-vm_ubuntu Build Reason: scheduler Build

svn commit: r1560237 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/jasper/compiler/ELFunctionMapper.java java/org/apache/jasper/compiler/ELNode.java java/org/apache/jasper/compiler/ELParser.java

2014-01-21 Thread kkolinko
Author: kkolinko Date: Wed Jan 22 02:46:08 2014 New Revision: 1560237 URL: http://svn.apache.org/r1560237 Log: Merging ELParser changes from trunk, to better align TC7 code with trunk here (proposed TC6 patch for BZ 56029 already ports these changes to Tomcat 6) This commit is a merge of

svn commit: r1560238 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/jasper/compiler/ELParser.java

2014-01-21 Thread kkolinko
Author: kkolinko Date: Wed Jan 22 02:55:20 2014 New Revision: 1560238 URL: http://svn.apache.org/r1560238 Log: Merging ELParser changes from trunk, to better align TC7 code with trunk here (proposed TC6 patch for BZ 56029 already ports these changes to Tomcat 6) Merged r1410742 from

svn commit: r1560240 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/jasper/compiler/ELParser.java

2014-01-21 Thread kkolinko
Author: kkolinko Date: Wed Jan 22 02:57:47 2014 New Revision: 1560240 URL: http://svn.apache.org/r1560240 Log: Merging ELParser changes from trunk, to better align TC7 code with trunk here (proposed TC6 patch for BZ 56029 already ports these changes to Tomcat 6) Merged r1560177 from

svn commit: r1560244 - /tomcat/tc6.0.x/trunk/STATUS.txt

2014-01-21 Thread kkolinko
Author: kkolinko Date: Wed Jan 22 03:28:04 2014 New Revision: 1560244 URL: http://svn.apache.org/r1560244 Log: votes Modified: tomcat/tc6.0.x/trunk/STATUS.txt Modified: tomcat/tc6.0.x/trunk/STATUS.txt URL:

Re: svn commit: r1560214 - in /tomcat/tc7.0.x/trunk: ./ test/org/apache/tomcat/websocket/TestWebSocketFrameClient.java

2014-01-21 Thread Konstantin Kolinko
2014/1/22 ma...@apache.org: Author: markt Date: Tue Jan 21 22:45:35 2014 New Revision: 1560214 URL: http://svn.apache.org/r1560214 Log: Back port unit test fixes / improvements Modified: tomcat/tc7.0.x/trunk/ (props changed)

[Bug 56050] New: NPE in WebappClassLoader.clearReferencesJdbc on shutdown

2014-01-21 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=56050 Bug ID: 56050 Summary: NPE in WebappClassLoader.clearReferencesJdbc on shutdown Product: Tomcat 7 Version: 7.0.50 Hardware: PC OS: Linux

[Bug 56050] NPE in WebappClassLoader.clearReferencesJdbc on shutdown

2014-01-21 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=56050 Mark Thomas ma...@apache.org changed: What|Removed |Added Status|NEW |RESOLVED

[Bug 56050] NPE in WebappClassLoader.clearReferencesJdbc on shutdown

2014-01-21 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=56050 Mark Thomas ma...@apache.org changed: What|Removed |Added Status|RESOLVED|REOPENED