Skip to content

Update the SPDX ID of license #220

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Goooler
Copy link
Member

@Goooler Goooler commented Jun 3, 2025

Seems the current used The Apache Software License, Version 2.0 is outdated, and it can't be recognized by Maven Central.

See https://central.sonatype.com/artifact/org.apache.ant/ant/1.10.15

image

Seems the current used `The Apache Software License, Version 2.0` is outdated, and it can't be recognized by Maven Central.

See https://central.sonatype.com/artifact/org.apache.ant/ant/1.10.15
@paulk-asert
Copy link

paulk-asert commented Jun 3, 2025

I am not sure it is the text in that field that is the problem. I actually just don't see the licenses or license tags appear at all in the ant pom. Perhaps those need to be added to the file here:

https://github.com/apache/ant/blob/master/src/etc/poms/ant/pom.xml

The parent pom seems to show the current wording still:
Screenshot 2025-06-03 at 9 00 24 pm

Or maybe the semantics are that those fields should be "inherited" from the parent pom but that website doesn't do that.

@Goooler
Copy link
Member Author

Goooler commented Jun 3, 2025

Seems license nodes are missing for all POM under https://github.com/apache/ant/tree/master/src/etc/poms, I'm not sure if we should append for them or not...

@bodewig
Copy link
Member

bodewig commented Jun 4, 2025

I would expect the licenses to be inherited from https://github.com/apache/ant/blob/master/src/etc/poms/pom.xml

The license does show up with mvn help:effective-pom.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants