[
https://issues.apache.org/jira/browse/MNG-8026?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17810800#comment-17810800
]
ASF GitHub Bot commented on MNG-8026:
-------------------------------------
cstamas commented on code in PR #1391:
URL: https://github.com/apache/maven/pull/1391#discussion_r1466156908
##########
api/maven-api-core/src/main/java/org/apache/maven/api/Artifact.java:
##########
@@ -105,6 +105,14 @@ default String key() {
*/
boolean isSnapshot();
+ /**
+ * The artifact properties.
+ *
+ * @return the artifact properties, never {@code null}
+ */
+ @Nonnull
+ ArtifactProperties getArtifactProperties();
Review Comment:
Or in other words: see this PR
https://github.com/apache/maven-resolver/pull/411
Basically Resolver can be now asked to:
* create Artifact as `new Artifact("g:a:v",
artifactTypeRegistry.get("annotation-processor"))`. (and this ctor is not new,
in a way, you could do the same even with existing ctors, this was just a
shorthand to make possible to use type with string "coords").
* the to resolve it to have it's full transitive hull
* and you again end up with "bunch of Artifacts", but you still want to sift
thru them, filter them by properties
No dependency in scope, but you are STILL interested in Artifact
properties....
Or similarly for ANY artifact, as if artifact you are resolving may have
type="modular-jar", then the needed info will be still present in
artifact.properties. Again, no dependencies involved, etc.
Am pretty much confident that this change is correct.
> Maven should define from ground up scopes used with Resolver
> ------------------------------------------------------------
>
> Key: MNG-8026
> URL: https://issues.apache.org/jira/browse/MNG-8026
> Project: Maven
> Issue Type: Task
> Reporter: Tamas Cservenak
> Priority: Major
> Fix For: 4.0.0, 4.0.0-alpha-13
>
>
> See MRESOLVER-471, it is actually role of Maven (as a resolver consumer
> project) to define all these things:
> * supported scopes
> * transformations of scopes to build path scopes
> * fine tune conflict resolution and related things (in session)
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
