You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@archiva.apache.org by Brett Porter <br...@apache.org> on 2008/02/28 23:11:12 UTC

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...

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/