[
https://issues.apache.org/jira/browse/MNG-7556?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17613974#comment-17613974
]
ASF GitHub Bot commented on MNG-7556:
-------------------------------------
michael-o commented on code in PR #810:
URL: https://github.com/apache/maven/pull/810#discussion_r989865835
##########
maven-core/src/main/java/org/apache/maven/internal/aether/DefaultRepositorySystemSessionFactory.java:
##########
@@ -288,14 +289,15 @@ else if ( !MAVEN_RESOLVER_TRANSPORT_AUTO.equals(
transport ) )
+ MAVEN_RESOLVER_TRANSPORT_NATIVE + ", " +
MAVEN_RESOLVER_TRANSPORT_AUTO );
}
- session.setTransferListener( request.getTransferListener() );
-
- session.setRepositoryListener( eventSpyDispatcher.chainListener( new
LoggingRepositoryListener( logger ) ) );
-
+ // TODO This duplicates
org.apache.maven.repository.internal.MavenRepositorySystemUtils.newSession()
Review Comment:
@gnodet pushed a potential cleanup in
ee91544d7e2ff4bdf96075e12da3a1d1abd10d28. In fact, the previous approach did
even ignore env vars, FWIW.
> Clean up notion between user properties and system properties
> -------------------------------------------------------------
>
> Key: MNG-7556
> URL: https://issues.apache.org/jira/browse/MNG-7556
> Project: Maven
> Issue Type: Task
> Reporter: Michael Osipov
> Assignee: Michael Osipov
> Priority: Major
> Fix For: 3.9.0-candidate, 4.0.0-alpha-1, 4.0.0
>
>
> For a very long time we have documented that the user can set system
> properties via {{mvn -Dfoo=bar}}, but actually those are user properties
> which are promoted to system properties and it some cases system properties
> cannot be modified *after* the JVM has been started. To properly set system
> properties there are basically two ways:
> * use {{MAVEN_OPTS}} environment variable
> * use {{.mvn/jvm.config}} file
> A third option in the future we could introduce, like other Java tools, a
> {{-J-Dfoo=bar}} option with the restriction of the the paragraph above.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
