Summary: | apache tomcat deployed pom does not contain url or license | ||
---|---|---|---|
Product: | Tomcat 7 | Reporter: | jieryn |
Component: | Packaging | Assignee: | Tomcat Developers Mailing List <dev> |
Status: | RESOLVED FIXED | ||
Severity: | trivial | ||
Priority: | P2 | ||
Version: | 7.0.26 | ||
Target Milestone: | --- | ||
Hardware: | PC | ||
OS: | Linux | ||
Attachments: | 2012-06-24_tc6_53034.patch |
Description
jieryn
2012-04-04 12:35:36 UTC
Fixed (I hope) in trunk (r1335711) and 7.0.x (r1335715) and will be included in 7.0.28 onwards. I recommend checking the updates before the release since it looks right to me based on the Maven docs but I am no Maven expert. It looks correct, thank you Mark! Regarding multiple licenses (AL + CDDL) in tomcat-jsp-api.pom, tomcat-servlet-api.pom: The comment in description of the "licenses" element [1] says that "If multiple licenses are listed, it is assumed that the user can select any of them, not that they must accept all." Actually CDDL applies only to DTD and schema files in those JARs. [1] http://maven.apache.org/xsd/maven-4.0.0.xsd [2] http://maven.apache.org/ref/3.0.4/maven-model/maven.html BTW, I think that the more correct URL would be with a trailing slash, <url>http://tomcat.apache.org/</url> So, how do you suggest we handle this? Merge the two entries into a single AL2+CDDL entry? There is "comments" element in License type. Let's use that to describe what CDDL applies to. Updated in trunk and 7.0.x and will be included in 7.0.28. There wasn't a nice way of doing this. Suggestions for better ways (ideally with a patch those does not involve repackaging the XSDs) welcome. Created attachment 28987 [details]
2012-06-24_tc6_53034.patch
Patch for Tomcat 6.
It is easier here than it was for Tomcat 7, because all files are under Apache License. There is no CDDL.
6.0.x also fixed for 6.0.36 onwards. |