Hi Tilmann,

no, the files LICENCE and NOTICE are generated by 'mvn install'. So if we need 
a LICENSE file in the source repository, we haven an issue here.

Regards Michael


>  So I propose to remove the files LICENSE.txt and NOTICE.txt and use
the files as generated by the maven-remote-resources-plugin

Just for my understanding, we would still have LICENSE/NOTICE files in
the repository, right? I think at least the LICENSE(.txt) file is pretty
much mandatory in the repository.

Otherwise that change sounds good, please go ahead and I will merge it
into the RC.

Til




On 5/19/22 23:22, Craig Russell wrote:
Hi Michael,

I'd agree that removing LICENSE.txt and NOTICE.txt from the root and changing 
the <name> in the root pom makes sense.

But let's also make it easier on ourselves by documenting the behavior. Maybe 
adding a paragraph to the README.md that describes that the files are 
automatically created when the top level project is built? And adding a comment 
to the pom that the <name> is used to generate the README and LICENSE files?

Then the question you raise whether we should create a JIRA and add it to 
3.2.1. I'm ok either way. It's not a critical fix but still would be useful to 
include in 3.2.1.

Craig

On May 19, 2022, at 2:10 PM, Michael Bouschen 
<m...@apache.org><mailto:m...@apache.org> wrote:

Hi,
Looked at the unzipped directory for source-release that becomes jdo-3.2.1-RC1.
The NOTICE seems a bit off. It refers to the JDO pom with a copyright.
But the NOTICE.txt is fine, except for the Copyright date which should be 
2005-2022.
Craig pushed a change to main.

I don't think we need LICENSE since we have LICENSE.txt which is fine.

AI Michael: try to figure out where the LICENSE and NOTICE are coming from. 
Perhaps the mvn-notice-plugin?
We added some feature for the Apache Felix bundle plug-in...
I figured out the files LICENSE and NOTICE are generated by the 
maven-remote-resources-plugin which is defined in the Apache parent pom.
The plugin definition contains the following comment for the 
maven-remote-resources-plugin:   <!-- We want to package up license resources 
in the JARs produced -->
It refers to org.apache:apache-jar-resource-bundle:1.4 as resource bundle. If 
you are interested you find it in your local maven repository. Go to 
~/.m2/repository/org/apache/apache-jar-resource-bundle/1.4 and take a look at 
apache-jar-resource-bundle-1.4.jar. The maven-remote-resources-plugin uses 
these files as templates when the plugin is executed.

According to "Assembling LICENSE and NOTICE files" see 
https://infra.apache.org/licensing-howto.html the files should be called 
LICENSE and NOTICE, so without the .txt suffix.

So I propose to remove the files LICENSE.txt and NOTICE.txt and use the files 
as generated by the maven-remote-resources-plugin. In order to get a better 
name into the generated NOTICE file I propose to rename the JDO root pom from 
"JDO Root POM" to "Apache Java Data Objects (JDO)". That means setting the 
<name> element in the JDO root pom: <name>Apache Java Data Objects (JDO)</name>

What do you think? If you agree I would check in the change into the 3.2.1 
branch.

Regards Michael


Craig L Russell
c...@apache.org<mailto:c...@apache.org>



--
Michael Bouschen
akquinet tech@spree GmbH
Bülowstraße 66 • D-10783 Berlin
Tel:   +49 30 235520-33
Fax:  +49 30 217520-12

E-Mail: michael.bousc...@akquinet.de<mailto:michael.bousc...@akquinet.de>
Web:   www.akquinet.de<http://www.akquinet.de/>

Geschäftsführung: Martin Weber, Dr. Torsten Fink, Heinz Wilming
Amtsgericht Berlin HRB 86780 • USt.-Id. Nr.: DE 225 964 680

[Facebook]<http://www.facebook.com/akquinet>  
[XING]<https://www.xing.com/companies/akquinetag>  
[LinkedIn]<https://www.linkedin.com/company/akquinet-ag>  
[Twitter]<https://twitter.com/akquinet>

Reply via email to