gzm55 commented on pull request #28: URL: https://github.com/apache/maven-wrapper/pull/28#issuecomment-1067071379
> The "naming" of maven.user.home might or might not be correct, that could be debated, it was borrowed from [gradle](https://docs.gradle.org/current/userguide/directory_layout.html#dir:gradle_user_home) and the previous wrapper was already using it. yes, that's the gradle pattern, not maven. In maven, some paths are fixed or only configure via cli options, such as ~/.m2/{settings.xml, toolchainx.xml}, $basedir/.mvn/{extensions.xml, maven.config, jvm.config}. whereas `maven.repo.local` is a configurable location, normally set via ~/.m2/settings.xml or the well-defined property. That is, the configuration locations are almost fixed, and data storage are configurable. To follow this convention, how about provide a property `maven.distro.local`, default to `${user.home}/.m2/wrapper/dists`? -- 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
