Ethanlm opened a new pull request #3436:
URL: https://github.com/apache/storm/pull/3436


   ## What is the purpose of the change
   
   With log4j version change, the license files need to be updated. The travis 
build is failing due to:
   ```
   DEPENDENCY-LICENSES and target/DEPENDENCY-LICENSES are different. Please 
update DEPENDENCY-LICENSES by running 'mvn 
license:aggregate-add-third-party@generate-and-check-licenses 
-Dlicense.skipAggregateAddThirdParty=false -B' in the project root
   Dependencies in LICENSE-binary that appear unused: 
   org.apache.logging.log4j:log4j-1.2-api:2.11.2
   org.apache.logging.log4j:log4j-api:2.11.2
   org.apache.logging.log4j:log4j-core:2.11.2
   org.apache.logging.log4j:log4j-slf4j-impl:2.11.2
   org.apache.logging.log4j:log4j-web:2.11.2
   Dependencies missing from LICENSE-binary: 
   org.apache.logging.log4j:log4j-1.2-api:2.17.1
   org.apache.logging.log4j:log4j-api:2.17.1
   org.apache.logging.log4j:log4j-core:2.17.1
   org.apache.logging.log4j:log4j-slf4j-impl:2.17.1
   org.apache.logging.log4j:log4j-web:2.17.1
   LICENSE-binary needs to be updated. Please remove any unnecessary 
dependencies from LICENSE-binary, and add any that are missing. You can copy 
any missing dependencies from DEPENDENCY-LICENSES
   Some license files are not up to date, see above for the relevant error 
message
   The command "if [[ $(uname -m) != 's390x' ]]; then /bin/bash 
./dev-tools/travis/travis-script.sh `pwd` $MODULES; fi" exited with 1.
   before_cache
   0.04s$ rm -rf $HOME/.m2/repository/org/apache/storm
   cache.2
   store build cache
   
   ```
   
   ## How was the change tested
   Ran
   ```
   python3 dev-tools/validate-license-files.py --skip-build-storm
   ```
   and passed


-- 
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-unsubscr...@storm.apache.org

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


Reply via email to