michael-o commented on PR #235:
URL: 
https://github.com/apache/maven-invoker-plugin/pull/235#issuecomment-2092931481

   > not sure I understand this naming change, At the end of the day, those 
properties will be system properties used to fork the Maven invocation. So they 
are the system properties of the Maven invocation you want to run. you are 
confusing with Maven core naming but for an end user POV they are system 
properties. Those one you set using `-D...` when using the command line.
   
   This has been a mess created long time ago. We need to clean up. Maven 
should use for itself user props only. If the JVM requires sys props they need 
to be set *before* Maven invocation. `System.setProperty()` cannot guarantee 
that the actual property is picked up by the receiver. Some read only at 
startup.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to