archiva-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Brett Porter <br...@apache.org>
Subject Re: svn commit: r631999 - in /maven/archiva/branches/springy: archiva-base/archiva-common/ archiva-base/archiva-configuration/src/test/java/org/apache/maven/archiva/configuration/ archiva-base/archiva-configuration/src/test/resources/org/apache/maven/archi...
Date Thu, 28 Feb 2008 22:11:12 GMT

On 29/02/2008, at 2:21 AM, nicolas@apache.org wrote:

> +      <groupId>org.codehaus.plexus</groupId>
> +      <artifactId>plexus-spring</artifactId>
> +      <version>1.0-SNAPSHOT</version>

should this be test scoped? I don't think it's needed for anything  
other than the test case, right? (occurs in a few places - I think the  
only place it is needed at runtime is in the webapp).

>         // Release existing
> -        release( archivaConfiguration );
> +//      FIXME spring equivalent ?  release( archivaConfiguration );

I don't know if spring takes care of managing them itself - but we  
need to look into this since we used to have leaks from the webapp  
when it never released the components.

Very cool stuff :)

- Brett


--
Brett Porter
brett@apache.org
http://blogs.exist.com/bporter/


Mime
View raw message