sultan commented on code in PR #953:
URL: https://github.com/apache/tomee/pull/953#discussion_r1016687887


##########
pom.xml:
##########
@@ -102,28 +101,16 @@
     <version.jakartaee-api>9.1-M2</version.jakartaee-api>
     <version.tomee-patch-plugin>0.9</version.tomee-patch-plugin>
 
-    <!--
-      JavaMail is both API and IMPL so we don't have them in the jakartaee-api 
uber jar.
-      We decided to add them here in the project so we can patch/update the 2 
libraries without having

Review Comment:
   we should not rely on the API from Impl but the API from Jakarta API which 
has both Jakarta Mail API and Jakarta Activation API available, i believe we 
should take the Impl from geronimo not the API.
   This is something i itented to change on another step. might i try on this 
PR ?



-- 
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]

Reply via email to