Upgrading Mina to 1.1.7 would allow the removal of backport-util-concurrent.jar from the dependencies.
Thanks Emmanuel for the original patch (sorry we didnt get this done earlier), and Keith for the newer one.
Keith's updated patch is now applied. I noticed just after committing it that the pom dependency stubs still needed to be renamed, so I have also done that.
Closing this JIRA out now.
Emmanuel's patch had gone stale over the last 15 months (removal of inVM, pom dep changes etc). I've created a new patch which achieves the same result against the current codebase.
Thank you Robbie. I don't use the patch, I use a trimmed down client that doesn't rely on Mina.
I hope to incorporate this upgrade into some upcoming work I will be doing. Can I ask if you are already using the patched source with this change in place?
I refreshed the patch on Github:
Moving to 0.9, due to insufficient testing
I updated the patch, it's now available as a branch on Github. Please let me know if this format is not suitable.
Is there any chance to get this into the 0.8 release? If someone can review this issue I'll happily update my patch.
Updated patch for the latest trunk
Martin, let me know if you need an updated patch when you are ready to review this change, I'll provide one.
Thanks for the patches I'll take a look at these over the next few days.
Additional patch to update the POMs
Additional patch to account for the latest changes in AddressBasedDestinationTest by rajith (rev 934563)
There are some unused classes in the mina package that could also be removed:
This patch replaces Mina 1.0.1 with Mina 1.1.7. The Queue and BlockingQueue classes from Mina were replaced by their equivalent in java.util.concurrent. VmPipeIdleStatusChecker has been removed since the issue it worked around has been fixed in Mina 1.1.7.
Mina 1.1.7 can be downloaded here :