Author: jvanzyl Date: Sat Mar 10 16:48:53 2007 New Revision: 516820 URL: http://svn.apache.org/viewvc?view=rev&rev=516820 Log: o testing repeatedly with an empty repository
Modified: maven/components/trunk/build.properties maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/WagonManager.java maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java maven/components/trunk/maven-core/src/main/java/org/apache/maven/extension/DefaultExtensionManager.java maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/version/DefaultPluginVersionManager.java maven/components/trunk/maven-core/src/main/resources/META-INF/plexus/components.xml maven/components/trunk/maven-embedder/pom.xml maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedder.java maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/execution/DefaultMavenExecutionRequestDefaultsPopulator.java maven/components/trunk/maven-embedder/src/test/embedder-test-project/pom.xml maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/embedder/MavenEmbedderTest.java maven/components/trunk/maven-embedder/src/test/resources/pom.xml maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/MavenMetadataSource.java maven/components/trunk/pom.xml Modified: maven/components/trunk/build.properties URL: http://svn.apache.org/viewvc/maven/components/trunk/build.properties?view=diff&rev=516820&r1=516819&r2=516820 ============================================================================== --- maven/components/trunk/build.properties (original) +++ maven/components/trunk/build.properties Sat Mar 10 16:48:53 2007 @@ -18,7 +18,7 @@ classworlds.version=1.2-alpha-7 plexus.version=1.0-alpha-19 plexus-archiver.version=1.0-alpha-8 -plexus-utils.version=1.4 +plexus-utils.version=1.4.1 plexus-interactivity-api.version=1.0-alpha-6-SNAPSHOT commons-cli.version=1.0 commons-lang.version=2.1 Modified: maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java?view=diff&rev=516820&r1=516819&r2=516820 ============================================================================== --- maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java (original) +++ maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/DefaultWagonManager.java Sat Mar 10 16:48:53 2007 @@ -328,6 +328,7 @@ else { getLogger().debug( "Trying repository " + repository.getId() ); + getRemoteFile( repository, artifact.getFile(), remotePath, downloadMonitor, policy.getChecksumPolicy(), false ); getLogger().debug( " Artifact resolved" ); @@ -785,6 +786,20 @@ public void setInteractive( boolean interactive ) { this.interactive = interactive; + } + + public void findAndRegisterWagons( PlexusContainer container ) + { + try + { + Map wagons = container.lookupMap( Wagon.ROLE ); + + registerWagons( wagons.keySet(), container ); + } + catch ( ComponentLookupException e ) + { + // no wagons found in the extension + } } public void registerWagons( Collection wagons, PlexusContainer extensionContainer ) Modified: maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/WagonManager.java URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/WagonManager.java?view=diff&rev=516820&r1=516819&r2=516820 ============================================================================== --- maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/WagonManager.java (original) +++ maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/manager/WagonManager.java Sat Mar 10 16:48:53 2007 @@ -117,4 +117,6 @@ void setInteractive( boolean interactive ); void registerWagons( Collection wagons, PlexusContainer extensionContainer ); + + void findAndRegisterWagons( PlexusContainer container ); } Modified: maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java?view=diff&rev=516820&r1=516819&r2=516820 ============================================================================== --- maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java (original) +++ maven/components/trunk/maven-artifact-manager/src/main/java/org/apache/maven/artifact/resolver/DefaultArtifactResolver.java Sat Mar 10 16:48:53 2007 @@ -295,7 +295,8 @@ } catch ( ArtifactNotFoundException anfe ) { - getLogger().debug( anfe.getMessage() ); + getLogger().debug( anfe.getMessage(), anfe ); + missingArtifacts.add( node.getArtifact() ); } } @@ -303,14 +304,6 @@ if ( missingArtifacts.size() > 0 ) { throw new MultipleArtifactsNotFoundException( originatingArtifact, missingArtifacts, remoteRepositories ); -// String message = "required artifacts missing:\n"; -// for ( Iterator i = missingArtifacts.iterator(); i.hasNext(); ) -// { -// Artifact missingArtifact = (Artifact) i.next(); -// message += " " + missingArtifact.getId() + "\n"; -// } -// message += "\nfor the artifact:"; -// throw new ArtifactResolutionException( message, originatingArtifact, remoteRepositories ); } return artifactResolutionResult; Modified: maven/components/trunk/maven-core/src/main/java/org/apache/maven/extension/DefaultExtensionManager.java URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/main/java/org/apache/maven/extension/DefaultExtensionManager.java?view=diff&rev=516820&r1=516819&r2=516820 ============================================================================== --- maven/components/trunk/maven-core/src/main/java/org/apache/maven/extension/DefaultExtensionManager.java (original) +++ maven/components/trunk/maven-core/src/main/java/org/apache/maven/extension/DefaultExtensionManager.java Sat Mar 10 16:48:53 2007 @@ -158,16 +158,7 @@ public void registerWagons() { - try - { - Map wagons = container.lookupMap( Wagon.ROLE ); - - wagonManager.registerWagons( wagons.keySet(), container ); - } - catch ( ComponentLookupException e ) - { - // no wagons found in the extension - } + wagonManager.findAndRegisterWagons( container ); } public void contextualize( Context context ) Modified: maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/version/DefaultPluginVersionManager.java URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/version/DefaultPluginVersionManager.java?view=diff&rev=516820&r1=516819&r2=516820 ============================================================================== --- maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/version/DefaultPluginVersionManager.java (original) +++ maven/components/trunk/maven-core/src/main/java/org/apache/maven/plugin/version/DefaultPluginVersionManager.java Sat Mar 10 16:48:53 2007 @@ -35,12 +35,9 @@ import org.apache.maven.model.Plugin; import org.apache.maven.model.ReportPlugin; import org.apache.maven.plugin.InvalidPluginException; -import org.apache.maven.plugin.registry.MavenPluginRegistryBuilder; -import org.apache.maven.plugin.registry.PluginRegistry; import org.apache.maven.project.MavenProject; import org.apache.maven.project.MavenProjectBuilder; import org.apache.maven.project.ProjectBuildingException; -import org.codehaus.plexus.components.interactivity.InputHandler; import org.codehaus.plexus.logging.AbstractLogEnabled; import org.codehaus.plexus.util.StringUtils; @@ -53,17 +50,10 @@ extends AbstractLogEnabled implements PluginVersionManager { - private MavenPluginRegistryBuilder mavenPluginRegistryBuilder; - private ArtifactFactory artifactFactory; - private InputHandler inputHandler; - private ArtifactMetadataSource artifactMetadataSource; - // TODO: [jc] Revisit to remove this piece of state. PLUGIN REGISTRY MAY BE UPDATED ON DISK OUT-OF-PROCESS! - private PluginRegistry pluginRegistry; - private MavenProjectBuilder mavenProjectBuilder; private RuntimeInformation runtimeInformation; @@ -118,21 +108,12 @@ } } - // we're NEVER going to persist POM-derived plugin versions. - String updatedVersion = null; - // third pass...we're always checking for latest install/deploy, so retrieve the version for LATEST metadata and // also set that resolved version as the <useVersion/> in settings.xml. if ( StringUtils.isEmpty( version ) ) { // 1. resolve the version to be used version = resolveMetaVersion( groupId, artifactId, project, localRepository, Artifact.LATEST_VERSION ); - - if ( version != null ) - { - // 2. Set the updatedVersion so the user will be prompted whether to make this version permanent. - updatedVersion = version; - } } // final pass...retrieve the version for RELEASE and also set that resolved version as the <useVersion/> @@ -141,12 +122,6 @@ { // 1. resolve the version to be used version = resolveMetaVersion( groupId, artifactId, project, localRepository, Artifact.RELEASE_VERSION ); - - if ( version != null ) - { - // 2. Set the updatedVersion so the user will be prompted whether to make this version permanent. - updatedVersion = version; - } } // if we're still empty here, and the current project matches the plugin in question, use the current project's Modified: maven/components/trunk/maven-core/src/main/resources/META-INF/plexus/components.xml URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-core/src/main/resources/META-INF/plexus/components.xml?view=diff&rev=516820&r1=516819&r2=516820 ============================================================================== --- maven/components/trunk/maven-core/src/main/resources/META-INF/plexus/components.xml (original) +++ maven/components/trunk/maven-core/src/main/resources/META-INF/plexus/components.xml Sat Mar 10 16:48:53 2007 @@ -611,14 +611,7 @@ <role-hint>default</role-hint> </requirement> <requirement> - <role>org.apache.maven.plugin.registry.MavenPluginRegistryBuilder</role> - </requirement> - <requirement> <role>org.apache.maven.artifact.factory.ArtifactFactory</role> - </requirement> - <requirement> - <role>org.codehaus.plexus.components.interactivity.InputHandler</role> - <role-hint>default</role-hint> </requirement> <requirement> <role>org.apache.maven.project.MavenProjectBuilder</role> Modified: maven/components/trunk/maven-embedder/pom.xml URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/pom.xml?view=diff&rev=516820&r1=516819&r2=516820 ============================================================================== --- maven/components/trunk/maven-embedder/pom.xml (original) +++ maven/components/trunk/maven-embedder/pom.xml Sat Mar 10 16:48:53 2007 @@ -86,6 +86,7 @@ <exclude>classworlds:classworlds</exclude> <exclude>junit:junit</exclude> <exclude>jmock:jmock</exclude> + <exclude>xml-apis:xml-apis</exclude> </excludes> <!-- <dependenciesToHide> @@ -128,6 +129,18 @@ <groupId>commons-logging</groupId> </exclusion> </exclusions> + </dependency> + <dependency> + <groupId>org.apache.maven.wagon</groupId> + <artifactId>wagon-http-lightweight</artifactId> + </dependency> + <dependency> + <groupId>org.apache.maven.wagon</groupId> + <artifactId>wagon-ssh</artifactId> + </dependency> + <dependency> + <groupId>org.apache.maven.wagon</groupId> + <artifactId>wagon-ssh-external</artifactId> </dependency> </dependencies> <distributionManagement> Modified: maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedder.java URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedder.java?view=diff&rev=516820&r1=516819&r2=516820 ============================================================================== --- maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedder.java (original) +++ maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/MavenEmbedder.java Sat Mar 10 16:48:53 2007 @@ -427,10 +427,14 @@ { request = defaultsPopulator.populateDefaults( request, this ); //mkleint: copied from DefaultLifecycleExecutor + project = readProject( new File( request.getPomFile() ) ); + Map handlers = findArtifactTypeHandlers( project ); + //is this necessary in this context, I doubt it..mkleint artifactHandlerManager.addHandlers( handlers ); + project = mavenProjectBuilder.buildWithDependencies( new File( request.getPomFile() ), request.getLocalRepository(), profileManager, request.getTransferListener() ); Modified: maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/execution/DefaultMavenExecutionRequestDefaultsPopulator.java URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/execution/DefaultMavenExecutionRequestDefaultsPopulator.java?view=diff&rev=516820&r1=516819&r2=516820 ============================================================================== --- maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/execution/DefaultMavenExecutionRequestDefaultsPopulator.java (original) +++ maven/components/trunk/maven-embedder/src/main/java/org/apache/maven/embedder/execution/DefaultMavenExecutionRequestDefaultsPopulator.java Sat Mar 10 16:48:53 2007 @@ -113,6 +113,8 @@ } else { + wagonManager.findAndRegisterWagons( container ); + wagonManager.setInteractive( request.isInteractiveMode() ); wagonManager.setDownloadMonitor( request.getTransferListener() ); Modified: maven/components/trunk/maven-embedder/src/test/embedder-test-project/pom.xml URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/src/test/embedder-test-project/pom.xml?view=diff&rev=516820&r1=516819&r2=516820 ============================================================================== --- maven/components/trunk/maven-embedder/src/test/embedder-test-project/pom.xml (original) +++ maven/components/trunk/maven-embedder/src/test/embedder-test-project/pom.xml Sat Mar 10 16:48:53 2007 @@ -17,25 +17,6 @@ under the License. --> -<!-- -Licensed to the Apache Software Foundation (ASF) under one -or more contributor license agreements. See the NOTICE file -distributed with this work for additional information -regarding copyright ownership. The ASF licenses this file -to you under the Apache License, Version 2.0 (the -"License"); you may not use this file except in compliance -with the License. You may obtain a copy of the License at - - http://www.apache.org/licenses/LICENSE-2.0 - -Unless required by applicable law or agreed to in writing, -software distributed under the License is distributed on an -"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY -KIND, either express or implied. See the License for the -specific language governing permissions and limitations -under the License. ---> - <project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd"> <modelVersion>4.0.0</modelVersion> Modified: maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/embedder/MavenEmbedderTest.java URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/embedder/MavenEmbedderTest.java?view=diff&rev=516820&r1=516819&r2=516820 ============================================================================== --- maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/embedder/MavenEmbedderTest.java (original) +++ maven/components/trunk/maven-embedder/src/test/java/org/apache/maven/embedder/MavenEmbedderTest.java Sat Mar 10 16:48:53 2007 @@ -247,9 +247,21 @@ throws Exception { MavenExecutionRequest request = new DefaultMavenExecutionRequest().setShowErrors( true ) - .setPomFile( getPomFile().getAbsolutePath() ); + .setPomFile( getPomFile().getAbsolutePath() ).setShowErrors( true ); MavenExecutionResult result = maven.readProjectWithDependencies( request ); + + if ( result.hasExceptions() ) + { + for ( Iterator i = result.getExceptions().iterator(); i.hasNext(); ) + { + Exception e = (Exception) i.next(); + + e.printStackTrace(); + } + + fail( "Exception is readProjectWithDependencies() test." ); + } assertNoExceptions( result ); Modified: maven/components/trunk/maven-embedder/src/test/resources/pom.xml URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-embedder/src/test/resources/pom.xml?view=diff&rev=516820&r1=516819&r2=516820 ============================================================================== --- maven/components/trunk/maven-embedder/src/test/resources/pom.xml (original) +++ maven/components/trunk/maven-embedder/src/test/resources/pom.xml Sat Mar 10 16:48:53 2007 @@ -21,31 +21,9 @@ <modelVersion>4.0.0</modelVersion> <groupId>org.apache.maven</groupId> <artifactId>maven-model</artifactId> - <name>Maven Model</name> - <version>2.0-beta-2-SNAPSHOT</version> - <description>Maven Model</description> - <build> - <plugins> - <plugin> - <groupId>org.codehaus.modello</groupId> - <artifactId>modello-maven-plugin</artifactId> - <executions> - <execution> - <goals> - <goal>xpp3-writer</goal> - <goal>java</goal> - <goal>xpp3-reader</goal> - <goal>xsd</goal> - </goals> - </execution> - </executions> - <configuration> - <version>4.0.0</version> - <model>maven.mdo</model> - </configuration> - </plugin> - </plugins> - </build> + <name>Maven Embedder Test Project</name> + <version>1.0</version> + <description>Maven Embedder Test Project</description> <dependencies> <dependency> <groupId>plexus</groupId> Modified: maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java?view=diff&rev=516820&r1=516819&r2=516820 ============================================================================== --- maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java (original) +++ maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java Sat Mar 10 16:48:53 2007 @@ -19,25 +19,6 @@ * under the License. */ -/* - * Licensed to the Apache Software Foundation (ASF) under one - * or more contributor license agreements. See the NOTICE file - * distributed with this work for additional information - * regarding copyright ownership. The ASF licenses this file - * to you under the Apache License, Version 2.0 (the - * "License"); you may not use this file except in compliance - * with the License. You may obtain a copy of the License at - * - * http://www.apache.org/licenses/LICENSE-2.0 - * - * Unless required by applicable law or agreed to in writing, - * software distributed under the License is distributed on an - * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY - * KIND, either express or implied. See the License for the - * specific language governing permissions and limitations - * under the License. - */ - import org.apache.maven.MavenTools; import org.apache.maven.artifact.Artifact; import org.apache.maven.artifact.ArtifactStatus; Modified: maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/MavenMetadataSource.java URL: http://svn.apache.org/viewvc/maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/MavenMetadataSource.java?view=diff&rev=516820&r1=516819&r2=516820 ============================================================================== --- maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/MavenMetadataSource.java (original) +++ maven/components/trunk/maven-project/src/main/java/org/apache/maven/project/artifact/MavenMetadataSource.java Sat Mar 10 16:48:53 2007 @@ -59,7 +59,7 @@ import java.util.Set; /** - * @author <a href="mailto:[EMAIL PROTECTED]">Jason van Zyl</a> + * @author Jason van Zyl * @author <a href="mailto:[EMAIL PROTECTED]">Brett Porter</a> * @version $Id$ */ Modified: maven/components/trunk/pom.xml URL: http://svn.apache.org/viewvc/maven/components/trunk/pom.xml?view=diff&rev=516820&r1=516819&r2=516820 ============================================================================== --- maven/components/trunk/pom.xml (original) +++ maven/components/trunk/pom.xml Sat Mar 10 16:48:53 2007 @@ -24,7 +24,7 @@ <parent> <groupId>org.apache.maven</groupId> <artifactId>maven-parent</artifactId> - <version>6-SNAPSHOT</version> + <version>5</version> <relativePath>../pom/maven/pom.xml</relativePath> </parent> <artifactId>maven</artifactId> @@ -177,6 +177,21 @@ <dependency> <groupId>org.apache.maven.wagon</groupId> <artifactId>wagon-file</artifactId> + <version>${wagonVersion}</version> + </dependency> + <dependency> + <groupId>org.apache.maven.wagon</groupId> + <artifactId>wagon-http-lightweight</artifactId> + <version>${wagonVersion}</version> + </dependency> + <dependency> + <groupId>org.apache.maven.wagon</groupId> + <artifactId>wagon-ssh</artifactId> + <version>${wagonVersion}</version> + </dependency> + <dependency> + <groupId>org.apache.maven.wagon</groupId> + <artifactId>wagon-ssh-external</artifactId> <version>${wagonVersion}</version> </dependency> <dependency>