ARROW-8786: [Packaging][rpm] Use bundled zstd in the CentOS 8 build #7166
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@github-actions crossbow submit centos-8-amd64 |
|
Revision: fbdebb9 Submitted crossbow builds: ursa-labs/crossbow @ actions-250
|
|
It seems that zstd is added to the BaseOS repository and removed from EPEL: But CentOS 8 that includes zstd isn't released yet. We'll be able to revert this change with the next CentOS 8 release. |
|
We also need to disable |
|
@github-actions crossbow submit centos-8-* |
|
Revision: e4ecf59 Submitted crossbow builds: ursa-labs/crossbow @ actions-253
|
kszucs
commented
May 13, 2020
| @@ -219,7 +222,9 @@ Requires: brotli | |||
| Requires: gflags | |||
| Requires: glog | |||
| %endif | |||
| %if %{have_zstd} | |||
|
Thanks Kou! |
kszucs
added a commit
that referenced
this pull request
May 14, 2020
There was an update in the epel-release repository in the last 12 hours, since then yum/dnf is unable to find libzstd-devel. libzstd is added to the BaseOS repository and removed from EPEL: https://src.fedoraproject.org/rpms/zstd/c/2eaa0955a5176319f2a59294ee339cb2fff87bc5?branch=epel8 But CentOS 8 that includes zstd isn't released yet. We'll be able to revert this change with the next CentOS 8 release. Closes #7166 from kszucs/ARROW-8786 Lead-authored-by: Krisztián Szűcs <szucs.krisztian@gmail.com> Co-authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was an update in the epel-release repository in the last 12 hours, since then yum/dnf is unable to find libzstd-devel.
libzstd is added to the BaseOS repository and removed from EPEL:
https://src.fedoraproject.org/rpms/zstd/c/2eaa0955a5176319f2a59294ee339cb2fff87bc5?branch=epel8
But CentOS 8 that includes zstd isn't released yet. We'll be able to revert this change with the next CentOS 8 release.