Author: evenisse
Date: Wed Apr 18 02:55:47 2007
New Revision: 529940

URL: http://svn.apache.org/viewvc?view=rev&rev=529940
Log:
[MRELEASE-122] Allow extensions without defined version

Added:
    
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-development/pom-without-extension-version/
    
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-development/pom-without-extension-version/expected-pom.xml
   (with props)
    
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-development/pom-without-extension-version/pom.xml
   (with props)
    
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-without-extension-version/
    
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-without-extension-version/expected-pom.xml
   (with props)
    
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-without-extension-version/pom.xml
   (with props)
Modified:
    
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java
    
maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractRewritingReleasePhaseTestCase.java

Modified: 
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java
URL: 
http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java?view=diff&rev=529940&r1=529939&r2=529940
==============================================================================
--- 
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java
 (original)
+++ 
maven/release/trunk/maven-release-manager/src/main/java/org/apache/maven/shared/release/phase/AbstractRewritePomsPhase.java
 Wed Apr 18 02:55:47 2007
@@ -372,9 +372,12 @@
             {
                 Extension extension = (Extension) i.next();
 
-                updateDomVersion( extension.getGroupId(), 
extension.getArtifactId(), mappedVersions,
-                                  resolvedSnapshotDependencies, 
extension.getVersion(), originalVersions, "extensions",
-                                  "extension", extensionRoot, projectId, 
properties, result );
+                if ( extension.getVersion() != null )
+                {
+                    updateDomVersion( extension.getGroupId(), 
extension.getArtifactId(), mappedVersions,
+                                      resolvedSnapshotDependencies, 
extension.getVersion(), originalVersions,
+                                      "extensions", "extension", 
extensionRoot, projectId, properties, result );
+                }
             }
         }
     }

Modified: 
maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractRewritingReleasePhaseTestCase.java
URL: 
http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractRewritingReleasePhaseTestCase.java?view=diff&rev=529940&r1=529939&r2=529940
==============================================================================
--- 
maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractRewritingReleasePhaseTestCase.java
 (original)
+++ 
maven/release/trunk/maven-release-manager/src/test/java/org/apache/maven/shared/release/phase/AbstractRewritingReleasePhaseTestCase.java
 Wed Apr 18 02:55:47 2007
@@ -371,6 +371,17 @@
         assertTrue( compareFiles( reactorProjects ) );
     }
 
+    public void testRewritePomExtensionUndefinedVersion()
+        throws Exception
+    {
+        List reactorProjects = createReactorProjects( 
"pom-without-extension-version" );
+        ReleaseDescriptor config = createDefaultConfiguration( reactorProjects 
);
+
+        phase.execute( config, null, reactorProjects );
+
+        assertTrue( compareFiles( reactorProjects ) );
+    }
+
     public void testRewriteBasicPomWithEditMode()
         throws Exception
     {

Added: 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-development/pom-without-extension-version/expected-pom.xml
URL: 
http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-development/pom-without-extension-version/expected-pom.xml?view=auto&rev=529940
==============================================================================
--- 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-development/pom-without-extension-version/expected-pom.xml
 (added)
+++ 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-development/pom-without-extension-version/expected-pom.xml
 Wed Apr 18 02:55:47 2007
@@ -0,0 +1,35 @@
+<!--
+  ~ 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>
+  <modelVersion>4.0.0</modelVersion>
+
+  <groupId>groupId</groupId>
+  <artifactId>artifactId</artifactId>
+  <version>1.1-SNAPSHOT</version>
+
+  <build>
+    <extensions>
+      <extension>
+        <groupId>groupId</groupId>
+        <artifactId>something</artifactId>
+      </extension>
+    </extensions>
+  </build>
+</project>

Propchange: 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-development/pom-without-extension-version/expected-pom.xml
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-development/pom-without-extension-version/expected-pom.xml
------------------------------------------------------------------------------
    svn:keywords = "Author Date Id Revision"

Added: 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-development/pom-without-extension-version/pom.xml
URL: 
http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-development/pom-without-extension-version/pom.xml?view=auto&rev=529940
==============================================================================
--- 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-development/pom-without-extension-version/pom.xml
 (added)
+++ 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-development/pom-without-extension-version/pom.xml
 Wed Apr 18 02:55:47 2007
@@ -0,0 +1,35 @@
+<!--
+  ~ 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>
+  <modelVersion>4.0.0</modelVersion>
+
+  <groupId>groupId</groupId>
+  <artifactId>artifactId</artifactId>
+  <version>1.0</version>
+
+  <build>
+    <extensions>
+      <extension>
+        <groupId>groupId</groupId>
+        <artifactId>something</artifactId>
+      </extension>
+    </extensions>
+  </build>
+</project>

Propchange: 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-development/pom-without-extension-version/pom.xml
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-development/pom-without-extension-version/pom.xml
------------------------------------------------------------------------------
    svn:keywords = "Author Date Id Revision"

Added: 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-without-extension-version/expected-pom.xml
URL: 
http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-without-extension-version/expected-pom.xml?view=auto&rev=529940
==============================================================================
--- 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-without-extension-version/expected-pom.xml
 (added)
+++ 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-without-extension-version/expected-pom.xml
 Wed Apr 18 02:55:47 2007
@@ -0,0 +1,35 @@
+<!--
+  ~ 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>
+  <modelVersion>4.0.0</modelVersion>
+
+  <groupId>groupId</groupId>
+  <artifactId>artifactId</artifactId>
+  <version>1.0</version>
+
+  <build>
+    <extensions>
+      <extension>
+        <groupId>groupId</groupId>
+        <artifactId>something</artifactId>
+      </extension>
+    </extensions>
+  </build>
+</project>

Propchange: 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-without-extension-version/expected-pom.xml
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-without-extension-version/expected-pom.xml
------------------------------------------------------------------------------
    svn:keywords = "Author Date Id Revision"

Added: 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-without-extension-version/pom.xml
URL: 
http://svn.apache.org/viewvc/maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-without-extension-version/pom.xml?view=auto&rev=529940
==============================================================================
--- 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-without-extension-version/pom.xml
 (added)
+++ 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-without-extension-version/pom.xml
 Wed Apr 18 02:55:47 2007
@@ -0,0 +1,35 @@
+<!--
+  ~ 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>
+  <modelVersion>4.0.0</modelVersion>
+
+  <groupId>groupId</groupId>
+  <artifactId>artifactId</artifactId>
+  <version>1.0-SNAPSHOT</version>
+
+  <build>
+    <extensions>
+      <extension>
+        <groupId>groupId</groupId>
+        <artifactId>something</artifactId>
+      </extension>
+    </extensions>
+  </build>
+</project>

Propchange: 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-without-extension-version/pom.xml
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: 
maven/release/trunk/maven-release-manager/src/test/resources/projects/rewrite-for-release/pom-without-extension-version/pom.xml
------------------------------------------------------------------------------
    svn:keywords = "Author Date Id Revision"


Reply via email to