slawekjaranowski commented on code in PR #1595:
URL: https://github.com/apache/maven/pull/1595#discussion_r1720812179
##########
maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java:
##########
@@ -1624,20 +1635,65 @@ static void populateProperties(
String mavenBuildVersion =
CLIReportingUtils.createMavenVersionString(buildProperties);
systemProperties.setProperty("maven.build.version", mavenBuildVersion);
- BasicInterpolator interpolator =
- createInterpolator(paths, systemProperties, userProperties,
userSpecifiedProperties);
- for (Map.Entry<Object, Object> e : userSpecifiedProperties.entrySet())
{
- String name = (String) e.getKey();
- String value = interpolator.interpolate((String) e.getValue());
- userProperties.setProperty(name, value);
- //
----------------------------------------------------------------------
- // I'm leaving the setting of system properties here as not to
break
- // the SystemPropertyProfileActivator. This won't harm embedding.
jvz.
- //
----------------------------------------------------------------------
- if (System.getProperty(name) == null) {
- System.setProperty(name, value);
- }
+ //
----------------------------------------------------------------------
+ // Options that are set on the command line become system properties
+ // and therefore are set in the session properties. System properties
+ // are most dominant.
+ //
----------------------------------------------------------------------
+
+ Properties userSpecifiedProperties =
+
commandLine.getOptionProperties(String.valueOf(CLIManager.SET_USER_PROPERTY));
+ userProperties.putAll(userSpecifiedProperties);
+
+ //
----------------------------------------------------------------------
+ // Load config files
+ //
----------------------------------------------------------------------
+ Function<String, String> callback =
+ or(paths::getProperty, prefix("cli.",
commandLine::getOptionValue), systemProperties::getProperty);
Review Comment:
Properties for testing:
```
${includes}=default.properties,env-${envName}.properties
```
command:
```
mvn -v -DenvName=test
```
fail with:
```
java.nio.file.NoSuchFileException:
..../repos/test/test/.mvn/env-${envName}.properties
```
--
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
