https://issues.apache.org/bugzilla/show_bug.cgi?id=55442
--- Comment #1 from Violeta Georgieva ---
According to the specification
"1.5.2 Evaluating functions
The expression with the syntax func(args...)(args...)... can mean any of the
following.
...
- A call to the constructor of an imported cl
Author: violetagg
Date: Tue Aug 20 08:38:29 2013
New Revision: 1515744
URL: http://svn.apache.org/r1515744
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=55442
Handle calls to a constructor of an imported class or to a static method,
imported statically.
Unit test based on a patch pr
https://issues.apache.org/bugzilla/show_bug.cgi?id=55442
Violeta Georgieva changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|--
https://issues.apache.org/bugzilla/show_bug.cgi?id=55454
Bug ID: 55454
Summary: NullPointerException caused by invalid contentType,
e.g. trailing semicolon
Product: Tomcat 7
Version: 7.0.42
Hardware: PC
Status:
On 20/08/2013 06:35, Rainer Jung wrote:
> On 19.08.2013 15:22, Mark Thomas wrote:
>> I plan to commit what I have so far once I am happy that the unit tests
>> still work.
>>
>> I do want to look at removing the Java 7 dependency from the WebSocket
>> implementation - at least for Tomcat 7 - but I
Author: markt
Date: Tue Aug 20 10:23:23 2013
New Revision: 1515761
URL: http://svn.apache.org/r1515761
Log:
Reduce a potential timing sensitivity
Modified:
tomcat/trunk/test/org/apache/tomcat/websocket/pojo/TestEncodingDecoding.java
Modified:
tomcat/trunk/test/org/apache/tomcat/websocket/po
Author: markt
Date: Tue Aug 20 11:13:33 2013
New Revision: 1515778
URL: http://svn.apache.org/r1515778
Log:
Further reduce a potential timing sensitivity
Modified:
tomcat/trunk/test/org/apache/tomcat/websocket/pojo/TestEncodingDecoding.java
Modified:
tomcat/trunk/test/org/apache/tomcat/webs
Author: markt
Date: Tue Aug 20 11:14:39 2013
New Revision: 1515779
URL: http://svn.apache.org/r1515779
Log:
Add the rather crucial missing i++ in the wait loop
Modified:
tomcat/trunk/test/org/apache/tomcat/websocket/pojo/TestEncodingDecoding.java
Modified:
tomcat/trunk/test/org/apache/tomca
The Buildbot has detected a new failure on builder tomcat-trunk while building
ASF Buildbot.
Full details are available at:
http://ci.apache.org/builders/tomcat-trunk/builds/4855
Buildbot URL: http://ci.apache.org/
Buildslave for this Build: bb-vm_ubuntu
Build Reason: scheduler
Build Source St
https://issues.apache.org/bugzilla/show_bug.cgi?id=55454
Mark Thomas changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
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/4856
Buildbot URL: http://ci.apache.org/
Buildslave for this Build: bb-vm_ubuntu
Build Reason: scheduler
Build Source
On Aug 20, 2013, at 6:14 AM, ma...@apache.org wrote:
> Author: markt
> Date: Tue Aug 20 11:14:39 2013
> New Revision: 1515779
>
> URL: http://svn.apache.org/r1515779
> Log:
> Add the rather crucial missing i++ in the wait loop
Lol. Don't you hate when you do that?
On 20/08/2013 13:45, Nick Williams wrote:
>
> On Aug 20, 2013, at 6:14 AM, ma...@apache.org wrote:
>
>> Author: markt
>> Date: Tue Aug 20 11:14:39 2013
>> New Revision: 1515779
>>
>> URL: http://svn.apache.org/r1515779
>> Log:
>> Add the rather crucial missing i++ in the wait loop
>
> Lol. Don't
On Aug 20, 2013, at 7:59 AM, Mark Thomas wrote:
> On 20/08/2013 13:45, Nick Williams wrote:
>>
>> On Aug 20, 2013, at 6:14 AM, ma...@apache.org wrote:
>>
>>> Author: markt
>>> Date: Tue Aug 20 11:14:39 2013
>>> New Revision: 1515779
>>>
>>> URL: http://svn.apache.org/r1515779
>>> Log:
>>> Add
I'm working on implementing bugzilla 55317. It's a pretty important change
(weaving) to a pretty import class (WebappClassLoader), so it obviously needs
some unit tests. However, I need some guidance:
1) I've never gotten all the existing tests to pass on my machine. Last time I
ran them it too
Author: markt
Date: Tue Aug 20 13:14:48 2013
New Revision: 1515817
URL: http://svn.apache.org/r1515817
Log:
Deprecate the Tomcat 7 proprietary WebSocket API
Modified:
tomcat/tc7.0.x/trunk/java/org/apache/catalina/connector/Request.java
tomcat/tc7.0.x/trunk/java/org/apache/catalina/websock
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/1350
Buildbot URL: http://ci.apache.org/
Buildslave for this Build: bb-vm_ubuntu
Build Reason: scheduler
Build Sourc
On 20/08/2013 14:13, Nick Williams wrote:
> I'm working on implementing bugzilla 55317. It's a pretty important
> change (weaving) to a pretty import class (WebappClassLoader), so it
> obviously needs some unit tests. However, I need some guidance:
>
> 1) I've never gotten all the existing tests t
https://issues.apache.org/bugzilla/show_bug.cgi?id=55454
Violeta Georgieva changed:
What|Removed |Added
Status|RESOLVED|REOPENED
Resolution|WO
2013/8/20 Nick Williams wrote:
>
> I'm working on implementing bugzilla 55317. It's a pretty important
change (weaving) to a pretty import class (WebappClassLoader), so it
obviously needs some unit tests. However, I need some guidance:
>
> 1) I've never gotten all the existing tests to pass on my m
On 20/08/13 14:32, Violeta Georgieva wrote:
2013/8/20 Nick Williams wrote:
I'm working on implementing bugzilla 55317. It's a pretty important
change (weaving) to a pretty import class (WebappClassLoader), so it
obviously needs some unit tests. However, I need some guidance:
1) I've never go
https://issues.apache.org/bugzilla/show_bug.cgi?id=55454
--- Comment #3 from Violeta Georgieva ---
What about if we do a change like this?
--- org/apache/tomcat/util/http/parser/HttpParser.java(revision 1515761)
+++ org/apache/tomcat/util/http/parser/HttpParser.java(working copy)
@@ -210
On 20/08/2013 14:20, build...@apache.org wrote:
> 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/1350
>
> Buildbot URL: http://ci.apache.org/
>
> Buildslave f
https://issues.apache.org/bugzilla/show_bug.cgi?id=55454
--- Comment #4 from Mark Thomas ---
Works for me.
--
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@tom
Author: markt
Date: Tue Aug 20 14:03:27 2013
New Revision: 1515827
URL: http://svn.apache.org/r1515827
Log:
Fix a couple of Eclipse warnings
Modified:
tomcat/trunk/java/org/apache/jasper/JspCompilationContext.java
tomcat/trunk/java/org/apache/tomcat/util/descriptor/tld/TldRuleSet.java
Mo
On Aug 20, 2013, at 8:25 AM, Mark Thomas wrote:
> On 20/08/2013 14:13, Nick Williams wrote:
>> I'm working on implementing bugzilla 55317. It's a pretty important
>> change (weaving) to a pretty import class (WebappClassLoader), so it
>> obviously needs some unit tests. However, I need some guida
Author: violetagg
Date: Tue Aug 20 15:02:23 2013
New Revision: 1515841
URL: http://svn.apache.org/r1515841
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=55454
Fixed NPE thrown when parsing an invalid content type.
Modified:
tomcat/trunk/java/org/apache/tomcat/util/http/parser/Ht
https://issues.apache.org/bugzilla/show_bug.cgi?id=55453
--- Comment #1 from Christopher Schultz ---
So you are saying that AbstractAjpProcessor.prepareResponse needs to have
similar logic?
Under what conditions is a 304 response being sent along with a body? Is this a
Tomcat problem (i.e. a sit
Mark,
On 8/20/13 9:25 AM, Mark Thomas wrote:
> On 20/08/2013 14:13, Nick Williams wrote:
>> I'm working on implementing bugzilla 55317. It's a pretty important
>> change (weaving) to a pretty import class (WebappClassLoader), so it
>> obviously needs some unit tests. However, I need some guidance:
On Aug 20, 2013, at 10:12 AM, Christopher Schultz wrote:
> Mark,
>
> On 8/20/13 9:25 AM, Mark Thomas wrote:
>> On 20/08/2013 14:13, Nick Williams wrote:
>>> I'm working on implementing bugzilla 55317. It's a pretty important
>>> change (weaving) to a pretty import class (WebappClassLoader), so i
https://issues.apache.org/bugzilla/show_bug.cgi?id=55453
--- Comment #2 from Dieter Paradeiser ---
*) same logic, would be solve the problem ;o)
**) ajp should also remove the body from response when status 304 is set
*) its a webapp, jira from atlassian
*) I know about the rfc, I also requested
https://issues.apache.org/bugzilla/show_bug.cgi?id=55454
Violeta Georgieva changed:
What|Removed |Added
Component|Jasper |Catalina
--
You are receiving
Author: violetagg
Date: Tue Aug 20 16:38:49 2013
New Revision: 1515862
URL: http://svn.apache.org/r1515862
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=55454
Merged revision 1515841 from tomcat/trunk:
Fixed NPE thrown when parsing an invalid content type.
Modified:
tomcat/tc7.0
https://issues.apache.org/bugzilla/show_bug.cgi?id=55454
Violeta Georgieva changed:
What|Removed |Added
Status|REOPENED|RESOLVED
Resolution|--
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/1351
Buildbot URL: http://ci.apache.org/
Buildslave for this Build: bb-vm_ubuntu
Build Reason: scheduler
Build Sourc
2013/8/12 Mark Thomas wrote:
>
>
> I'm wondering about adding enableWebSockets() and enableJSPs() methods
> to Tomcat which would automatically add the relevant SCI to any Context
> created via one of the addContext(...) methods. Thoughts?
>
+1
Author: markt
Date: Tue Aug 20 17:16:05 2013
New Revision: 1515889
URL: http://svn.apache.org/r1515889
Log:
Missed renaming the fileset for the JSR-356 WebSocket implementation in one
location.
Modified:
tomcat/tc7.0.x/trunk/build.xml
Modified: tomcat/tc7.0.x/trunk/build.xml
URL:
http://sv
On 20/08/2013 17:47, build...@apache.org wrote:
> 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/1351
>
> Buildbot URL: http://ci.apache.org/
>
> Buildslave f
Nick,
On 8/20/13 11:15 AM, Nick Williams wrote:
>
> On Aug 20, 2013, at 10:12 AM, Christopher Schultz wrote:
>
>> Mark,
>>
>> On 8/20/13 9:25 AM, Mark Thomas wrote:
>>> On 20/08/2013 14:13, Nick Williams wrote:
I'm working on implementing bugzilla 55317. It's a pretty important
change
Author: markt
Date: Tue Aug 20 17:31:57 2013
New Revision: 1515893
URL: http://svn.apache.org/r1515893
Log:
Remove duplicate test
Modified:
tomcat/tc7.0.x/trunk/test/org/apache/tomcat/util/http/parser/TestMediaType.java
Modified:
tomcat/tc7.0.x/trunk/test/org/apache/tomcat/util/http/parser
Nick,
On 8/20/13 1:25 PM, Christopher Schultz wrote:
> tcnative can be built on Max OS X like this:
>
> $ cd $CATALINA_HOME/bin
> $ tar xzf tomcat-native.tar.gz
> $ cd tomcat-native-*-src/jni/native
> $ ./configure --with-apr=/usr/bin/apr-1-config --with-ssl
> --with-java-home=`/usr/libexec/java_
On Aug 20, 2013, at 12:41 PM, Christopher Schultz wrote:
> Nick,
>
> On 8/20/13 1:25 PM, Christopher Schultz wrote:
>> tcnative can be built on Max OS X like this:
>>
>> $ cd $CATALINA_HOME/bin
>> $ tar xzf tomcat-native.tar.gz
>> $ cd tomcat-native-*-src/jni/native
>> $ ./configure --with-apr=
Nick,
On 8/20/13 1:41 PM, Christopher Schultz wrote:
> Nick,
>
> On 8/20/13 1:25 PM, Christopher Schultz wrote:
>> tcnative can be built on Max OS X like this:
>>
>> $ cd $CATALINA_HOME/bin
>> $ tar xzf tomcat-native.tar.gz
>> $ cd tomcat-native-*-src/jni/native
>> $ ./configure --with-apr=/usr/b
Nick,
On 8/20/13 1:47 PM, Nick Williams wrote:
>
> On Aug 20, 2013, at 12:41 PM, Christopher Schultz wrote:
>
>> Nick,
>>
>> On 8/20/13 1:25 PM, Christopher Schultz wrote:
>>> tcnative can be built on Max OS X like this:
>>>
>>> $ cd $CATALINA_HOME/bin
>>> $ tar xzf tomcat-native.tar.gz
>>> $ cd
On 20/08/2013 18:58, Christopher Schultz wrote:
> In case you want to run the entire test suite but don't want to get
> APR failures, you can also set -Dexecute.test.apr=false and it will
> skip all the APR tests.
The unit tests are smart enough that even with that set to true, the
tests will be
On 20/08/2013 18:47, Nick Williams wrote:
> My remaining original concern was the best approach for weaving byte
> code in Tomcat's unit tests, which I detailed in an earlier message.
Use the Weaver to completely replace the byte code of the weaved class
with the byte code from another class? That
On Tue, Aug 20, 2013 at 1:10 PM, Mark Thomas wrote:
> On 20/08/2013 18:47, Nick Williams wrote:
>> My remaining original concern was the best approach for weaving byte
>> code in Tomcat's unit tests, which I detailed in an earlier message.
>
> Use the Weaver to completely replace the byte code of
Author: markt
Date: Tue Aug 20 19:07:25 2013
New Revision: 1515926
URL: http://svn.apache.org/r1515926
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=55439
Don't try a forced stop (kill -9 $PID) if Tomcat has already been stopped and
the PID file cleared else a bunch of errors will r
Author: markt
Date: Tue Aug 20 19:11:24 2013
New Revision: 1515929
URL: http://svn.apache.org/r1515929
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=55439
Don't try a forced stop (kill -9 $PID) if Tomcat has already been stopped and
the PID file cleared else a bunch of errors will r
https://issues.apache.org/bugzilla/show_bug.cgi?id=55439
Mark Thomas changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On 18/08/2013 00:42, b...@apache.org wrote:
> build-only:
> [javac] Compiling 97 source files to
> /srv/gump/public/workspace/jakarta-tomcat-jasper_tc5/build/shared/classes
> [javac]
> /srv/gump/public/workspace/jakarta-tomcat-jasper_tc5/src/share/org/apache/jasper/compiler/JDTCompiler
On 20/08/2013 00:50, Bill Barker wrote:
> validate:
> [mkdir] Created dir:
> /srv/gump/public/workspace/tomcat-7.0.x/output/res/checkstyle
> [checkstyle] Running Checkstyle 5.7-SNAPSHOT on 2388 files
> [checkstyle]
> /srv/gump/public/workspace/tomcat-7.0.x/modules/jdbc-pool/src/test/java/o
The Buildbot has detected a new failure on builder tomcat-trunk while building
ASF Buildbot.
Full details are available at:
http://ci.apache.org/builders/tomcat-trunk/builds/4859
Buildbot URL: http://ci.apache.org/
Buildslave for this Build: bb-vm_ubuntu
Build Reason: scheduler
Build Source St
2013/8/20 Mark Thomas :
> On 20/08/2013 00:50, Bill Barker wrote:
>
>
>
>> validate:
>> [mkdir] Created dir:
>> /srv/gump/public/workspace/tomcat-7.0.x/output/res/checkstyle
>> [checkstyle] Running Checkstyle 5.7-SNAPSHOT on 2388 files
>> [checkstyle]
>> /srv/gump/public/workspace/tomcat-7.0
On Aug 20, 2013, at 1:10 PM, Mark Thomas wrote:
> On 20/08/2013 18:47, Nick Williams wrote:
>> My remaining original concern was the best approach for weaving byte
>> code in Tomcat's unit tests, which I detailed in an earlier message.
>
> Use the Weaver to completely replace the byte code of th
To whom it may engage...
This is an automated request, but not an unsolicited one. For
more information please visit http://gump.apache.org/nagged.html,
and/or contact the folk at gene...@gump.apache.org.
Project jakarta-tomcat-jasper_tc5 has an issue affecting its community
integratio
To whom it may engage...
This is an automated request, but not an unsolicited one. For
more information please visit http://gump.apache.org/nagged.html,
and/or contact the folk at gene...@gump.apache.org.
Project tomcat-tc7.0.x-validate has an issue affecting its community
integration.
To whom it may engage...
This is an automated request, but not an unsolicited one. For
more information please visit http://gump.apache.org/nagged.html,
and/or contact the folk at gene...@gump.apache.org.
Project tomcat-trunk-test has an issue affecting its community integration.
This i
To whom it may engage...
This is an automated request, but not an unsolicited one. For
more information please visit http://gump.apache.org/nagged.html,
and/or contact the folk at gene...@gump.apache.org.
Project tomcat-tc7.0.x-test has an issue affecting its community integration.
This
On 2013-08-20, Mark Thomas wrote:
> On 18/08/2013 00:42, b...@apache.org wrote:
>
>> build-only:
>> [javac] Compiling 97 source files to
>> /srv/gump/public/workspace/jakarta-tomcat-jasper_tc5/build/shared/classes
>> [javac]
>> /srv/gump/public/workspace/jakarta-tomcat-jasper_tc5/src/
On 2013-08-21, Konstantin Kolinko wrote:
> 2013/8/20 Mark Thomas :
>> On 20/08/2013 00:50, Bill Barker wrote:
>>
>>> validate:
>>> [mkdir] Created dir:
>>> /srv/gump/public/workspace/tomcat-7.0.x/output/res/checkstyle
>>> [checkstyle] Running Checkstyle 5.7-SNAPSHOT on 2388 files
>>> [chec
Stefan Bodewig wrote:
>On 2013-08-20, Mark Thomas wrote:
>
>> On 18/08/2013 00:42, b...@apache.org wrote:
>
>>
>
>>> build-only:
>>> [javac] Compiling 97 source files to
>/srv/gump/public/workspace/jakarta-tomcat-jasper_tc5/build/shared/classes
>>> [javac]
>/srv/gump/public/workspace/jaka
On 2013-08-21, Mark Thomas wrote:
> Dropping the Tomcat 5 builds is fine. I assumed it was still there
> because something else you wanted to keep depended on it.
No, the Tomcat 4/5 builds are really only there because I didn't want to
pull the trigger too soon. I am subscribed to dev@tomcat and
On Aug 20, 2013, at 7:42 AM, Nick Williams
wrote:
>
> Well, yea. I had figured that part out. :-) I just didn't articulate my
> question well enough.
>
> The first step is obviously loading the class through the WACL through the
> unit test (so this has to be a class not already loaded). I as
64 matches
Mail list logo