[
https://issues.apache.org/jira/browse/MRESOLVER-494?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17816562#comment-17816562
]
ASF GitHub Bot commented on MRESOLVER-494:
------------------------------------------
cstamas commented on code in PR #428:
URL: https://github.com/apache/maven-resolver/pull/428#discussion_r1485956826
##########
maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/collect/DependencyCollectorDelegate.java:
##########
@@ -394,8 +393,8 @@ protected VersionRangeResult cachedResolveRangeResult(
return rangeResult;
}
- protected static boolean isLackingDescriptor(Artifact artifact) {
- return artifact.getProperty(ArtifactProperties.LOCAL_PATH, null) !=
null;
+ protected static boolean isLackingDescriptor(RepositorySystemSession
session, Artifact artifact) {
+ return
session.getSystemScopeHandler().getSystemPath(artifact.getProperties()) != null;
Review Comment:
If you look at "system" scope definition, there is this sentence "Dependency
in this scope does not have artifact descriptor either.". And this here is the
check "does it have descriptor". And since Resolver 1.0 the actual check is
done like this: `artifact.getProperty( ArtifactProperties.LOCAL_PATH, null ) !=
null` (presence of property and not actual scope).
> LOCAL_PATH Artifact property really belongs to "system" scope (or is at least
> very related to it)
> -------------------------------------------------------------------------------------------------
>
> Key: MRESOLVER-494
> URL: https://issues.apache.org/jira/browse/MRESOLVER-494
> Project: Maven Resolver
> Issue Type: Improvement
> Components: Resolver
> Reporter: Tamas Cservenak
> Assignee: Tamas Cservenak
> Priority: Major
> Fix For: 2.0.0-alpha-8, 2.0.0
>
>
> LOCAL_PATH Artifact property really belongs to "system" scope (or is at least
> very related to it).
> It may need to be removed as well?
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
