You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by jv...@apache.org on 2009/05/12 22:50:53 UTC

svn commit: r774079 - in /maven/components/branches/MNG-2766: maven-compat/src/main/java/org/apache/maven/artifact/resolver/ maven-core/src/main/java/org/apache/maven/ maven-core/src/main/java/org/apache/maven/execution/ maven-core/src/main/java/org/ap...

Author: jvanzyl
Date: Tue May 12 20:50:52 2009
New Revision: 774079

URL: http://svn.apache.org/viewvc?rev=774079&view=rev
Log: (empty)

Modified:
    maven/components/branches/MNG-2766/maven-compat/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
    maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/DefaultMaven.java
    maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/execution/MavenSession.java
    maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/lifecycle/LifecycleExecutor.java
    maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java
    maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java
    maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/project/MavenProject.java
    maven/components/branches/MNG-2766/maven-core/src/test/java/org/apache/maven/AbstractCoreMavenComponentTestCase.java
    maven/components/branches/MNG-2766/maven-core/src/test/java/org/apache/maven/MavenTest.java
    maven/components/branches/MNG-2766/maven-core/src/test/java/org/apache/maven/lifecycle/LifecycleExecutorTest.java
    maven/components/branches/MNG-2766/maven-core/src/test/java/org/apache/maven/project/EmptyLifecycleExecutor.java
    maven/components/branches/MNG-2766/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java
    maven/components/branches/MNG-2766/maven-repository/src/main/java/org/apache/maven/repository/LegacyRepositorySystem.java
    maven/components/branches/MNG-2766/maven-repository/src/main/java/org/apache/maven/repository/RepositorySystem.java

