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

Reply via email to