gnodet commented on code in PR #1597:
URL: https://github.com/apache/maven/pull/1597#discussion_r1670863134
##########
apache-maven/src/assembly/maven/conf/settings.xml:
##########
@@ -29,14 +29,14 @@ under the License.
|
| -s /path/to/user/settings.xml
|
- | 2. Global Level. This settings.xml file provides configuration for all
Maven
+ | 2. System Level. This settings.xml file provides configuration for all
Maven
Review Comment:
One possibility would be to remove the explicit levels and just use list of
files.
For example:
```
maven.install.conf = ${maven.conf} # this property should not be
defined
maven.user.conf = ${user.home}/.m2
maven.project.conf = ${session.rootDirectory}/.mvn
# Toolchains
maven.install.toolchains = ${maven.install.conf}/toolchains.xml
maven.project.toolchains = ${maven.project.conf}/toolchains.xml
maven.user.toolchains = ${maven.user.conf}/toolchains.xml
maven.toolchains = ?${maven.install.toolchains}, \
?${maven.project.toolchains}, \
?${maven.user.toolchains}
# Extensions
maven.install.extensions = ${maven.install.conf}/extensions.xml
maven.project.extensions = ${maven.project.conf}/extensions.xml
maven.user.extensions = ${maven.user.conf}/extensions.xml
maven.extensions = ?${maven.install.extensions}, \
?${maven.project.extensions}, \
?${maven.user.extensions}
```
This would solve the problem and even allow the user to switch precedence
very easily, as the list order would be the one used.
The `-gt` flag would set override the `maven.install.toolchains`.
--
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