Modified: maven/components/branches/MNG-2766/maven-compat/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java
URL: http://svn.apache.org/viewvc/maven/components/branches/MNG-2766/maven-compat/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java?rev=774079&r1=774078&r2=774079&view=diff
==============================================================================
--- maven/components/branches/MNG-2766/maven-compat/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java (original)
+++ maven/components/branches/MNG-2766/maven-compat/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java Tue May 12 20:50:52 2009
@@ -324,7 +324,7 @@
                                                          List<ConflictResolver> conflictResolvers )
         throws ArtifactResolutionException, ArtifactNotFoundException
     {
-        ArtifactResolutionRequest request = new ArtifactResolutionRequest().setArtifact( originatingArtifact ).setResolveRoot( false ).setArtifactDependencies( artifacts ).setManagedVersionMap( managedVersions )
+        ArtifactResolutionRequest request = new ArtifactResolutionRequest().setArtifact( originatingArtifact ).setManagedVersionMap( managedVersions )
             .setLocalRepository( localRepository ).setRemoteRepostories( remoteRepositories ).setMetadataSource( source ).setFilter( filter ).setListeners( listeners );
 
         return resolveWithExceptions( request );
@@ -465,7 +465,7 @@
                                 
         // After the collection we will have the artifact object in the result but they will not be resolved yet.
         result = artifactCollector.collect( artifacts, rootArtifact, managedVersions, localRepository, remoteRepositories, source, filter, listeners, null );
-                
+                        
         // We have metadata retrieval problems, or there are cycles that have been detected
         // so we give this back to the calling code and let them deal with this information
         // appropriately.

Modified: maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/DefaultMaven.java
URL: http://svn.apache.org/viewvc/maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/DefaultMaven.java?rev=774079&r1=774078&r2=774079&view=diff
==============================================================================
--- maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/DefaultMaven.java (original)
+++ maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/DefaultMaven.java Tue May 12 20:50:52 2009
@@ -126,7 +126,7 @@
         {                        
             ProjectSorter projectSorter = new ProjectSorter( projects.values() );
                         
-            session = new MavenSession( container, request, projectSorter.getSortedProjects() );            
+            session = new MavenSession( container, request, result, projectSorter.getSortedProjects() );            
         }
         catch ( CycleDetectedException e )
         {            
@@ -154,13 +154,11 @@
             return result;
         }        
 
-        try
-        {
-            lifecycleExecutor.execute( session );
-        }        
-        catch ( Exception e )
-        {            
-            return processResult( result, e );
+        lifecycleExecutor.execute( session );
+        
+        if ( session.getResult().hasExceptions() )
+        {        
+            return processResult( result, session.getResult().getExceptions().get( 0 ) );
         }
 
         result.setTopologicallySortedProjects( session.getProjects() );

Modified: maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/execution/MavenSession.java
URL: http://svn.apache.org/viewvc/maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/execution/MavenSession.java?rev=774079&r1=774078&r2=774079&view=diff
==============================================================================
--- maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/execution/MavenSession.java (original)
+++ maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/execution/MavenSession.java Tue May 12 20:50:52 2009
@@ -20,11 +20,13 @@
  */
 
 import java.util.Arrays;
+import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.Properties;
 
 import org.apache.maven.artifact.repository.ArtifactRepository;
+import org.apache.maven.plugin.descriptor.PluginDescriptor;
 import org.apache.maven.project.MavenProject;
 import org.apache.maven.project.ProjectBuilderConfiguration;
 import org.apache.maven.settings.Settings;
@@ -41,6 +43,8 @@
     
     private MavenExecutionRequest request;
 
+    private MavenExecutionResult result;
+    
     private MavenProject currentProject;
         
     /**
@@ -57,17 +61,18 @@
         this.request = request;
     }
 
-    public MavenSession( PlexusContainer container, MavenExecutionRequest request, MavenProject project )
+    public MavenSession( PlexusContainer container, MavenExecutionRequest request, MavenExecutionResult result, MavenProject project )
         throws CycleDetectedException, DuplicateProjectException
     {
-        this( container, request, Arrays.asList( new MavenProject[]{ project } ) );        
+        this( container, request, result, Arrays.asList( new MavenProject[]{ project } ) );        
     }    
 
-    public MavenSession( PlexusContainer container, MavenExecutionRequest request, List<MavenProject> projects )
+    public MavenSession( PlexusContainer container, MavenExecutionRequest request, MavenExecutionResult result, List<MavenProject> projects )
         throws CycleDetectedException, DuplicateProjectException
     {
         this.container = container;
         this.request = request;
+        this.result = result;
         this.currentProject = projects.get( 0 );
         this.projects = projects;        
     }    
@@ -146,4 +151,42 @@
     {
         return topLevelProject;
     }
+
+    public MavenExecutionResult getResult()
+    {
+        return result;
+    }        
+    
+    // Backward compat
+    public Map<String,Map<String,Object>> getPluginContext( PluginDescriptor pluginDescriptor, MavenProject project )
+    {
+        return new HashMap<String,Map<String,Object>>();
+    }    
+
+    /*
+    private Map pluginContextsByProjectAndPluginKey = new HashMap();
+    
+    public Map getPluginContext( PluginDescriptor plugin, MavenProject project )
+    {
+        Map pluginContextsByKey = (Map) pluginContextsByProjectAndPluginKey.get( project.getId() );
+
+        if ( pluginContextsByKey == null )
+        {
+            pluginContextsByKey = new HashMap();
+
+            pluginContextsByProjectAndPluginKey.put( project.getId(), pluginContextsByKey );
+        }
+
+        Map pluginContext = (Map) pluginContextsByKey.get( plugin.getPluginLookupKey() );
+
+        if ( pluginContext == null )
+        {
+            pluginContext = new HashMap();
+            pluginContextsByKey.put( plugin.getPluginLookupKey(), pluginContext );
+        }
+
+        return pluginContext;
+    }
+    */
+    
 }
\ No newline at end of file

Modified: maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/lifecycle/LifecycleExecutor.java
URL: http://svn.apache.org/viewvc/maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/lifecycle/LifecycleExecutor.java?rev=774079&r1=774078&r2=774079&view=diff
==============================================================================
--- maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/lifecycle/LifecycleExecutor.java (original)
+++ maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/lifecycle/LifecycleExecutor.java Tue May 12 20:50:52 2009
@@ -67,8 +67,7 @@
     void populateDefaultConfigurationForPlugins( Collection<Plugin> plugins, MavenProject project, ArtifactRepository localRepository )
         throws LifecycleExecutionException;
     
-    void execute( MavenSession session )
-        throws LifecycleExecutionException, MojoFailureException, MojoExecutionException;
+    void execute( MavenSession session );
     
     Xpp3Dom getDefaultPluginConfiguration( String groupId, String artifactId, String version, String goal, MavenProject project, ArtifactRepository localRepository ) 
         throws LifecycleExecutionException;    

Modified: maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java
URL: http://svn.apache.org/viewvc/maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java?rev=774079&r1=774078&r2=774079&view=diff
==============================================================================
--- maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java (original)
+++ maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java Tue May 12 20:50:52 2009
@@ -261,8 +261,7 @@
             .setLocalRepository( localRepository )
             .setRemoteRepostories( project.getRemoteArtifactRepositories() )
             .setFilter( filter )
-            .setResolveTransitively( true )
-            .setResolveRoot( true ); // We are setting this to false because the artifact itself has been resolved.
+            .setResolveTransitively( true );
         
         ArtifactResolutionResult result = repositorySystem.resolve( request );
         resolutionErrorHandler.throwErrors( request, result );

Modified: maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java
URL: http://svn.apache.org/viewvc/maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java?rev=774079&r1=774078&r2=774079&view=diff
==============================================================================
--- maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java (original)
+++ maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java Tue May 12 20:50:52 2009
@@ -176,8 +176,7 @@
             {
                 pluginConfigurationExpander.expandPluginConfiguration( project.getModel() );
 
-                lifecycle.populateDefaultConfigurationForPlugins( project.getModel().getBuild().getPlugins(), project,
-                                                                  configuration.getLocalRepository() );
+                lifecycle.populateDefaultConfigurationForPlugins( project.getModel().getBuild().getPlugins(), project, configuration.getLocalRepository() );
             }
         }
         catch ( IOException e )
@@ -277,11 +276,14 @@
         throws ProjectBuildingException
     {
         MavenProject project = build( pomFile, configuration );
-        Artifact pomArtifact = repositorySystem.createProjectArtifact( project.getGroupId(), project.getArtifactId(), project.getVersion() );
-        pomArtifact.setFile( pomFile );
+        Artifact artifact = repositorySystem.createArtifact( project.getGroupId(), project.getArtifactId(), project.getVersion(), project.getPackaging() );
 
-        ArtifactResolutionRequest request = new ArtifactResolutionRequest().setArtifact( pomArtifact ).setResolveTransitively( true ).setArtifactDependencies( project.getDependencyArtifacts() )
-            .setLocalRepository( configuration.getLocalRepository() ).setRemoteRepostories( project.getRemoteArtifactRepositories() ).setManagedVersionMap( project.getManagedVersionMap() );
+        ArtifactResolutionRequest request = new ArtifactResolutionRequest()
+            .setArtifact( artifact )
+            .setResolveTransitively( true )
+            .setLocalRepository( configuration.getLocalRepository() )
+            .setRemoteRepostories( project.getRemoteArtifactRepositories() )
+            .setManagedVersionMap( project.getManagedVersionMap() );
 
         ArtifactResolutionResult result = repositorySystem.resolve( request );
 
@@ -294,7 +296,7 @@
         }
 
         project.setArtifacts( result.getArtifacts() );
-        project.getArtifacts().remove( pomArtifact );
+        project.getArtifacts().remove( artifact );
 
         return new MavenProjectBuildingResult( project, result );
     }

Modified: maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/project/MavenProject.java
URL: http://svn.apache.org/viewvc/maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/project/MavenProject.java?rev=774079&r1=774078&r2=774079&view=diff
==============================================================================
--- maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/project/MavenProject.java (original)
+++ maven/components/branches/MNG-2766/maven-core/src/main/java/org/apache/maven/project/MavenProject.java Tue May 12 20:50:52 2009
@@ -500,7 +500,7 @@
         list.add( getBuild().getOutputDirectory() );
 
         for ( Artifact a : getArtifacts() )
