You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by jd...@apache.org on 2009/06/09 19:07:28 UTC
svn commit: r783068 - in
/maven/components/branches/maven-2.2.0-RC/maven-project/src:
main/java/org/apache/maven/project/
main/java/org/apache/maven/project/interpolation/
main/resources/META-INF/plexus/ test/java/org/apache/maven/project/
test/java/or...
Author: jdcasey
Date: Tue Jun 9 17:07:27 2009
New Revision: 783068
URL: http://svn.apache.org/viewvc?rev=783068&view=rev
Log:
[MNG-4167] Remove coordinate interpolator. Not reverting to VersionExpressionTransformation, since both cause problems.
Removed:
maven/components/branches/maven-2.2.0-RC/maven-project/src/main/java/org/apache/maven/project/interpolation/CoordinateInterpolator.java
maven/components/branches/maven-2.2.0-RC/maven-project/src/main/java/org/apache/maven/project/interpolation/DefaultCoordinateInterpolator.java
maven/components/branches/maven-2.2.0-RC/maven-project/src/test/java/org/apache/maven/project/interpolation/AbstractCoordinateInterpolationTest.java
maven/components/branches/maven-2.2.0-RC/maven-project/src/test/java/org/apache/maven/project/interpolation/DefaultCoordinateInterpolatorTest.java
Modified:
maven/components/branches/maven-2.2.0-RC/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java
maven/components/branches/maven-2.2.0-RC/maven-project/src/main/resources/META-INF/plexus/components.xml
maven/components/branches/maven-2.2.0-RC/maven-project/src/test/java/org/apache/maven/project/MavenProjectDynamismTest.java
maven/components/branches/maven-2.2.0-RC/maven-project/src/test/resources/org/apache/maven/project/ProjectClasspathTest.xml
maven/components/branches/maven-2.2.0-RC/maven-project/src/test/resources/org/apache/maven/project/canonical/CanonicalProjectBuilderTest.xml
Modified: maven/components/branches/maven-2.2.0-RC/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java
URL: http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.0-RC/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java?rev=783068&r1=783067&r2=783068&view=diff
==============================================================================
--- maven/components/branches/maven-2.2.0-RC/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java (original)
+++ maven/components/branches/maven-2.2.0-RC/maven-project/src/main/java/org/apache/maven/project/DefaultMavenProjectBuilder.java Tue Jun 9 17:07:27 2009
@@ -19,25 +19,6 @@
* under the License.
*/
-import java.io.File;
-import java.io.FileNotFoundException;
-import java.io.IOException;
-import java.io.Reader;
-import java.io.StringReader;
-import java.net.URL;
-import java.util.ArrayList;
-import java.util.Collections;
-import java.util.Date;
-import java.util.HashMap;
-import java.util.HashSet;
-import java.util.Iterator;
-import java.util.LinkedHashSet;
-import java.util.LinkedList;
-import java.util.List;
-import java.util.Map;
-import java.util.Set;
-import java.util.TreeMap;
-
import org.apache.maven.artifact.Artifact;
import org.apache.maven.artifact.ArtifactStatus;
import org.apache.maven.artifact.ArtifactUtils;
@@ -82,7 +63,6 @@
import org.apache.maven.project.inheritance.ModelInheritanceAssembler;
import org.apache.maven.project.injection.ModelDefaultsInjector;
import org.apache.maven.project.injection.ProfileInjector;
-import org.apache.maven.project.interpolation.CoordinateInterpolator;
import org.apache.maven.project.interpolation.ModelInterpolationException;
import org.apache.maven.project.interpolation.ModelInterpolator;
import org.apache.maven.project.path.PathTranslator;
@@ -102,6 +82,25 @@
import org.codehaus.plexus.util.StringUtils;
import org.codehaus.plexus.util.xml.pull.XmlPullParserException;
+import java.io.File;
+import java.io.FileNotFoundException;
+import java.io.IOException;
+import java.io.Reader;
+import java.io.StringReader;
+import java.net.URL;
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.Date;
+import java.util.HashMap;
+import java.util.HashSet;
+import java.util.Iterator;
+import java.util.LinkedHashSet;
+import java.util.LinkedList;
+import java.util.List;
+import java.util.Map;
+import java.util.Set;
+import java.util.TreeMap;
+
/*:apt
-----
@@ -173,8 +172,6 @@
private ModelInterpolator modelInterpolator;
- private CoordinateInterpolator coordinateInterpolator;
-
private ArtifactRepositoryFactory artifactRepositoryFactory;
// ----------------------------------------------------------------------
@@ -908,19 +905,6 @@
// Only track the file of a POM in the source tree
project.setFile( projectDescriptor );
-
- try
- {
- coordinateInterpolator.interpolateArtifactCoordinates( project );
- }
- catch ( IOException e )
- {
- throw new ProjectBuildingException( project.getId(), "Failed to write POM with interpolated coordinate expressions.", e );
- }
- catch ( ModelInterpolationException e )
- {
- throw new InvalidProjectModelException( projectId, pomLocation, e.getMessage(), e );
- }
}
project.setManagedVersionMap( createManagedVersionMap( projectId,
Modified: maven/components/branches/maven-2.2.0-RC/maven-project/src/main/resources/META-INF/plexus/components.xml
URL: http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.0-RC/maven-project/src/main/resources/META-INF/plexus/components.xml?rev=783068&r1=783067&r2=783068&view=diff
==============================================================================
--- maven/components/branches/maven-2.2.0-RC/maven-project/src/main/resources/META-INF/plexus/components.xml (original)
+++ maven/components/branches/maven-2.2.0-RC/maven-project/src/main/resources/META-INF/plexus/components.xml Tue Jun 9 17:07:27 2009
@@ -49,21 +49,6 @@
|
-->
<component>
- <role>org.apache.maven.project.interpolation.CoordinateInterpolator</role>
- <role-hint>default</role-hint>
- <implementation>org.apache.maven.project.interpolation.DefaultCoordinateInterpolator</implementation>
- <requirements>
- <requirement>
- <role>org.apache.maven.project.path.PathTranslator</role>
- </requirement>
- </requirements>
- </component>
-<!--
- |
- |
- |
- -->
- <component>
<role>org.apache.maven.project.interpolation.ModelInterpolator</role>
<implementation>org.apache.maven.project.interpolation.StringSearchModelInterpolator</implementation>
<requirements>
@@ -112,9 +97,6 @@
<role>org.apache.maven.project.interpolation.ModelInterpolator</role>
</requirement>
<requirement>
- <role>org.apache.maven.project.interpolation.CoordinateInterpolator</role>
- </requirement>
- <requirement>
<role>org.apache.maven.project.inheritance.ModelInheritanceAssembler</role>
</requirement>
<requirement>
Modified: maven/components/branches/maven-2.2.0-RC/maven-project/src/test/java/org/apache/maven/project/MavenProjectDynamismTest.java
URL: http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.0-RC/maven-project/src/test/java/org/apache/maven/project/MavenProjectDynamismTest.java?rev=783068&r1=783067&r2=783068&view=diff
==============================================================================
--- maven/components/branches/maven-2.2.0-RC/maven-project/src/test/java/org/apache/maven/project/MavenProjectDynamismTest.java (original)
+++ maven/components/branches/maven-2.2.0-RC/maven-project/src/test/java/org/apache/maven/project/MavenProjectDynamismTest.java Tue Jun 9 17:07:27 2009
@@ -32,7 +32,6 @@
import org.apache.maven.model.Dependency;
import org.apache.maven.model.Plugin;
import org.apache.maven.model.Resource;
-import org.apache.maven.project.interpolation.CoordinateInterpolator;
import org.apache.maven.project.interpolation.ModelInterpolationException;
import org.codehaus.plexus.PlexusTestCase;
import org.codehaus.plexus.util.StringUtils;
Modified: maven/components/branches/maven-2.2.0-RC/maven-project/src/test/resources/org/apache/maven/project/ProjectClasspathTest.xml
URL: http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.0-RC/maven-project/src/test/resources/org/apache/maven/project/ProjectClasspathTest.xml?rev=783068&r1=783067&r2=783068&view=diff
==============================================================================
--- maven/components/branches/maven-2.2.0-RC/maven-project/src/test/resources/org/apache/maven/project/ProjectClasspathTest.xml (original)
+++ maven/components/branches/maven-2.2.0-RC/maven-project/src/test/resources/org/apache/maven/project/ProjectClasspathTest.xml Tue Jun 9 17:07:27 2009
@@ -72,9 +72,6 @@
<role-hint>test</role-hint>
</requirement>
<requirement>
- <role>org.apache.maven.project.interpolation.CoordinateInterpolator</role>
- </requirement>
- <requirement>
<role>org.apache.maven.project.interpolation.ModelInterpolator</role>
</requirement>
<requirement>
Modified: maven/components/branches/maven-2.2.0-RC/maven-project/src/test/resources/org/apache/maven/project/canonical/CanonicalProjectBuilderTest.xml
URL: http://svn.apache.org/viewvc/maven/components/branches/maven-2.2.0-RC/maven-project/src/test/resources/org/apache/maven/project/canonical/CanonicalProjectBuilderTest.xml?rev=783068&r1=783067&r2=783068&view=diff
==============================================================================
--- maven/components/branches/maven-2.2.0-RC/maven-project/src/test/resources/org/apache/maven/project/canonical/CanonicalProjectBuilderTest.xml (original)
+++ maven/components/branches/maven-2.2.0-RC/maven-project/src/test/resources/org/apache/maven/project/canonical/CanonicalProjectBuilderTest.xml Tue Jun 9 17:07:27 2009
@@ -72,9 +72,6 @@
<role-hint>test</role-hint>
</requirement>
<requirement>
- <role>org.apache.maven.project.interpolation.CoordinateInterpolator</role>
- </requirement>
- <requirement>
<role>org.apache.maven.project.interpolation.ModelInterpolator</role>
</requirement>
<requirement>