Skip to content

[MNG-8712] dependency version is a requirement, not effective #2279

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
hboutemy merged 4 commits into from
May 1, 2025
Merged

[MNG-8712] dependency version is a requirement, not effective #2279

Changes from 3 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
570c584
[MNG-8712] dependency version is a requirement, not effective
hboutemy Apr 28, 2025
2c218e5
fix reported issues
hboutemy Apr 29, 2025
647f2d5
simplification
hboutemy Apr 29, 2025
86341cb
version range may break predictability
hboutemy May 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions maven-model/src/main/mdo/maven.mdo
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1087,8 +1087,10 @@
<version>3.0.0+</version>
<description>
<![CDATA[
The version of the dependency, e.g. <code>3.2.1</code>. In Maven 2, this can also be
specified as a range of versions.
The version requirement of the dependency, e.g. <code>3.2.1</code>. The actual version will be resolved based on the usage context.
Version requirement can also be specified as a range of versions, e.g. <code>[3.2.0,)</code>. This is discouraged as it breaks <i>predictability</i> of resolved version.
Copy link
Copy Markdown
Member

@cstamas cstamas Apr 30, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Version requirement can also be specified as a range of versions, e.g. <code>[3.2.0,)</code>. This is discouraged as it breaks <i>predictability</i> of resolved version.
Version requirement can also be specified as a range of versions, e.g. <code>[3.2.0,)</code>. User have to be aware that range versions may break <i>predictability</i> of resolved version in some cases.

Copy link
Copy Markdown
Member Author

@hboutemy hboutemy May 1, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

too long: going back to "as it may break predictability"

See <a href="https://s.apache.org/dependency-version">dependency version requirement documentation</a>
and <a href="https://s.apache.org/transitive-dependencies-resolution">transitive dependencies resolution</a> for more details.
]]>
</description>
<type>String</type>
Expand Down