-        {            
+        {                        
             if ( a.getArtifactHandler().isAddedToClasspath() )
             {
                 // TODO: let the scope handler deal with this
@@ -516,63 +516,12 @@
         for( String s : list )
         {
             System.out.println( ">>>>> " + s );
-        }
-        */
-            
+        } 
+        */           
         
         return list;
     }
 
-    public List<Artifact> getCompileArtifacts()
-    {
-        List<Artifact> list = new ArrayList<Artifact>( getArtifacts().size() );
-
-        for ( Artifact a : getArtifacts() )
-        {
-            // TODO: classpath check doesn't belong here - that's the other method
-            if ( a.getArtifactHandler().isAddedToClasspath() )
-            {
-                // TODO: let the scope handler deal with this
-                if ( Artifact.SCOPE_COMPILE.equals( a.getScope() ) || Artifact.SCOPE_PROVIDED.equals( a.getScope() ) || Artifact.SCOPE_SYSTEM.equals( a.getScope() ) )
-                {
-                    list.add( a );
-                }
-            }
-        }
-        return list;
-    }
-
-    public List<Dependency> getCompileDependencies()
-    {
-        Set<Artifact> artifacts = getArtifacts();
-
-        if ( ( artifacts == null ) || artifacts.isEmpty() )
-        {
-            return Collections.emptyList();
-        }
-
-        List<Dependency> list = new ArrayList<Dependency>( artifacts.size() );
-
-        for ( Artifact a : getArtifacts()  )
-        {
-            // TODO: let the scope handler deal with this
-            if ( Artifact.SCOPE_COMPILE.equals( a.getScope() ) || Artifact.SCOPE_PROVIDED.equals( a.getScope() ) || Artifact.SCOPE_SYSTEM.equals( a.getScope() ) )
-            {
-                Dependency dependency = new Dependency();
-
-                dependency.setArtifactId( a.getArtifactId() );
-                dependency.setGroupId( a.getGroupId() );
-                dependency.setVersion( a.getVersion() );
-                dependency.setScope( a.getScope() );
-                dependency.setType( a.getType() );
-                dependency.setClassifier( a.getClassifier() );
-
-                list.add( dependency );
-            }
-        }
-        return list;
-    }
-
     public List<String> getTestClasspathElements()
         throws DependencyResolutionRequiredException
     {

Modified: maven/components/branches/MNG-2766/maven-core/src/test/java/org/apache/maven/AbstractCoreMavenComponentTestCase.java
URL: http://svn.apache.org/viewvc/maven/components/branches/MNG-2766/maven-core/src/test/java/org/apache/maven/AbstractCoreMavenComponentTestCase.java?rev=774079&r1=774078&r2=774079&view=diff
==============================================================================
--- maven/components/branches/MNG-2766/maven-core/src/test/java/org/apache/maven/AbstractCoreMavenComponentTestCase.java (original)
+++ maven/components/branches/MNG-2766/maven-core/src/test/java/org/apache/maven/AbstractCoreMavenComponentTestCase.java Tue May 12 20:50:52 2009
@@ -6,6 +6,7 @@
 
 import org.apache.maven.artifact.repository.ArtifactRepository;
 import org.apache.maven.execution.DefaultMavenExecutionRequest;
+import org.apache.maven.execution.DefaultMavenExecutionResult;
 import org.apache.maven.execution.MavenExecutionRequest;
 import org.apache.maven.execution.MavenSession;
 import org.apache.maven.model.Model;
@@ -106,7 +107,7 @@
             project = createStubMavenProject();
         }
                         
-        MavenSession session = new MavenSession( getContainer(), request, project );
+        MavenSession session = new MavenSession( getContainer(), request, new DefaultMavenExecutionResult(), project );
         
         return session;
     }      

Modified: maven/components/branches/MNG-2766/maven-core/src/test/java/org/apache/maven/MavenTest.java
URL: http://svn.apache.org/viewvc/maven/components/branches/MNG-2766/maven-core/src/test/java/org/apache/maven/MavenTest.java?rev=774079&r1=774078&r2=774079&view=diff
==============================================================================
--- maven/components/branches/MNG-2766/maven-core/src/test/java/org/apache/maven/MavenTest.java (original)
+++ maven/components/branches/MNG-2766/maven-core/src/test/java/org/apache/maven/MavenTest.java Tue May 12 20:50:52 2009
@@ -2,8 +2,11 @@
 
 import java.io.File;
 
+import org.apache.maven.exception.ExceptionHandler;
+import org.apache.maven.exception.ExceptionSummary;
 import org.apache.maven.execution.MavenExecutionRequest;
 import org.apache.maven.execution.MavenExecutionResult;
