tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeremy Boynes <jboy...@apache.org>
Subject Re: svn commit: r1507870 - in /tomcat/trunk: java/org/apache/catalina/startup/ContextConfig.java java/org/apache/catalina/startup/WebappServiceLoader.java test/org/apache/catalina/startup/TestWebappServiceLoader.java
Date Wed, 31 Jul 2013 15:39:30 GMT
On Jul 31, 2013, at 8:08 AM, Jeremy Boynes <jboynes@apache.org> wrote:

> On Jul 31, 2013, at 7:31 AM, Mark Thomas <markt@apache.org> wrote:
> 
>> On 29/07/2013 20:48, Mark Thomas wrote:
>>> On 29/07/2013 01:01, jboynes@apache.org wrote:
>>>> Author: jboynes
>>>> Date: Sun Jul 28 23:01:23 2013
>>>> New Revision: 1507870
>>>> 
>>>> URL: http://svn.apache.org/r1507870
>>>> Log:
>>>> Fix for https://issues.apache.org/bugzilla/show_bug.cgi?id=55287
>>>> 
>>>> Decouple service loading for ServletContainerInitializers from ContextConfig
and 
>>>> ensure services defined by the parent ClassLoader are discovered.
>>>> 
>>>> Added:
>>>>   tomcat/trunk/java/org/apache/catalina/startup/WebappServiceLoader.java
  (with props)
>>>>   tomcat/trunk/test/org/apache/catalina/startup/TestWebappServiceLoader.java
  (with props)
>>> 
>>> This new test doesn't assert anything. It is also triggering an unused
>>> code warning in Eclipse.
>> 
>> ping
> 
> I deleted the test as ineffectual: http://svn.apache.org/r1508261
> The existing test suite is testing integration for the loader but I still think it needs
more unit tests. I've been sick last couple days so haven't had a chance to add them.
> 
> I don't use Eclipse so a pointer to the problem it's reporting would be helpful.

Scratch that - not thinking straight.
Deleted other case in r1508932.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org


Mime
View raw message