Hi Rob,

> Am 05.06.2017 um 15:50 schrieb Rob Tompkins <chtom...@gmail.com>:
> 
> 
>> On Jun 5, 2017, at 4:34 AM, Benedikt Ritter <brit...@apache.org> wrote:
>> 
>> Hi,
>> 
>>> Am 03.06.2017 um 18:54 schrieb Rob Tompkins <chtom...@gmail.com 
>>> <mailto:chtom...@gmail.com>>:
>>> 
>>> This should be done now with the entries being “commons.module.name”
>> 
>> I’d recommend using dashes in the second part of the name, since my 
>> understanding of points is to declare name spaces. So I’d suggest to use 
>> commons.automatic-module-name and not commons.automatic.module.name.
> 
> I’m ok with re-namespacing. I’ll try to get to that after I push out the file 
> upload 1.3.3 release.

Please make sure that this actually works and generates the desired MANIFEST 
entry. As I’ve said in my comment to one of the commits, I don’t understand who 
this is supposed to work without changing and releasing parent pom.

Cheers,
Benedikt

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