+import org.apache.maven.execution.MavenSession;
 import org.codehaus.plexus.component.annotations.Requirement;
 
 public class MavenTest
@@ -12,11 +15,15 @@
     @Requirement
     private Maven maven;
 
+    @Requirement
+    private ExceptionHandler exceptionHandler;
+
     protected void setUp()
         throws Exception
     {
         super.setUp();
         maven = lookup( Maven.class );
+        exceptionHandler = lookup( ExceptionHandler.class );
     }
 
     protected String getProjectsDirectory()
@@ -24,15 +31,17 @@
         return "src/test/projects/lifecycle-executor";
     }
 
-    // -----------------------------------------------------------------------------------------------
-    // 
-    // -----------------------------------------------------------------------------------------------
-
-    public void testMaven()
+    public void testLifecycleExecutionUsingADefaultLifecyclePhase()
         throws Exception
     {
         File pom = getProject( "project-with-additional-lifecycle-elements" );
         MavenExecutionRequest request = createMavenExecutionRequest( pom );
-        MavenExecutionResult result = maven.execute( request );        
+        MavenExecutionResult result = maven.execute( request );
+        if ( result.hasExceptions() )
+        {
+            ExceptionSummary es = exceptionHandler.handleException( result.getExceptions().get( 0 ) );
+            System.out.println( es.getMessage() );
+            fail( "Maven did not execute correctly." );
+        }
     }
 }

Modified: maven/components/branches/MNG-2766/maven-core/src/test/java/org/apache/maven/lifecycle/LifecycleExecutorTest.java
URL: http://svn.apache.org/viewvc/maven/components/branches/MNG-2766/maven-core/src/test/java/org/apache/maven/lifecycle/LifecycleExecutorTest.java?rev=774079&r1=774078&r2=774079&view=diff
==============================================================================
--- maven/components/branches/MNG-2766/maven-core/src/test/java/org/apache/maven/lifecycle/LifecycleExecutorTest.java (original)
+++ maven/components/branches/MNG-2766/maven-core/src/test/java/org/apache/maven/lifecycle/LifecycleExecutorTest.java Tue May 12 20:50:52 2009
@@ -5,6 +5,8 @@
 import java.util.List;
 
 import org.apache.maven.AbstractCoreMavenComponentTestCase;
+import org.apache.maven.exception.ExceptionHandler;
+import org.apache.maven.exception.ExceptionSummary;
 import org.apache.maven.execution.MavenSession;
 import org.apache.maven.model.Plugin;
 import org.apache.maven.plugin.MojoExecution;
@@ -23,6 +25,7 @@
     {
         super.setUp();
         lifecycleExecutor = (DefaultLifecycleExecutor) lookup( LifecycleExecutor.class );
+        lookup( ExceptionHandler.class );
     }
 
     protected String getProjectsDirectory()
@@ -70,17 +73,7 @@
         assertEquals( "surefire:test", lifecyclePlan.get( 6 ).getMojoDescriptor().getFullGoalName() );
         assertEquals( "jar:jar", lifecyclePlan.get( 7 ).getMojoDescriptor().getFullGoalName() );        
     }    
-    
-    public void testLifecycleExecutionUsingADefaultLifecyclePhase()
-        throws Exception
-    {
-        File pom = getProject( "project-with-additional-lifecycle-elements" );
-        MavenSession session = createMavenSession( pom );        
-        assertEquals( "project-with-additional-lifecycle-elements", session.getCurrentProject().getArtifactId() );
-        assertEquals( "1.0", session.getCurrentProject().getVersion() );                                
-        lifecycleExecutor.execute( session );
-    }    
-    
+        
     public void testLifecyclePluginsRetrievalForDefaultLifecycle()
         throws Exception
     {

Modified: maven/components/branches/MNG-2766/maven-core/src/test/java/org/apache/maven/project/EmptyLifecycleExecutor.java
URL: http://svn.apache.org/viewvc/maven/components/branches/MNG-2766/maven-core/src/test/java/org/apache/maven/project/EmptyLifecycleExecutor.java?rev=774079&r1=774078&r2=774079&view=diff
==============================================================================
--- maven/components/branches/MNG-2766/maven-core/src/test/java/org/apache/maven/project/EmptyLifecycleExecutor.java (original)
+++ maven/components/branches/MNG-2766/maven-core/src/test/java/org/apache/maven/project/EmptyLifecycleExecutor.java Tue May 12 20:50:52 2009
@@ -51,7 +51,6 @@
     }
 
     public void execute( MavenSession session )
