You are viewing a plain text version of this content. The canonical link for it is here.

Posted to gitbox@activemq.apache.org by "mattrpav (via GitHub)" <gi...@apache.org> on 2023/04/04 14:11:58 UTC

[GitHub] [activemq] mattrpav commented on pull request #995: [AMQ-9237] Add META-INF/services to activemq-client-jakarta

mattrpav commented on PR #995:
URL: https://github.com/apache/activemq/pull/995#issuecomment-1496046835

   @cshannon I manually verified the client works for tcp:// and ssl://, and the jar contains the META-INF/services/*.
   
   I agree we need a unit test, but I'm not seeing a quick path to getting a test in-build, since we'd have to implement a new test pattern. The activemq-run maven plugin splits off a thread, not a process on 'fork = true'. 


-- 
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: gitbox-unsubscribe@activemq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org