commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Phil Steitz" <>
Subject RE: [all] Including license files (was Re: cvs commit: jakarta-commons/math project.xml)
Date Mon, 13 Dec 2004 18:46:31 GMT
Apologies to all if my reply to Martin's post sent earlier this AM eventually arrives, but
there is a wrinkle that I need to mention regarding the commit below.  While this does work
for maven;  unfortunately the build.xml generated by maven's ant goal does not include script
to add LICENSE.txt.  If I add an include for LICENSE.txt below, it generates a good build.xml,
but causes maven's jar goal to add two copies of LICENSE.txt (with different dates).   So,
for now at least, the build.xml that I committed last night for [math] does not correspond
to what maven generates (I added LICENSE.txt manually).  I posted a query on this to the maven
user list.

	-----Original Message----- 
	From: Mark R. Diggory [] 
	Sent: Mon 12/13/2004 11:26 AM 
	To: Jakarta Commons Developers List 
	Subject: Re: [all] Including license files (was Re: cvs commit: jakarta-commons/math project.xml)

	Martin Cooper wrote:
	 > Um, a quick grep tells me that Attributes, Chain and Resources all have:
	 >   <extend>../commons-build/project.xml</extend>
	Thats incorrect for resources, in the cvs it doesn't extend:
	I thought I removed those from Attributes and Chain too, I must not have
	committed them. Everything is on my machine at home, I'll make sure they
	get in later today once I get there.
	> and around 30 components in the Sandbox have the Sandbox equivalent.
	> So many components are in fact still doing that. If they're not
	> supposed to, what changes do we need to make to their project.xml
	> files?
	They shouldn't be extending if possible. This change would be good to do
	to all of them, it also means that some site information needs to be
	present in the project.xml "template" and placed into all those project
	xmls in the sandbox too.
	is a good example of that content.
	I know it is a replication of content to place this in all project.xml
	files, but its really not much information, and it can already be
	overiden by each component.
	On a side note, I think we are abusing the gumpRepository ID by making
	it "jakarta" for all the components.
	We really can't rely on pom inheritance because it creates headaches
	when building the distributions. It much easier if every component only
	has dependencies required for compilation, not for generation of
	> TIA.
	> --
	> Martin Cooper
	> On Mon, 13 Dec 2004 09:42:22 -0500, Mark R. Diggory
	> <> wrote:
	>>There actually isn't an extend. All projects are dependent only on
	>>commons-site.jsl and the menu includes. There actually isn't any
	>>inheritance of the project.xml because there were issues with users
	>>building distributions with maven when the dependency (commons-build)
	>>wasn't present there.
	>>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
	>>>Martin Cooper
	>>>On 13 Dec 2004 04:07:18 -0000, <>
	>>>>psteitz     2004/12/12 20:07:18
	>>>>Modified:    math     project.xml
	>>>>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:
	>>>>For additional commands, e-mail:
	>>>To unsubscribe, e-mail:
	>>>For additional commands, e-mail:
	>>To unsubscribe, e-mail:
	>>For additional commands, e-mail:
	Mark Diggory
	Software Developer
	Harvard MIT Data Center
	To unsubscribe, e-mail:
	For additional commands, e-mail:

View raw message