Author: aheritier
Date: Mon Sep 19 09:37:34 2005
New Revision: 290197

URL: http://svn.apache.org/viewcvs?rev=290197&view=rev
Log:
improve naming consistency : rename plugin:available by 
assert:assertPluginAvailable

Modified:
    maven/maven-1/plugins/trunk/checkstyle/plugin.jelly
    maven/maven-1/plugins/trunk/dist/plugin.jelly
    maven/maven-1/plugins/trunk/faq/plugin.jelly
    maven/maven-1/plugins/trunk/jar/plugin.jelly
    maven/maven-1/plugins/trunk/jdiff/plugin.jelly
    maven/maven-1/plugins/trunk/plugin/plugin.jelly
    maven/maven-1/plugins/trunk/plugin/xdocs/changes.xml
    maven/maven-1/plugins/trunk/plugin/xdocs/tags.xml

Modified: maven/maven-1/plugins/trunk/checkstyle/plugin.jelly
URL: 
http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/checkstyle/plugin.jelly?rev=290197&r1=290196&r2=290197&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/checkstyle/plugin.jelly (original)
+++ maven/maven-1/plugins/trunk/checkstyle/plugin.jelly Mon Sep 19 09:37:34 2005
@@ -28,15 +28,15 @@
   xmlns:ant="jelly:ant"
   xmlns:util="jelly:util"
   xmlns:doc="doc"
-  xmlns:plugin="plugin"
+  xmlns:assert="assert"
   xmlns:maven="jelly:maven"
   xmlns:define="jelly:define"
   xmlns:checkstyle="checkstyle">
 
   <j:if test="${bootstrapping == null}">
-    <!-- fake test because the plugin:available tag is avalaible in the 
maven-plugin-plugin 1.7 and newer. Thus maven will break if it's not present. 
-->  
-    <plugin:available groupId="maven" artifactId="maven-plugin-plugin" 
minRelease="1.7" neededBy="maven-faq-plugin"/>
-    <plugin:available groupId="maven" artifactId="maven-xdoc-plugin" 
minRelease="1.10" neededBy="maven-faq-plugin"/>
+    <!-- fake test because the assert:assertPluginAvailable tag is avalaible 
in the maven-plugin-plugin 1.7 and newer. Thus maven will break if it's not 
present. -->  
+    <assert:assertPluginAvailable groupId="maven" 
artifactId="maven-plugin-plugin" minRelease="1.7" 
neededBy="${plugin.artifactId}"/>
+    <assert:assertPluginAvailable groupId="maven" 
artifactId="maven-xdoc-plugin" minRelease="1.10" 
neededBy="${plugin.artifactId}"/>
   </j:if>
 
   <!--

Modified: maven/maven-1/plugins/trunk/dist/plugin.jelly
URL: 
http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/dist/plugin.jelly?rev=290197&r1=290196&r2=290197&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/dist/plugin.jelly (original)
+++ maven/maven-1/plugins/trunk/dist/plugin.jelly Mon Sep 19 09:37:34 2005
@@ -23,11 +23,13 @@
   xmlns:ant="jelly:ant"
   xmlns:maven="jelly:maven"
   xmlns:artifact="artifact"
-  xmlns:plugin="plugin"
+  xmlns:assert="assert"
   xmlns:util="jelly:util">
 
   <j:if test="${bootstrapping == null}">
-    <plugin:available groupId="maven" artifactId="maven-artifact-plugin" 
minRelease="1.3" neededBy="maven-dist-plugin"/>
+    <!-- fake test because the assert:assertPluginAvailable tag is avalaible 
in the maven-plugin-plugin 1.7 and newer. Thus maven will break if it's not 
present. -->  
+    <assert:assertPluginAvailable groupId="maven" 
artifactId="maven-plugin-plugin" minRelease="1.7" 
neededBy="${plugin.artifactId}"/>
+    <assert:assertPluginAvailable groupId="maven" 
artifactId="maven-artifact-plugin" minRelease="1.3" 
neededBy="${plugin.artifactId}"/>
   </j:if>
 
   <j:new var="distTypeHandler" 
className="org.apache.maven.dist.DistributionArtifactTypeHandler" />

Modified: maven/maven-1/plugins/trunk/faq/plugin.jelly
URL: 
http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/faq/plugin.jelly?rev=290197&r1=290196&r2=290197&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/faq/plugin.jelly (original)
+++ maven/maven-1/plugins/trunk/faq/plugin.jelly Mon Sep 19 09:37:34 2005
@@ -26,13 +26,13 @@
   xmlns:ant="jelly:ant"
   xmlns:define="jelly:define"
   xmlns:doc="doc"
