You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@continuum.apache.org by ct...@apache.org on 2008/10/05 08:57:40 UTC

svn commit: r701753 - /continuum/branches/continuum-1.2.x/continuum-release/src/main/resources/META-INF/plexus/components.xml

Author: ctan
Date: Sat Oct  4 23:57:40 2008
New Revision: 701753

URL: http://svn.apache.org/viewvc?rev=701753&view=rev
Log:
[CONTINUUM-1572] fixed NPE

Modified:
    continuum/branches/continuum-1.2.x/continuum-release/src/main/resources/META-INF/plexus/components.xml

Modified: continuum/branches/continuum-1.2.x/continuum-release/src/main/resources/META-INF/plexus/components.xml
URL: http://svn.apache.org/viewvc/continuum/branches/continuum-1.2.x/continuum-release/src/main/resources/META-INF/plexus/components.xml?rev=701753&r1=701752&r2=701753&view=diff
==============================================================================
--- continuum/branches/continuum-1.2.x/continuum-release/src/main/resources/META-INF/plexus/components.xml (original)
+++ continuum/branches/continuum-1.2.x/continuum-release/src/main/resources/META-INF/plexus/components.xml Sat Oct  4 23:57:40 2008
@@ -46,6 +46,9 @@
         <requirement>
           <role>org.apache.maven.continuum.utils.WorkingDirectoryService</role>
         </requirement>
+        <requirement>
+          <role>org.apache.maven.continuum.installation.InstallationService</role>
+        </requirement>
       </requirements>
     </component>
 



Re: svn commit: r701753 - /continuum/branches/continuum-1.2.x/continuum-release/src/main/resources/META-INF/plexus/components.xml

Posted by Emmanuel Venisse <em...@gmail.com>.
OK

Emmanuel

On Mon, Oct 6, 2008 at 10:33 AM, Marica Tan <ct...@exist.com> wrote:

> Hi,
> The fix for CONTINUUM-1572 requires InstallationService. There's a test
> failure because of this and I'm thinking of moving the implementation of
> InstallationService and ProfileService from continuum-core to
> continuum-commons, which will be more appropriate since they're now being
> used during release.
>
> WDYT?
>
> Thanks,
> --
> Marica
>
> On Sun, Oct 5, 2008 at 2:57 PM, <ct...@apache.org> wrote:
>
> > Author: ctan
> > Date: Sat Oct  4 23:57:40 2008
> > New Revision: 701753
> >
> > URL: http://svn.apache.org/viewvc?rev=701753&view=rev
> > Log:
> > [CONTINUUM-1572] fixed NPE
> >
> > Modified:
> >
> >
>  continuum/branches/continuum-1.2.x/continuum-release/src/main/resources/META-INF/plexus/components.xml
> >
> > Modified:
> >
> continuum/branches/continuum-1.2.x/continuum-release/src/main/resources/META-INF/plexus/components.xml
> > URL:
> >
> http://svn.apache.org/viewvc/continuum/branches/continuum-1.2.x/continuum-release/src/main/resources/META-INF/plexus/components.xml?rev=701753&r1=701752&r2=701753&view=diff
> >
> >
> ==============================================================================
> > ---
> >
> continuum/branches/continuum-1.2.x/continuum-release/src/main/resources/META-INF/plexus/components.xml
> > (original)
> > +++
> >
> continuum/branches/continuum-1.2.x/continuum-release/src/main/resources/META-INF/plexus/components.xml
> > Sat Oct  4 23:57:40 2008
> > @@ -46,6 +46,9 @@
> >         <requirement>
> >
> > <role>org.apache.maven.continuum.utils.WorkingDirectoryService</role>
> >         </requirement>
> > +        <requirement>
> > +
> >  <role>org.apache.maven.continuum.installation.InstallationService</role>
> > +        </requirement>
> >       </requirements>
> >     </component>
> >
> >
> >
> >
>

Re: svn commit: r701753 - /continuum/branches/continuum-1.2.x/continuum-release/src/main/resources/META-INF/plexus/components.xml

Posted by Marica Tan <ct...@exist.com>.
Hi,
The fix for CONTINUUM-1572 requires InstallationService. There's a test
failure because of this and I'm thinking of moving the implementation of
InstallationService and ProfileService from continuum-core to
continuum-commons, which will be more appropriate since they're now being
used during release.

WDYT?

Thanks,
--
Marica

On Sun, Oct 5, 2008 at 2:57 PM, <ct...@apache.org> wrote:

> Author: ctan
> Date: Sat Oct  4 23:57:40 2008
> New Revision: 701753
>
> URL: http://svn.apache.org/viewvc?rev=701753&view=rev
> Log:
> [CONTINUUM-1572] fixed NPE
>
> Modified:
>
>  continuum/branches/continuum-1.2.x/continuum-release/src/main/resources/META-INF/plexus/components.xml
>
> Modified:
> continuum/branches/continuum-1.2.x/continuum-release/src/main/resources/META-INF/plexus/components.xml
> URL:
> http://svn.apache.org/viewvc/continuum/branches/continuum-1.2.x/continuum-release/src/main/resources/META-INF/plexus/components.xml?rev=701753&r1=701752&r2=701753&view=diff
>
> ==============================================================================
> ---
> continuum/branches/continuum-1.2.x/continuum-release/src/main/resources/META-INF/plexus/components.xml
> (original)
> +++
> continuum/branches/continuum-1.2.x/continuum-release/src/main/resources/META-INF/plexus/components.xml
> Sat Oct  4 23:57:40 2008
> @@ -46,6 +46,9 @@
>         <requirement>
>
> <role>org.apache.maven.continuum.utils.WorkingDirectoryService</role>
>         </requirement>
> +        <requirement>
> +
>  <role>org.apache.maven.continuum.installation.InstallationService</role>
> +        </requirement>
>       </requirements>
>     </component>
>
>
>
>