ascheman commented on issue #11612: URL: https://github.com/apache/maven/issues/11612#issuecomment-3858456199
> If you're not including the files and they're present, that should be a hard fail. If you are including them it's OK to warn, or maybe not even that. OK, then I'll change the AC8 the following way, which makes AC7 invalid then: **AC8 (Legacy configuration error)**: When a modular project has legacy sources or resources that **exist but would be ignored**, the build must **fail with an ERROR** (not just a WARNING). This includes: - Explicit `<sourceDirectory>`/`<testSourceDirectory>` differing from defaults - Default `src/main/java` or `src/test/java` directories existing on filesystem - Explicit `<resources>`/`<testResources>` differing from Super POM defaults This prevents silent loss of user-configured sources/resources. **Note: AC8 supersedes AC7.** AC7 originally specified WARNING for ignored legacy directories. AC8 upgrades this to ERROR. Tests and comments referencing AC7 should be updated to mention that AC8 supersedes AC7 (not just replace AC7 with AC8 - preserve the history that AC7 existed but is no longer valid). -- 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]