-  xmlns:plugin="plugin"
+  xmlns:assert="assert"
   xmlns:faq="faq">
 
   <j:if test="${bootstrapping == null}">
-    <!-- fake test because the plugin:available tag is avalaible in the 
maven-plugin-plugin 1.7 and newer. Thus maven will break if it's not present. 
-->  
-    <plugin:available groupId="maven" artifactId="maven-plugin-plugin" 
minRelease="1.7" neededBy="maven-faq-plugin"/>
-    <plugin:available groupId="maven" artifactId="maven-xdoc-plugin" 
minRelease="1.9.2" neededBy="maven-faq-plugin"/>
+    <!-- fake test because the assert:assertPluginAvailable tag is avalaible 
in the maven-plugin-plugin 1.7 and newer. Thus maven will break if it's not 
present. -->  
+    <assert:assertPluginAvailable groupId="maven" 
artifactId="maven-plugin-plugin" minRelease="1.7" 
neededBy="${plugin.artifactId}"/>
+    <assert:assertPluginAvailable groupId="maven" 
artifactId="maven-xdoc-plugin" minRelease="1.9.2" 
neededBy="${plugin.artifactId}"/>
   </j:if>
 
   <define:taglib uri="faq">

Modified: maven/maven-1/plugins/trunk/jar/plugin.jelly
URL: 
http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/jar/plugin.jelly?rev=290197&r1=290196&r2=290197&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/jar/plugin.jelly (original)
+++ maven/maven-1/plugins/trunk/jar/plugin.jelly Mon Sep 19 09:37:34 2005
@@ -24,12 +24,14 @@
   xmlns:license="license"
   xmlns:maven="jelly:maven"
   xmlns:artifact="artifact"
-  xmlns:plugin="plugin"
+  xmlns:assert="assert"
   xmlns:util="jelly:util"
   xmlns:doc="doc">
 
   <j:if test="${bootstrapping == null}">
-    <plugin:available groupId="maven" artifactId="maven-artifact-plugin" 
minRelease="1.3" neededBy="maven-jar-plugin"/>
+    <!-- fake test because the assert:assertPluginAvailable tag is avalaible 
in the maven-plugin-plugin 1.7 and newer. Thus maven will break if it's not 
present. -->  
+    <assert:assertPluginAvailable groupId="maven" 
artifactId="maven-plugin-plugin" minRelease="1.7" 
neededBy="${plugin.artifactId}"/>
+    <assert:assertPluginAvailable groupId="maven" 
artifactId="maven-artifact-plugin" minRelease="1.3" 
neededBy="${plugin.artifactId}"/>
   </j:if>
 
   <!-- ================================================================== -->

Modified: maven/maven-1/plugins/trunk/jdiff/plugin.jelly
URL: 
http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/jdiff/plugin.jelly?rev=290197&r1=290196&r2=290197&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/jdiff/plugin.jelly (original)
+++ maven/maven-1/plugins/trunk/jdiff/plugin.jelly Mon Sep 19 09:37:34 2005
@@ -23,11 +23,13 @@
   xmlns:maven="jelly:maven"
   xmlns:doc="doc"
   xmlns:scm="scm"
-  xmlns:plugin="plugin"
+  xmlns:assert="assert"
   xmlns:ant="jelly:ant">
 
   <j:if test="${bootstrapping == null}">
-    <plugin:available groupId="maven" artifactId="maven-scm-plugin" 
minRelease="1.5" neededBy="maven-jdiff-plugin"/>
+    <!-- fake test because the assert:assertPluginAvailable tag is avalaible 
in the maven-plugin-plugin 1.7 and newer. Thus maven will break if it's not 
present. -->  
+    <assert:assertPluginAvailable groupId="maven" 
artifactId="maven-plugin-plugin" minRelease="1.7" 
neededBy="${plugin.artifactId}"/>
+    <assert:assertPluginAvailable groupId="maven" 
artifactId="maven-scm-plugin" minRelease="1.5" neededBy="${plugin.artifactId}"/>
   </j:if>
 
   <goal name="maven-jdiff-plugin:register">

Modified: maven/maven-1/plugins/trunk/plugin/plugin.jelly
URL: 
http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/plugin/plugin.jelly?rev=290197&r1=290196&r2=290197&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/plugin/plugin.jelly (original)
+++ maven/maven-1/plugins/trunk/plugin/plugin.jelly Mon Sep 19 09:37:34 2005
@@ -55,9 +55,47 @@
           <ant:include name="**/.processed" />
         </ant:fileset>
       </ant:delete>
