Skip to content
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.

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

[MRESOLVER-471] Resolver should be oblivious about scopes. #412

Merged
merged 3 commits into from
Jan 24, 2024

Conversation

cstamas
Copy link
Member

@cstamas cstamas commented Jan 24, 2024

In Resolver 1.x it defined scopes (that were redefined by consumer project Maven), leading to duplication and possible digression if Maven was about to support new dependency scopes. Moreover, Resolver 1.x also contained (based on these scopes) logic, "scope refinement", "scope derive to buildpath scope" and so on. All these logic just like the definition of scopes belong to consumer project: Maven.

Resolver really cares about only one scope: "system", as dependencies in this scope should be handled very differently than others (have no POMs and just file presence should be checked on resolving them).


https://issues.apache.org/jira/browse/MRESOLVER-471

In Resolver 1.x it defined scopes (that were redefined by consumer
project Maven), leading to duplication and possible digression if
Maven was about to support new dependency scopes. Moreover, Resolver
1.x also contained (based on these scopes) logic, "scope refinement",
"scope derive to buildpath scope" and so on. All these logic just
like the definition of scopes belong to consumer project: Maven.

Resolver really cares about only one scope: "system", as dependencies
in this scope should be handled very differently than others (have
no POMs and just file presence should be checked on resolving them).

---

https://issues.apache.org/jira/browse/MRESOLVER-471
@cstamas cstamas self-assigned this Jan 24, 2024
This class will need to be moved to Maven maven-resolver-provider.
@cstamas cstamas merged commit 5c4f150 into apache:master Jan 24, 2024
4 checks passed
@cstamas cstamas deleted the MRESOLVER-471 branch January 24, 2024 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant