gemmellr commented on PR #4567: URL: https://github.com/apache/activemq-artemis/pull/4567#issuecomment-1651509389
I hadn't even thought of that problem until now, re: the new build failures. By just putting the log annotation processor in the processorPath for every module it then also applies to those modules built before the processor module, such as the unit test support module, which the processor module itself depends on for tests, thus creating a cycle. The build then only partly works at all due to it having downloaded stale snapshot output from a _previous_ build and seemingly satisfy the cycle that exists. Even that ugliness will presumably fail for release builds or the first post-release builds though, when no prior deployed release or snapshot will exist yet for it to download. Even if its possible to work around that with yet more configuration/complexity, again just leaving this as it was seems preferable to me. -- 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]