-    </define:tag>
+    </define:tag>    
+    
+  </define:taglib>
     
-  <define:tag name="available">
+  <define:taglib uri="assert">
+    <define:tag name="assertFileExists">
+      <!-- @file : Full file path -->
+      <!-- @msg  : optional message to be displayed -->
+      <u:file var="fileAsFile" name="${file}"/>
+      <j:if test="${!(fileAsFile.exists())}">
+        <ant:fail>${file} does not exist.${msg}</ant:fail>
+      </j:if>      
+    </define:tag>
+    <define:tag name="assertFileContains">
+      <!-- @file : Full file path -->
+      <!-- @match: the matching value expected  -->
+      <!-- @msg  : optional message to be displayed -->
+      <assert:assertFileExists file="${file}" msg="${msg}"/>
+      <u:loadText var="fileContent" file="${file}"/>
+      <j:if test="${fileContent.indexOf(match) lt 0}">
+        <ant:fail>${file} does not contain string [${match}].${msg}</ant:fail>
+      </j:if>
+    </define:tag>
+    <define:tag name="assertFileNotFound">
+      <!-- @file : Full file path -->
+      <!-- @msg  : optional message to be displayed -->
+      <u:file var="fileAsFile" name="${file}"/>
+      <j:if test="${fileAsFile.exists()}">
+        <ant:fail>${file} found and not expected.${msg}</ant:fail>
+      </j:if>      
+    </define:tag>
+    <define:tag name="assertEquals">
+      <!-- @expected  : the expected value -->
+      <!-- @value     : the actual value   -->
+      <!-- @msg       : optional message to be displayed -->
+      <j:if test="${not(expected.equals(value))}">
+        <ant:fail>Expected [${expected}] but got [${value}].${msg}</ant:fail>
+      </j:if>
+    </define:tag>
+
+  <define:tag name="assertPluginAvailable">
     <!-- Check parameters -->
     <maven:param-check value="${groupId}" fail="true" message="'groupId' must 
be specified"/>
     <maven:param-check value="${artifactId}" fail="true" message="'artifactId' 
must be specified"/>
@@ -157,49 +195,12 @@
 ===============================================================================
       </ant:fail>
     </j:if>
-  </define:tag>    
-    
-  </define:taglib>
-    
-  <define:taglib uri="assert">
-    <define:tag name="assertFileExists">
-      <!-- @file : Full file path -->
-      <!-- @msg  : optional message to be displayed -->
-      <u:file var="fileAsFile" name="${file}"/>
-      <j:if test="${!(fileAsFile.exists())}">
-        <ant:fail>${file} does not exist.${msg}</ant:fail>
-      </j:if>      
-    </define:tag>
-    <define:tag name="assertFileContains">
-      <!-- @file : Full file path -->
-      <!-- @match: the matching value expected  -->
-      <!-- @msg  : optional message to be displayed -->
-      <assert:assertFileExists file="${file}" msg="${msg}"/>
-      <u:loadText var="fileContent" file="${file}"/>
-      <j:if test="${fileContent.indexOf(match) lt 0}">
-        <ant:fail>${file} does not contain string [${match}].${msg}</ant:fail>
-      </j:if>
-    </define:tag>
-    <define:tag name="assertFileNotFound">
-      <!-- @file : Full file path -->
-      <!-- @msg  : optional message to be displayed -->
-      <u:file var="fileAsFile" name="${file}"/>
-      <j:if test="${fileAsFile.exists()}">
-        <ant:fail>${file} found and not expected.${msg}</ant:fail>
-      </j:if>      
-    </define:tag>
-    <define:tag name="assertEquals">
-      <!-- @expected  : the expected value -->
-      <!-- @value     : the actual value   -->
-      <!-- @msg       : optional message to be displayed -->
-      <j:if test="${not(expected.equals(value))}">
-        <ant:fail>Expected [${expected}] but got [${value}].${msg}</ant:fail>
-      </j:if>
-    </define:tag>
+  </define:tag>
+
   </define:taglib>
   
   <j:if test="${bootstrapping == null}">
-    <plugin:available groupId="maven" artifactId="maven-artifact-plugin" 
minRelease="1.3" neededBy="maven-plugin-plugin"/>
+    <assert:assertPluginAvailable groupId="maven" 
artifactId="maven-artifact-plugin" minRelease="1.3" 
neededBy="${plugin.artifactId}"/>
   </j:if>
 
   <goal name="plugin" prereqs="plugin:plugin" description="Build a plugin jar" 
