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

[Zlib] update zlib version to zlib-1.2.12 #117

Merged
merged 1 commit into from
Mar 31, 2022

Conversation

ThomasCJY
Copy link
Contributor

@ThomasCJY ThomasCJY commented Mar 29, 2022

Issue: #117

According to zlib

"Due to the bug fixes, any installations of 1.2.11 should be replaced with 1.2.12."

@linke-autra
Copy link

LGTM

@comius
Copy link
Collaborator

comius commented Mar 31, 2022

Mirrored the zlib

@comius comius merged commit b828a0a into bazelbuild:master Mar 31, 2022
@Wyverald
Copy link
Member

Wyverald commented Apr 4, 2022

Do we want to do a new release of rules_proto with this included? As of right now, anything that uses an older rules_proto without overriding the zlib dependency will break.

@comius
Copy link
Collaborator

comius commented Apr 4, 2022

Do we want to do a new release of rules_proto with this included? As of right now, anything that uses an older rules_proto without overriding the zlib dependency will break.

I created/tagged 4.0.0-39.19.2-2 release.

@Wyverald
Copy link
Member

Wyverald commented Apr 4, 2022

Umm, is it supposed to be 4.0.0-3.19.2-2 instead? (there's an extra 9 :P)

@comius
Copy link
Collaborator

comius commented Apr 4, 2022

Umm, is it supposed to be 4.0.0-3.19.2-2 instead? (there's an extra 9 :P)

Thanks, fixed :)

tetromino pushed a commit to bazelbuild/stardoc that referenced this pull request Apr 4, 2022
To include bazelbuild/rules_proto#117, without which builds fail due to a missing zlib version.
BenHenning added a commit to oppia/oppia-proto-api that referenced this pull request May 27, 2024
This ensures that an older zlib isn't referenced per
bazelbuild/rules_proto#117.
BenHenning added a commit to oppia/oppia-android that referenced this pull request May 27, 2024
This ensures the fixes explained in
bazelbuild/rules_proto#117 and
bazelbuild/bazel#16192 are included to fix CI
breakages on this branch.
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.

5 participants