ptlrs opened a new pull request, #10880:
URL: https://github.com/apache/ozone/pull/10880

   ## What changes were proposed in this pull request?
   The run configurations in Intellij are invalid. This can be seen from the 
little red cross marks on the run configurations. The error says that the class 
in the run configuration doesn't exist in the class path.
   
   Since Intellij 2025.1 release, the way Intellij processes pom files has 
changed. It creates separate .main and .test modules in Maven projects.
   
   See 
https://youtrack.jetbrains.com/projects/IDEA/issues/IDEA-371535/Confusing-separate-.main-and-.test-modules-in-Maven-projects
   
   If there is a difference between the main and test configurations, then 
Intellij will create separate main and test modules. As such the default 
classpath won't work. The run configurations are pointing to classes which are 
not in the default classpath but instead in the classpath with the .main 
suffix. 
   
   This PR adds a `default-testCompile` for the test sections to bring them on 
par with the main configurations. After this Intellij doesn't split the module 
into `main` and `test` and the existing run configurations become valid again. 
   
   ## What is the link to the Apache JIRA
   https://issues.apache.org/jira/browse/HDDS-16001
   
   
   ## How was this patch tested?
   Tested locally 
   CI: https://github.com/ptlrs/ozone/actions/runs/30307554649


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to