[GitHub] [geode] rhoughton-pivot commented on pull request #5291: [DRAFT] Pull up transitive dependency to clean up javax.activation and mail licenses.

2020-07-20 Thread GitBox 


rhoughton-pivot commented on pull request #5291:
URL: https://github.com/apache/geode/pull/5291#issuecomment-661343930


   @metatype The module that pulls those in was also getting the symbols 
transitively. I'm not convinced that anything we can call can hit any symbols 
from these particular `javax` jars. If this change to drop the duplication of 
license in our source-license file doesn't work, can we close this PR as 
will-not-fix?



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

[GitHub] [geode] rhoughton-pivot commented on pull request #5291: [DRAFT] Pull up transitive dependency to clean up javax.activation and mail licenses.

2020-07-06 Thread GitBox 


rhoughton-pivot commented on pull request #5291:
URL: https://github.com/apache/geode/pull/5291#issuecomment-654502849


   @jinmeiliao @onichols-pivotal I have not updated the source-distribution 
license file as part of the draft. I'd be happy to pair with someone on how to 
do that, once I get signoff that this is a working dependency change for us.



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

[GitHub] [geode] rhoughton-pivot commented on pull request #5291: [DRAFT] Pull up transitive dependency to clean up javax.activation and mail licenses.

2020-06-23 Thread GitBox 


rhoughton-pivot commented on pull request #5291:
URL: https://github.com/apache/geode/pull/5291#issuecomment-648487629


   Tried to get Spring to handle this, but they punted: 
https://github.com/spring-projects/spring-security/issues/8733



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