/>

Modified: maven/maven-1/plugins/trunk/plugin/xdocs/changes.xml
URL: 
http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/plugin/xdocs/changes.xml?rev=290197&r1=290196&r2=290197&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/plugin/xdocs/changes.xml (original)
+++ maven/maven-1/plugins/trunk/plugin/xdocs/changes.xml Mon Sep 19 09:37:34 
2005
@@ -24,7 +24,7 @@
   </properties>
   <body>
     <release version="1.7-SNAPSHOT" date="In SVN">
-      <action dev="aheritier" type="add">new plugin:available tag to check if 
a minimal release of a plugin is present.</action>
+      <action dev="aheritier" type="add">new assert:assertPluginAvailable tag 
to check if a minimal release of a plugin is present.</action>
     </release>
     <release version="1.6" date="2005-06-03">
       <action dev="vmassol" type="add">Added new

Modified: maven/maven-1/plugins/trunk/plugin/xdocs/tags.xml
URL: 
http://svn.apache.org/viewcvs/maven/maven-1/plugins/trunk/plugin/xdocs/tags.xml?rev=290197&r1=290196&r2=290197&view=diff
==============================================================================
--- maven/maven-1/plugins/trunk/plugin/xdocs/tags.xml (original)
+++ maven/maven-1/plugins/trunk/plugin/xdocs/tags.xml Mon Sep 19 09:37:34 2005
@@ -31,7 +31,6 @@
           <ol>
             <li><a href='#uninstall_Tag'>uninstall</a></li>
             <li><a href='#clearCache_Tag'>clearCache</a></li>
-            <li><a href='#available_Tag'>available</a></li>
           </ol>
         </li>
         <li><a href='#assert_Tag_Library'>assert</a>
@@ -40,6 +39,7 @@
             <li><a href='#assertFileContains_Tag'>assertFileContains</a></li>
             <li><a href='#assertFileNotFound_Tag'>assertFileNotFound</a></li>
             <li><a href='#assertEquals_Tag'>assertEquals</a></li>
+            <li><a href='#assertPluginAvailable_Tag'>available</a></li>
           </ol>
         </li>
       </ol>
@@ -71,32 +71,6 @@
         </p>
         <p>There are no attributes for this tag</p>
       </subsection>
-      <subsection name='available Tag'>
-        <p>Fails the build if the given plugin does not exist or hasn't the 
minimal release number. Otherwise no effect.</p>
-        <table>
-          <tr><th>Attribute</th><th>Optional?</th><th>Description</th></tr>
-          <tr>
-            <td>groupId</td>
-            <td>No</td>
-            <td>The plugin's groupId</td>
-          </tr>
-          <tr>
-            <td>artifactId</td>
-            <td>No</td>
-            <td>The plugin's artifactId</td>
-          </tr>
-          <tr>
-            <td>minRelease</td>
-            <td>No</td>
-            <td>The minimum plugin's release</td>
-          </tr>
-          <tr>
-            <td>neededBy</td>
-            <td>No</td>
-            <td>The name of the plugin or project using this tag. It is used 
when the tag displays the error for the user.</td>
-          </tr>
-        </table>
-      </subsection>
     </section>
     <section name='assert Tag Library'>
       <p>
@@ -174,6 +148,32 @@
             <td>msg</td>
             <td>Yes</td>
             <td>A message to be displayed when the values aren't equal.</td>
+          </tr>
+        </table>
+      </subsection>
+      <subsection name='assertPluginAvailable Tag'>
+        <p>Fails the build if the given plugin does not exist or hasn't the 
minimal release number. Otherwise no effect.</p>
+        <table>
+          <tr><th>Attribute</th><th>Optional?</th><th>Description</th></tr>
+          <tr>
+            <td>groupId</td>
+            <td>No</td>
+            <td>The plugin's groupId</td>
+          </tr>
+          <tr>
+            <td>artifactId</td>
+            <td>No</td>
+            <td>The plugin's artifactId</td>
+          </tr>
+          <tr>
+            <td>minRelease</td>
+            <td>No</td>
+            <td>The minimum plugin's release</td>
+          </tr>
+          <tr>
+            <td>neededBy</td>
+            <td>No</td>
+            <td>The name of the plugin or project using this tag. It is used 
when the tag displays the error for the user.</td>
           </tr>
         </table>
       </subsection>



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to