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

Grzegorz Grzybek commented on MRESOLVER-394:
--------------------------------------------

[~cstamas] good points and I fully agree that you have wider view on this than 
me. I was looking only from consistency perspective.

I see that Maven itself creates the session via {{}} only in one place:
{code:java}
org.apache.maven.internal.aether.DefaultRepositorySystemSessionFactory.newRepositorySession()
{code}

this factory is a {{@Named}} bean from maven-core 
(/META-INF/sisu/javax.inject.Named).

So I guess I was too quick and narrow-minded here. Let's close as won't fix. 
Thanks for explanation!

> Provide "static" supplier for RepositorySystemSession
> -----------------------------------------------------
>
>                 Key: MRESOLVER-394
>                 URL: https://issues.apache.org/jira/browse/MRESOLVER-394
>             Project: Maven Resolver
>          Issue Type: Improvement
>          Components: Resolver
>            Reporter: Grzegorz Grzybek
>            Priority: Major
>             Fix For: 1.9.15
>
>
> Similar to MRESOLVER-387, but for DefaultRepositorySystemSession, so we can 
> also deprecate 
> {{org.apache.maven.repository.internal.MavenRepositorySystemUtils.newSession()}}.



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

Reply via email to