This should be done now with the entries being “commons.module.name” Cheers, -Rob
> On May 24, 2017, at 11:31 AM, Rob Tompkins <chtom...@gmail.com> wrote: > > >> On May 24, 2017, at 11:11 AM, Stephen Colebourne <scolebou...@joda.org> >> wrote: >> >> On 24 May 2017 at 15:55, Rob Tompkins <chtom...@gmail.com> wrote: >>> We should simply add that entry, "commons.automatic-module-name," to every >>> component pom’s properties section now, and then when the next parent >>> migration happens, the changes will express naturally. It might be worth >>> adding a comment on the property in each pom? >>> >>> I’d be happy to do that between now and Monday. >> >> As I said upthread, there is an argument to only add it to components >> once they have been checked to see if they are valid for use as a >> module. > > Right. > >> >> That said, I'm willing to go with it as an approach because AFAICT if >> a component isn't a good modular citizen, the Automatic-Module-Name >> MANIFEST entry won't do much harm. > > Yes. > >> >> Of course, strictly speaking we don't know if Automatic-Module-Name >> will be part of the final JDK 9, as private discussions are currently >> ongoing between the key players. Since it will cause no harm if >> wrongly present, I'm OK with this too, >> >> If you are going to do it, I'd suggest using ${commons.module-name}, > > Makes sense to me there. I’m not the best at coming up with names. :-) > >> as you will be adding the official module name for the component. That >> it is only used as the automatic module name right now is a detail. > > I will start chipping away at this tomorrow or Friday, assuming that there > aren’t any objections between now and then. > > -Rob > >> >> Stephen >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org >> For additional commands, e-mail: dev-h...@commons.apache.org >> > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org