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

[skip-ci] RPM: use default gobuild macro on RHEL #5938

Merged
merged 2 commits into from
Jan 24, 2025

Conversation

lsm5
Copy link
Member

@lsm5 lsm5 commented Jan 24, 2025

What type of PR is this?

/kind other

What this PR does / why we need it:

rpm spec cleanup

How to verify it

rhel builds will work as expected, rpmlint will no longer complain

Which issue(s) this PR fixes:

None

Special notes for your reviewer:

None

Does this PR introduce a user-facing change?

None

The default gobuild macro on RHEL now includes support for BUILDTAGS. So
we don't need to redefine it here.

`libtrust_openssl` buildtag will be set in the spec file for RHEL environments.

Signed-off-by: Lokesh Mandvekar <[email protected]>
Copy link
Contributor

openshift-ci bot commented Jan 24, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lsm5

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@lsm5 lsm5 changed the title Rpm gobuild macro RPM: use default gobuild macro on RHEL Jan 24, 2025
@lsm5 lsm5 added the No New Tests Allow PR to proceed without adding regression tests label Jan 24, 2025
@lsm5 lsm5 changed the title RPM: use default gobuild macro on RHEL [skip-ci] RPM: use default gobuild macro on RHEL Jan 24, 2025
@lsm5 lsm5 force-pushed the rpm-gobuild-macro branch from c53f546 to ca957c4 Compare January 24, 2025 11:32
@lsm5 lsm5 marked this pull request as ready for review January 24, 2025 13:16
@lsm5
Copy link
Member Author

lsm5 commented Jan 24, 2025

@containers/buildah-maintainers PTAL

@nalind
Copy link
Member

nalind commented Jan 24, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Jan 24, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit 865427c into containers:main Jan 24, 2025
32 checks passed
@lsm5 lsm5 deleted the rpm-gobuild-macro branch February 4, 2025 12:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved kind/other lgtm No New Tests Allow PR to proceed without adding regression tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants