gzm55 commented on pull request #28:
URL: https://github.com/apache/maven-wrapper/pull/28#issuecomment-1067678139
function `mavenUserHome()` now has two usages:
1. in MavenWrapperMain.java, `private static void addSystemProperties( Path
rootDir )`. in the function wrapper loads two property files into system. the
files are {maven.user.home}/maven.properties and {basedir}/maven.properties.
This logic would come from a
[document](https://github.com/apache/maven/blob/master/maven-core/src/site/apt/configuration-management.apt)
for maven develop team, and maven 2.x, 3.x, 4.x do not load these two path. To
avoid `mvn ...` and `mvnw ...` running with different system properties,
`addSystemProperties()` should not be called within wrapper.
2. pass `mavenUserHome()` to `PathAssembler.mavenUserHome` which will only
affect `PathAssembler.getBaseDir()`. (PS: `getBaseDir()` return working dir
when user pass "PROJECT" to `getBaseDir()`, it should be a bug). This base dir
is used as a storage base for distribution and unzip files.
Then i would prefer add a property `maven.wrapper.repo.local` default to
~/.m2.
--
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
