earldouglas commented on PR #991: URL: https://github.com/apache/tomcat/pull/991#issuecomment-4244675777
> the definition of the `<licenses>` element in a POM is that it defines a choice from multiple licenses I'm not sure that's the case. My read of <https://maven.apache.org/pom.html#Licenses> is that there should be one entry for each license that applies to the project. In this case, two licenses always apply, and so both should get its own entry. Combining them into a single entry is misleading, since both the `name` and `url` are invalid. This breaks principle of least surprise for the user, as they are forced to manually read the invalid entry to interpret it. This also breaks standards, which can prevent the use of this library and its POM as a dependency. > Tomcat 7.0.x is unsupported Would this PR be merged into a supported branch? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
