For reference, the NOTICE file template is here: http://svn.apache.org/repos/asf/maven/resources/trunk/apache-jar-resource-bundle/src/main/resources/META-INF/NOTICE.vm
Maybe it should be enhanced to support additional fields? Thomas On Thu, Oct 22, 2015 at 7:42 PM, Thomas Weise <[email protected]> wrote: > Hi Justin, > > Once build there are several NOTICE files that do not match the top level >> one. e.g. ./api/target/classes/META-INF/NOTICE. The notice in the final jar >> under META-INF/NOTICE is incorrect. You may want to change NOTICE say >> "Apache Apex (incubating)” rather than just “Apache Apex”. >> > > The NOTICE files are added to the .jar files by a plugin that is setup in > the Apache POM. They come from org.apache:apache-jar-resource-bundle:1.4 > > I see examples of not matching the top level NOTICE elsewhere where this > POM is used. I also see other releases with .jar artifacts that have no > NOTICE file in it. > > The text in the non matching files is added by the plugin by substituting > the Maven module name. So for example, the name element in api/pom.xml > should be changed from "Apache Apex API" to "Apache Apex (incubating) API". > > What is your recommendation, same NOTICE file in all .jar artifacts or > generated NOTICE file with (changed) name of module? > > >> The url in the pom.xml is incorrect, it should be >> http://apex.incubator.apache.org, I’d also change the name to "Apache >> Apex (incubating)”. >> <name>Apache Apex</name> >> <url>http://apex.apache.org</url> >> >> > Will fix that. > > >> When checking what tagged in version control and what in the source >> release I noticed: >> - DEPENDANCIES file isn’t checked into version control >> > > Is it recommended to have a top level DEPENDENCIES file in source control? > > > - dt-app-archetype and dt-conf-archtype directories are not in version >> control and are empty. Are these needed? >> >> > Residual from build, will ensure they don't get packaged. > > >> Finally is there a guide to making a release that any committer could >> follow? >> > > I took notes, will cleanup and publish along with release. > > https://malhar.atlassian.net/browse/APEX-216 > > Thanks, > Thomas > >
