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

Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2020/06/17 16:27:17 UTC

[GitHub] [beam] tysonjh commented on pull request #12020: [BEAM-9066] Add javax.annotation-api dependency.

tysonjh commented on pull request #12020:
URL: https://github.com/apache/beam/pull/12020#issuecomment-645478796


   > Most important is noting the license, which is the same as JDK stdlib stuff: GPL 2.0 with classpath exception. (even with source retention best to be clear and safe)
   
   I found that the annotation is under both CDDL 1.1 and GPL 2.0 so I updated the comments accordingly. Could you review this please? I'd like to be sure it is correct.
   
   This PR doesn't fix the linked issue but is required AFAICT for building Beam locally with Java 11. I'm not sure exactly how the precommit phase is set up to do this so I need to investigate before merging this.


----------------------------------------------------------------
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:
users@infra.apache.org