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

Posted to dev@storm.apache.org by GitBox <gi...@apache.org> on 2022/04/06 14:27:57 UTC

[GitHub] [storm] pjfanning commented on a diff in pull request #3463: [STORM-3845] Update Active MQ version to 5.17.0

pjfanning commented on code in PR #3463:
URL: https://github.com/apache/storm/pull/3463#discussion_r844019797


##########
DEPENDENCY-LICENSES:
##########
@@ -612,9 +611,9 @@ List of third-party dependencies grouped by their license type.
     Common Development and Distribution License (CDDL) v1.1
 
         * Java Message Service (javax.jms:jms:1.1 - http://java.sun.com/products/jms)
-        * Java Transaction API (javax.transaction:jta:1.1 - http://java.sun.com/products/jta)
         * jsp-api (javax.servlet:jsp-api:2.0 - no url defined)
         * jsp-api (javax.servlet.jsp:jsp-api:2.1 - no url defined)
+        * jta (javax.transaction:jta:1.1 - no url defined)

Review Comment:
   I think this line (an removes line 615) which is likely to be one issue with build



-- 
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: dev-unsubscribe@storm.apache.org

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