-        throws LifecycleExecutionException, MojoFailureException, MojoExecutionException
     {
     }
 

Modified: maven/components/branches/MNG-2766/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java
URL: http://svn.apache.org/viewvc/maven/components/branches/MNG-2766/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java?rev=774079&r1=774078&r2=774079&view=diff
==============================================================================
--- maven/components/branches/MNG-2766/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java (original)
+++ maven/components/branches/MNG-2766/maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java Tue May 12 20:50:52 2009
@@ -173,9 +173,16 @@
         {
             ExceptionSummary es = result.getExceptionSummary();
             
+            if ( es == null )
+            {
+                result.getExceptions().get( 0 ).printStackTrace();
+            }
+            else
+            {
             System.out.println( es.getMessage() );
             
             es.getException().printStackTrace();
+            }
             
             return 1;
         }

Modified: maven/components/branches/MNG-2766/maven-repository/src/main/java/org/apache/maven/repository/LegacyRepositorySystem.java
URL: http://svn.apache.org/viewvc/maven/components/branches/MNG-2766/maven-repository/src/main/java/org/apache/maven/repository/LegacyRepositorySystem.java?rev=774079&r1=774078&r2=774079&view=diff
==============================================================================
--- maven/components/branches/MNG-2766/maven-repository/src/main/java/org/apache/maven/repository/LegacyRepositorySystem.java (original)
+++ maven/components/branches/MNG-2766/maven-repository/src/main/java/org/apache/maven/repository/LegacyRepositorySystem.java Tue May 12 20:50:52 2009
@@ -79,14 +79,14 @@
         return artifactFactory.createArtifact( groupId, artifactId, version, scope, type );
     }
 
-    public Artifact createArtifactWithClassifier( String groupId, String artifactId, String version, String type, String classifier )
+    public Artifact createArtifact( String groupId, String artifactId, String version, String packaging )
     {
-        return artifactFactory.createArtifactWithClassifier( groupId, artifactId, version, type, classifier );
+        return artifactFactory.createBuildArtifact( groupId, artifactId, version, packaging );
     }
-
-    public Artifact createBuildArtifact( String groupId, String artifactId, String version, String packaging )
+    
+    public Artifact createArtifactWithClassifier( String groupId, String artifactId, String version, String type, String classifier )
     {
-        return artifactFactory.createBuildArtifact( groupId, artifactId, version, packaging );
+        return artifactFactory.createArtifactWithClassifier( groupId, artifactId, version, type, classifier );
     }
 
     public Artifact createProjectArtifact( String groupId, String artifactId, String metaVersionId )

Modified: maven/components/branches/MNG-2766/maven-repository/src/main/java/org/apache/maven/repository/RepositorySystem.java
URL: http://svn.apache.org/viewvc/maven/components/branches/MNG-2766/maven-repository/src/main/java/org/apache/maven/repository/RepositorySystem.java?rev=774079&r1=774078&r2=774079&view=diff
==============================================================================
--- maven/components/branches/MNG-2766/maven-repository/src/main/java/org/apache/maven/repository/RepositorySystem.java (original)
+++ maven/components/branches/MNG-2766/maven-repository/src/main/java/org/apache/maven/repository/RepositorySystem.java Tue May 12 20:50:52 2009
@@ -46,9 +46,11 @@
 
     static final String DEFAULT_REMOTE_REPO_URL = "http://repo1.maven.org/maven2";
 
+    Artifact createArtifact( String groupId, String artifactId, String version, String packaging );
+
     Artifact createArtifact( String groupId, String artifactId, String version, String scope, String type );
 
-    Artifact createProjectArtifact( String groupId, String artifactId, String metaVersionId );
+    Artifact createProjectArtifact( String groupId, String artifactId, String version );
 
     Artifact createPluginArtifact( Plugin plugin );