rmannibucau commented on pull request #665: URL: https://github.com/apache/maven/pull/665#issuecomment-1020942506
@laeubi well there are two comments to this point: 1. it does not justify to use a new way to interpolate values, once again I think we should tend to stick to our own convention and usage and not multiply syntaxes for the same purpose 2. the extensions are loaded from the clirequest after the env/system props so still the standard interpolation can be used even if part of the project is not available (worse case you just need to adapt `PluginParameterExpressionEvaluator` to ignore the session related variables). I still think trying to read the raw root pom in a fast pass to extract the properties if present since it will be available in most cases and it would enable to use defaults there but not a blocker to me to not have defaults there since `.mvn/jvm.config` enables to set them if needed in all cases. -- 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
