On 2 January 2017 at 18:49, Michael Osipov <micha...@apache.org> wrote:

> Am 2017-01-01 um 15:51 schrieb Stephen Connolly:
>
>> On 1 January 2017 at 00:55, Michael Osipov <micha...@apache.org> wrote:
>>
>> I just went through the list my issues. Here is a safe list I would
>>> merge/cherry-pick into new master:
>>>
>>> FIX-3.5.0:
>>> MNG-5567,
>>>
>>
>>
>> Affects behaviour, I recommend 3.5.1 but I will not object if others feel
>> 3.5.0
>>
>
> This indeed changes behavior but behavioral changes should not be in a
> patch version like 3.5.1, but in a minor. Moreover, Java supports ZIP files
> first-class citizens, we simply failed/forgot this here.
>

Well that is why I see this as a patch fix. Changing behaviour where the
original behaviour was a bug or a regression does not mean we have to bump
a minor version.

I think this and the plugin classpath building *bugs/regressions* are both
perfectly valid to fix in a patch.

>
> I still think that 3.5.0 is appropriate.


I still think that it raises the risk of marking the "no-op" change from
aether to resolver as breaking existing builds.

My plan is 3.5.0 in the next two weeks and 3.5.1, etc at a 6 week cadence
thereafter. If you want I am happy to do 3.5.0 and 3.5.1 in more rapid
sequence, but I really would like to keep dependency and classpath changes
out of 3.5.0 and push them for 3.5.1 (where they are bugs / regressions) or
3.6.0 (which should be as soon as we think we are ready, not a year or
two's time


>
>
> MNG-5954,
>>>
>>
>>
>> I'd second this for 3.5.1 but I'm not objecting if others feel it is low
>> risk to drop-in replacement with 3.3.9
>>
>
> Do not share, it is never read. I even asked on dev list and Jason
> replied, it is safe to be nuked.


If somebody else seconds, then it is agreed (unless we have an objector)...
just I'm not going to second it for 3.5.0


>
>
>
>> MNG-6029,
>>>
>>
>>
>> I'd second this for 3.5.1
>> I have concerns with introducing for 3.5.0 as it affects how the classpath
>> gets built and could cause behaviour differences between 3.3.9 and 3.5.0
>> (as it causes the test classpath to actually get resolved)
>>
>> -1 for 3.5.0 but I am open to change my position
>>
>
> It is actually two-fold, duplicate code and wrong scope. Fix has been
> there for seven months without causing any IT failures. I think is is a
> safe bet for an RC.


But it breaks my goal for 3.5.0... the duplicate code part is OK for
3.5.0... but the fixing the scope bug is why I remain -1. Perfectly fine
for 3.5.1


>
>
> MNG-6102,
>>>
>>
>>
>> I'd second this for 3.5.1 but I'm not objecting if others feel it is low
>> risk to drop-in replacement with 3.3.9
>>
>
> This is change in behavior and not subject to a bugfix. The default
> behaves like before.


This does not affect dependency / classpath resolution, but we are growing
changes for 3.5.0 and I would like to try and keep the non
s/aether/resolver/g changes to a minimum. If you can find another seconder
it's in.


>
>
> MNG-6106,
>>
>>
>> I'd second this for 3.5.1 but I'm not objecting if others feel it is low
>> risk to drop-in replacement with 3.3.9
>>
>
> This is actually a bug because maven.home can never be user.home/.m2



This does not affect dependency / classpath resolution, but we are growing
changes for 3.5.0 and I would like to try and keep the non
s/aether/resolver/g changes to a minimum. If you can find another seconder
it's in.


>
>
> MNG-6115,
>>>
>>
>>
>> We need to decide on colourised logs for 3.5.0 or 3.5.1
>>
>
> Colorize is a new feature, not a bug fix. Either 3.5 or 3.6. It could also
> be squashed into MNG-3507.


It's a new feature for logging, but it doesn't add new APIs to the *core*
of maven. So I think we could put it in 3.5.1 if we decided our version
policy was "effects on plugin API / dependency resolution"

The squashed diff should be quite small, so I am fine with us adding it for
3.5.0... I just am not stepping up to sponsor it


>
>
> MNG-6136,
>>>
>>
>>
>> I'd second this for 3.5.1 but I'm not objecting if others feel it is low
>> risk to drop-in replacement with 3.3.9
>>
>> MNG-6137,
>>>
>>
>>
>> I'd second this for 3.5.1 but I'm not objecting if others feel it is low
>> risk to drop-in replacement with 3.3.9
>>
>
> Rather 3.5 or 3.6, consider that the last Wagon release is quite old and
> currently, there duplicate dependencies on Maven's classpath as layed out
> in MNG-6137.


I don't see this being something that needs to wait for 3.6.0... 3.5.1
seems perfectly ok to me

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

Reply via email to