pjfanning commented on PR #3469: URL: https://github.com/apache/storm/pull/3469#issuecomment-1090316813
> > this duplicates #3461 and that PR built ok > > Thanks. Merged PR#3461. Looks like mvn command does not generate an acceptable DEPENDENCY-LICENSES file. I normally end up modifying the python that generates the license and get it to print the right licenses as part of the CI build. And then modify the PR with the right values and remove the python change. Far from ideal. It would be nicer if the CI build just produced the DEPENDENCY-LICENSES license file itself instead of expecting developers to keep it updated. -- 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
