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

packaging/rpm-ostree.spec: add libzstd-devel BuildRequires #5038

Merged
merged 1 commit into from
Aug 10, 2024

Conversation

jmarrero
Copy link
Member

@jmarrero jmarrero commented Aug 9, 2024

After the initial changes to ostree-ext introducing support for
zstd:chunked, which can be found in this pull request:
ostreedev/ostree-rs-ext#615,
an additional PR modified the dependency on zstd during build
time to use pkg-config.
You can review these changes in the pull request:
ostreedev/ostree-rs-ext#628.

These updates also required modifications to bootc, detailed in
this pull request: containers/bootc#586.
This commit mirrors those changes in rpm-ostree.

With these changes, it will be possible to build RPMs on systems
that do not have zstd-devel pre-installed in the build root.

Copy link
Member

@jlebon jlebon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's add a commit message.

@@ -42,6 +42,7 @@ BuildRequires: rust
%bcond_without zchunk
%endif

BuildRequires: libzstd-devel
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's put it together with the other "Core requirements" BRs below? And add a comment that it's a dep of the ostree-ext crate.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done, also added a commit message with what I understand changed on ostree-ext that now requires this. But I could be wrong.

After the initial changes to ostree-ext introducing support for
zstd:chunked, which can be found in this pull request:
ostreedev/ostree-rs-ext#615,
an additional PR modified the dependency on zstd during build
time to use pkg-config.
You can review these changes in the pull request:
ostreedev/ostree-rs-ext#628.

These updates also required modifications to bootc, detailed in
this pull request: containers/bootc#586.
This commit mirrors those changes in rpm-ostree.

With these changes, it will be possible to build RPMs on systems
that do not have zstd-devel pre-installed in the build root.
Copy link

openshift-ci bot commented Aug 10, 2024

@jmarrero: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/fcos-e2e eb5ca09 link true /test fcos-e2e

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

Copy link
Member

@jlebon jlebon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice commit message. 👍
Kola failure looks legit but unrelated. Should probably peel that off into a separate issue.

@jmarrero jmarrero merged commit cd2cfd6 into coreos:main Aug 10, 2024
14 of 17 checks passed
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.

2 participants