Hi, I created a jira and PR to cleanup parent pom and one of the submodules: https://github.com/apache/aries/pull/410/files
wt., 28 sty 2025 o 14:42 Jean-Baptiste Onofré <j...@nanthrax.net> napisał(a): > Hi Dominik > > Yeah, it sounds good to me. > > Regards > JB > > On Sun, Jan 26, 2025 at 10:12 PM Dominik Przybysz <alien11...@apache.org> > wrote: > > > > Hi team, > > our parent pom used ianal-maven-plugin which does not have any new > version > > and in other repos was replaced by geronimo tools-maven-plugin. The same > I > > have done in https://github.com/apache/aries/pull/404. > > > > We discussed a little bit with Romain Manni-Bucau in the PR, but I > haven't > > seen other available replacement. > > > > The plugin apache-whisker-maven-plugin [1] seems to be created to solve > the > > problem in the future, but it's not officially released yet. > > > > On the other hand I see that apache parent pom in version 33 [2] adds > > plugin run: > > > > <!-- We want to package up license resources in the JARs produced > --> > > <plugin> > > <groupId>org.apache.maven.plugins</groupId> > > <artifactId>maven-remote-resources-plugin</artifactId> > > <executions> > > <execution> > > <id>process-resource-bundles</id> > > <goals> > > <goal>process</goal> > > </goals> > > <configuration> > > <resourceBundles> > > > > > <resourceBundle>org.apache.apache.resources:apache-jar-resource-bundle:${version.apache-resource-bundles}</resourceBundle> > > </resourceBundles> > > </configuration> > > </execution> > > </executions> > > </plugin> > > > > Maybe it's enough to have the parent pom upgraded in all artifacts and > then > > the necessary legal files are generated automatically. > > > > WDYT? Can we just drop checking legal files and rely on plugin adding > > necessary files? > > > > [1] > > > https://creadur.apache.org/whisker/apache-whisker-maven-plugin/index.html > > [2] > > > https://github.com/apache/maven-apache-parent/blob/apache-33/pom.xml#L361-L379 > > > > > > -- > > Regards, > > Dominik Przybysz > -- Regards, Dominik Przybysz
