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.

Sign up for GitHub

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

Jump to bottom

fix build #8162

Merged
merged 2 commits into from
Dec 13, 2019
Merged

fix build #8162

merged 2 commits into from
Dec 13, 2019

Conversation

Vxider
Copy link
Contributor

@Vxider Vxider commented Dec 12, 2019

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

fix build on gcc 8.2.1

@akuzm
Copy link
Contributor

akuzm commented Dec 12, 2019

Oops, seems like we don't have a gcc 8 build in CI. I'll check if we can add it.

@akuzm akuzm merged commit 142932e into ClickHouse:master Dec 13, 2019
@akuzm
Copy link
Contributor

akuzm commented Dec 13, 2019

Oops, seems like we don't have a gcc 8 build in CI. I'll check if we can add it.

We won't add it now because the CI is overloaded, maybe later if it breaks too often.

@KochetovNicolai KochetovNicolai added the pr-build Pull request with build/testing/packaging improvement label Dec 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
pr-build Pull request with build/testing/packaging improvement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@Vxider @akuzm @KochetovNicolai