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

fix: emit helpful error when native gem fails to load glibc #2156

Merged
merged 1 commit into from Dec 31, 2020

Conversation

flavorjones
Copy link
Member

What problem is this PR intended to solve?

Specifically, older systems may not meet our expectations for glibc,
and this commit attempts to help those poor souls find a path forward.

Closes #2081

Have you included adequate test coverage?

I manually tested this change using the OCI image at:

registry.gitlab.com/gitlab-org/gitlab-omnibus-builder/centos_6:0.0.72

Does this change affect the behavior of either the C or the Java implementations?

No.

Specifically, older systems may not meet our expectations for glibc,
and this commit attempts to help those poor souls find a path forward.

I manually tested this change using the OCI image at:

  registry.gitlab.com/gitlab-org/gitlab-omnibus-builder/centos_6:0.0.72

Closes #2081
@codeclimate
Copy link

codeclimate bot commented Dec 31, 2020

Code Climate has analyzed commit d01d7d9 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 33.3% (80% is the threshold).

This pull request will bring the total coverage in the repository to 94.3%.

View more on Code Climate.

@flavorjones flavorjones merged commit 7f304f5 into master Dec 31, 2020
@flavorjones flavorjones deleted the 2081-sane-error-message-on-old-glibc branch December 31, 2020 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Native gems: centos6 support
1 participant