cstamas commented on code in PR #1604:
URL: https://github.com/apache/maven/pull/1604#discussion_r1671260570


##########
maven-embedder/src/main/java/org/apache/maven/cli/MavenCli.java:
##########
@@ -320,12 +320,14 @@ void initialize(CliRequest cliRequest) throws 
ExitException {
         // We need to locate the top level project which may be pointed at 
using
         // the -f/--file option.  However, the command line isn't parsed yet, 
so
         // we need to iterate through the args to find it and act upon it.
-        Path topDirectory = Paths.get(cliRequest.workingDirectory);
+        Path cwd = Paths.get(cliRequest.workingDirectory);
+        // MavenCliTest extensively uses MULTIMODULE_PROJECT_DIRECTORY
+        Path topDirectory = cliRequest.multiModuleProjectDirectory.toPath();

Review Comment:
   In other words, in "normal" case you'd have full CLI args and all and they 
would be same, they would end up same. It is ONLY the UT, that sets this Java 
System Property for different test 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

Reply via email to