Skip to content

Comments

[branch-2.8] Fix license files#15044

Closed
nicoloboschi wants to merge 1 commit intoapache:branch-2.8from
nicoloboschi:branch-2.8-fix-license
Closed

[branch-2.8] Fix license files#15044
nicoloboschi wants to merge 1 commit intoapache:branch-2.8from
nicoloboschi:branch-2.8-fix-license

Conversation

@nicoloboschi
Copy link
Contributor

Motivation

License check fails in branch-2.8

org.yaml-snakeyaml-1.30.jar unaccounted for in LICENSE
org.yaml-snakeyaml-1.27.jar mentioned in LICENSE, but not bundled
snakeyaml-1.30.jar unaccounted for in lib/presto/LICENSE
snakeyaml-1.30.jar unaccounted for in lib/presto/LICENSE
snakeyaml-1.27.jar mentioned in lib/presto/LICENSE, but not bundled

It looks like there are issues with the LICENSE/NOTICE.
Error: Process completed with exit code 2.

Modifications

Update snakeyaml version in license files

  • no-need-doc

@nicoloboschi
Copy link
Contributor Author

/pulsarbot rerun-failure-checks

1 similar comment
@nicoloboschi
Copy link
Contributor Author

/pulsarbot rerun-failure-checks

@nicoloboschi
Copy link
Contributor Author

License check passed. The test failure is caused by #14628 (comment)

better to merge to unblock other pulls against branch-2.8

@codelipenghui @lhotari @michaeljmarshall

@michaeljmarshall
Copy link
Member

@nicoloboschi - I think we should probably just cherry pick #13722 to branch-2.8. @lhotari - is there any reason we shouldn't do that?

@codelipenghui
Copy link
Contributor

@michaeljmarshall I have cherry-picked #13722 into branch-2.8.

@michaeljmarshall
Copy link
Member

Thank you @codelipenghui. Closing this, since the underlying issue is now resolved. @nicoloboschi - is it correct to assume that we're seeing this on branch-2.9 and branch-2.10?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants