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. 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. 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}, 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. Stephen --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org