[jira] [Comment Edited] (MNG-7563) REGRESSION: User properties now override model properties in dependencies

["Michael Osipov (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/MNG-7563?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17648035#comment-17648035 ] 

Michael Osipov edited comment on MNG-7563 at 12/15/22 12:50 PM:
----------------------------------------------------------------

You brought up very reasonable points and the discussion was truly professional. Therefore, I had to agree with -- especially because It was my fault that I did not analyze the impact good enough. Would you be willing to create an IT which depicts your usecase to make sure that a further change won't break it? I would highly appreciate it.


was (Author: michael-o):
You brought up very reasonable points and the dicussion was truly professional. Therefore, I had to agree anyway with especially because my fault was that I did not analyze the impact good enough. Would you be willing to create an IT which depicts your usecase to make sure that a further change won't break it? I would highly appreciate it.

> REGRESSION: User properties now override model properties in dependencies
> -------------------------------------------------------------------------
>
>                 Key: MNG-7563
>                 URL: https://issues.apache.org/jira/browse/MNG-7563
>             Project: Maven
>          Issue Type: Bug
>          Components: Dependencies, POM
>    Affects Versions: 3.8.5, 3.8.6
>            Reporter: Hervé Guillemet
>            Assignee: Michael Osipov
>            Priority: Major
>             Fix For: 3.8.x-candidate, 3.9.0-candidate, 4.0.x-candidate
>
>         Attachments: poms.zip
>
>
> An important change has been introduced in 3.8.5 that breaks some existing builds: Java system properties now take precedence over default values of user properties in dependency POMs. This look like a bug since it's now easy to affect dependency behaviors with system properties, a practice that has been discouraged. But maybe do you consider this as a new feature ?
> As an example, 3 poms are attached to this ticket.
> After installing projects b and c, building project a with:
> {{mvn package -Ddep=x}}
> used to succeed until 3.8.4 (-D is ignored) but throws error with 3.8.5 and 3.8.6 (-D override the default).
> Note that without the setting of the default value for property {{dep}} in project b, the build fails with any version of Maven.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)