On 09/11/2018 19:45, James Perkins wrote:
There is some logic to attempt to load the module, if it succeeds, then the dependency is added as a required one. Could this be replaced by an optional dependency and no module loading? These modules would have to to be added thanks to addAdditionalOptionalPackages.The main reason those modules are dependencies of the logging subsystem is because they are not really required for the subsystem itself with the exception of org.jboss.logging. Those module dependencies are added to deployments for convenience. So really they're not required at all unless the user is using one of those logging frameworks and they want the logging subsystem to control the logging output.
This may be somewhat of an edge case because if the user doesn't use, say slf4j, then there is no real reason to include the dependency unless of course another module uses it. I do wonder if we should remove the org.jboss.logging.jul-to-slf4j-stub module and just put the library as a resource root to org.slf4j, but that's slightly OT :)
James R. PerkinsJBoss by Red Hat
_______________________________________________ wildfly-dev mailing list firstname.lastname@example.org https://lists.jboss.org/mailman/listinfo/wildfly-dev
wildfly-dev mailing list