desruisseaux commented on PR #11505: URL: https://github.com/apache/maven/pull/11505#issuecomment-3699134934
Maybe you can go ahead with phase 2 when you have time. Looking on the wiki page, I suggest the following changes: * Instead of `record ModuleConfig(boolean hasMain, boolean hasTest, boolean hasMainResources, boolean hasTestResources)`, handle any combination of `<lang>` and `<scope>` with something like `Map<Something, Boolean>`. The mapping to legacy `<resources>`, `<testResources>`, etc. would still special cases, but the rest (e.g., generate automatically a `Source` object) can be handled by a universal code. * Remove rule R3, i.e., accept duplicated source for scope='X', lang='Y', module='Z'. The compiler plugin 4 already accepts that, the need for multi-source directories in Maven 3 was strong enough to justify an external plugin, and Maven 3 already accepts that for resources if I remember right. -- 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]
