BLasan commented on pull request #1597: URL: https://github.com/apache/fineract/pull/1597#issuecomment-781375531
> Hi - instead of excluding this from the check, you really should look to exclude the duplicate inclusion of the same dependency. It only makes sense to remove it from the check if there is no other way to avoid it - eg if two different jars, which are both required, contain the same class. But that's not the case here, so excluding one of the dependencies would be the right way forward IMHO. Tried to avoid having the duplicated jar. But if we could only add com/sun/activation/resources/ as a dependency we would be able to run the tests successfully. But I couldn't find a solution for that. If you could point me where the `com.sun.mail.javax.mail` dependency gets installed, I would be able to come with some good solution. I didn't find any imports for javax mail. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
