merge changes for rev 193190 from the alpha-3 tag

git-svn-id: https://svn.apache.org/repos/asf/maven/components/trunk@193195 13f79535-47bb-0310-9956-ffa450edef68

maven-artifact-ant/sample.build.xml

@@ -20,7 +20,7 @@
   <target name="initTaskDefs">
     <path id="maven.classpath">
       <!-- don't forget to set the value! -->
-      <pathelement location="target/maven-artifact-ant-2.0-SNAPSHOT-dep.jar"/>
+      <pathelement location="target/maven-artifact-ant-2.0-alpha-3-dep.jar"/>
     </path>
 
     <typedef resource="org/apache/maven/artifact/ant/antlib.xml" uri="urn:maven-artifact-ant">
@@ -63,16 +63,16 @@
       <fileset refid="my.dependency.fileset"/>
     </copy>
 
-    <artifact:install file="target/maven-artifact-ant-2.0-SNAPSHOT.jar">
+    <artifact:install file="target/maven-artifact-ant-2.0-alpha-3.jar">
       <pom refid="maven.project"/>
     </artifact:install>
 
-    <artifact:deploy file="target/maven-artifact-ant-2.0-SNAPSHOT.jar">
+    <artifact:deploy file="target/maven-artifact-ant-2.0-alpha-3.jar">
       <remoteRepository refid="deploy.repository"/>
       <pom refid="maven.project"/>
     </artifact:deploy>
 
-    <artifact:deploy file="target/maven-artifact-ant-2.0-SNAPSHOT.jar">
+    <artifact:deploy file="target/maven-artifact-ant-2.0-alpha-3.jar">
       <remoteRepository url="scp://localhost/tmp/deployment-repo">
         <authentication username="brett" privateKey="${user.home}/.ssh/id_dsa"/>
       </remoteRepository>
@@ -128,7 +128,7 @@
       <remoteRepository refid="deploy.repository"/>
     </artifact:dependencies>
 
-    <artifact:deploy file="target/maven-artifact-ant-2.0-SNAPSHOT.jar">
+    <artifact:deploy file="target/maven-artifact-ant-2.0-alpha-3.jar">
       <localRepository refid="local.repository.scm"/>
       <remoteRepository refid="deploy.repository"/>
       <pom refid="maven.project"/>

maven-core/pom.xml

@@ -133,7 +133,7 @@
   <distributionManagement>
     <site>
       <id>website</id>
-      <url>scp://www.apache.org//www/maven.apache.org/m2</url>
+      <url>scp://minotaur.apache.org//www/maven.apache.org/m2</url>
     </site>
   </distributionManagement>
   <build>

maven-core/src/assemble/bin.xml

@@ -28,6 +28,9 @@
       <includes>
         <include>maven-core-*.jar</include>
       </includes>
+      <excludes>
+        <exclude>maven-core-*-sources.jar</exclude>
+      </excludes>
     </fileSet>
   </fileSets>
   <dependencySets>
@@ -35,6 +38,7 @@
       <outputDirectory>core</outputDirectory>
       <includes>
         <include>plexus:plexus-container-default</include>
+        <include>plexus:plexus-utils</include>
       </includes>
     </dependencySet>
     <dependencySet>
@@ -48,7 +52,6 @@
       <excludes>
         <exclude>plexus:plexus-container-default</exclude>
         <exclude>classworlds:classworlds</exclude>
-        <!-- included in default container - not needed -->
         <exclude>plexus:plexus-utils</exclude>
         <!-- transitively picked up via default container - not needed -->
         <exclude>junit:junit</exclude>

maven-core/src/main/java/org/apache/maven/cli/MavenCli.java

@@ -55,10 +55,12 @@
 import java.io.File;
 import java.io.IOException;
 import java.io.InputStreamReader;
+import java.io.InputStream;
 import java.net.URL;
 import java.util.Collections;
 import java.util.Enumeration;
 import java.util.List;
+import java.util.Properties;
 
 /**
  * @author <a href="mailto:jason@maven.org">Jason van Zyl</a>
@@ -416,35 +418,20 @@ else if ( commandLine.hasOption( CLIManager.CHECKSUM_WARNING_POLICY ) )
 
     private static void showVersion()
     {
-        // TODO: is there a beter way? Maybe read the manifest?
+        InputStream resourceAsStream;
-
-        String version = "unknown";
-
         try
         {
-            for ( Enumeration e = MavenCli.class.getClassLoader().getResources( "/META-INF/maven/pom.xml" );
+            Properties properties = new Properties();
-                  e.hasMoreElements(); )
+            resourceAsStream = MavenCli.class.getClassLoader().getResourceAsStream(
-            {
+                "META-INF/maven/org.apache.maven/maven-core/pom.properties" );
-                URL resource = (URL) e.nextElement();
+            properties.load( resourceAsStream );
-                if ( resource.getPath().indexOf( "maven-core" ) >= 0 )
-                {
-                    MavenXpp3Reader reader = new MavenXpp3Reader();
-                    Model model = reader.read( new InputStreamReader( resource.openStream() ) );
-                    version = model.getVersion();
-                    break;
-                }
-            }
 
-            System.out.println( "Maven version: " + version );
+            System.out.println( "Maven version: " + properties.getProperty( "version", "unknown" ) );
         }
         catch ( IOException e )
         {
             System.err.println( "Unable determine version from JAR file: " + e.getMessage() );
         }
-        catch ( XmlPullParserException e )
-        {
-            System.err.println( "Unable to parse POM in JAR file: " + e.getMessage() );
-        }
     }
 
     // ----------------------------------------------------------------------

maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java

@@ -562,7 +562,7 @@ private void ensurePluginContainerIsComplete( PluginDescriptor pluginDescriptor,
                 
                 ArtifactRepository localRepository = session.getLocalRepository();
                 Set dependencies = metadataSource.retrieve( pluginArtifact, localRepository,
-                                                            remoteRepositories );
+                                                            project.getPluginArtifactRepositories() );
 
                 ArtifactResolutionResult result = artifactResolver.resolveTransitively( dependencies, pluginArtifact,
                                                                                         localRepository,
@@ -1057,17 +1057,6 @@ private void downloadDependencies( MavenProject project, MavenSession context, A
 
             artifactResolver.resolve( artifact, remoteArtifactRepositories, localRepository );
         }
-
-        // TODO: is this really necessary?
-        for ( Iterator it = project.getPluginArtifacts().iterator(); it.hasNext(); )
-        {
-            Artifact artifact = (Artifact) it.next();
-
-            artifactResolver.resolve( artifact, remoteArtifactRepositories, localRepository );
-        }
-
-        // TODO: is this really necessary?
-        artifactResolver.resolve( project.getParentArtifact(), remoteArtifactRepositories, localRepository );
     }
 
 }