> On Jun 6, 2017, at 7:48 AM, Benedikt Ritter <brit...@apache.org> wrote:
> 
> 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.

I was just trying to get ahead of the implied release of the parent Pom. I 
agree that they do nothing until the consuming component up versions into the 
new parent. Maybe that's too much pre-usefulness work?

> 
> 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
> 

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to