Martin Cooper wrote:
Since every component needs to do this, would it work to add this to
commons-build/project.xml, so that it's included as part of the
<extend>?

That would make sense, but IIRC, we decided some time back to move away from POM inheritance for configuring j-c components, using commons-build just to drive the web site generation stuff and manually including the common stuff in either project.xml or maven.xml for each component. I may have this wrong. Maybe Mark or someone more maven-knowledgeable can comment on the best way to accomplish this.


One more thing that I noticed is that unless you also add
<include>LICENSE.txt</include> below, the ant-generated build.xml does not include LICENSE.txt in the jar. Unfortunately, when I add that to project.xml, I end up with two copies of LICENSE.txt in maven-generated jars. I think that this is because the maven jar plugin puts LICENSE.txt in automatically; but the ant plugin does not generate code to add it. Also, the maven jar task creates a new copy (so has a different data), causing the dup in the jar. Could be this qualifies as a bug in the ant plugin. Anyone have any ideas of how to get around this? As of this moment, build.xml for [math] includes an extra line not generated by the ant plugin against the current version of project.xml.


Phil

-- Martin Cooper


On 13 Dec 2004 04:07:18 -0000, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote:

psteitz     2004/12/12 20:07:18

Modified:    math     project.xml
Log:
Get NOTICE.txt included in META-INF of distribution jars.

Revision  Changes    Path
1.56      +10 -1     jakarta-commons/math/project.xml

Index: project.xml
===================================================================
RCS file: /home/cvs/jakarta-commons/math/project.xml,v
retrieving revision 1.55
retrieving revision 1.56
diff -u -r1.55 -r1.56
--- project.xml       10 Dec 2004 05:12:52 -0000      1.55
+++ project.xml       13 Dec 2004 04:07:17 -0000      1.56
@@ -164,6 +164,15 @@
         </resource>
       </resources>
     </unitTest>
+    <resources>
+      <resource>
+        <directory>${basedir}</directory>
+        <includes>
+          <include>NOTICE.txt</include>
+        </includes>
+        <targetPath>META-INF</targetPath>
+      </resource>
+    </resources>
   </build>
   <reports>
     <report>maven-changes-plugin</report>

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




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



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

Reply via email to