Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(deps): add (and bump) cross deps versions to .requirements #13117

Merged
merged 1 commit into from
Jul 5, 2024

Conversation

bungle
Copy link
Member

@bungle bungle commented May 30, 2024

Summary

This commit moves the versions and SHA256 digests to .requirements.

It also bumps version of libzlib from 1.2.13 to 1.3.1, and libxcrypt from 4.4.27 to 4.4.36.

KAG-4615
KAG-4616

Checklist

  • The Pull Request has tests
  • A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
  • There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

@github-actions github-actions bot added build/bazel cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee labels May 30, 2024
@bungle
Copy link
Member Author

bungle commented May 30, 2024

It is not obvious to me where these are even used, I mean are those cross_deps even built?

@bungle
Copy link
Member Author

bungle commented May 30, 2024

@fffonion do you know do we need these (e.g. libxcrypt?), or can these build/cross_deps be removed? Are these used on release builds or something?

@bungle bungle requested a review from fffonion May 30, 2024 14:35
### Summary

The cross deps versions placed directly to repective dep `repositories.bzl`.

This commit moves the versions and SHA256 digests to `.requirerements`.

It also bumps version of `libzlib` from `1.2.13` to `1.3.1`, and `libxcrypt`
from `4.4.27` to `4.4.36`.

Signed-off-by: Aapo Talvensaari <aapo.talvensaari@gmail.com>
@bungle bungle merged commit 681d2a7 into master Jul 5, 2024
26 checks passed
@bungle bungle deleted the chore/bump-cross-deps branch July 5, 2024 12:14
@team-gateway-bot
Copy link
Collaborator

Successfully created cherry-pick PR for master:

@fffonion
Copy link
Contributor

fffonion commented Jul 8, 2024

we shouldn't bump the version of those dependencies, they should remain/align with the oldest distros we ship. they are not bundled in the binary package, but only to provide headers and .so to make compiler and linker happy in cross compilation.

@fffonion
Copy link
Contributor

fffonion commented Jul 8, 2024

#13345

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build/bazel cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants