Skip to content

kernel-clk6.12: add ciq-kmod Requires and ship %clk_version macro#1253

Merged
bmastbergen merged 1 commit into
ciq-6.12.yfrom
jtate/clk-kmod-requires-6.12
Jun 5, 2026
Merged

kernel-clk6.12: add ciq-kmod Requires and ship %clk_version macro#1253
bmastbergen merged 1 commit into
ciq-6.12.yfrom
jtate/clk-kmod-requires-6.12

Conversation

@josephtate
Copy link
Copy Markdown

Summary

  • Add Requires: ciq-kmod to the kernel-clk6.12 metapackage so the patched kmod binary (which handles xz-compressed symvers in CLK 6.x kernels) is pulled in automatically when the CLK kernel is installed
  • Create /usr/lib/rpm/macros.d/macros.kernel-clk6.12 in kernel-clk6.12-devel containing %clk_version 6.12, so kmod spec files can reference the CLK version without hardcoding it
  • The macros file and %files entry are both guarded to the base (non-variant) devel package only

Test plan

  • Build kernel-clk6.12 and kernel-clk6.12-devel RPMs
  • Verify rpm -qR kernel-clk6.12-*.rpm includes ciq-kmod
  • Verify rpm -ql kernel-clk6.12-devel-*.rpm includes /usr/lib/rpm/macros.d/macros.kernel-clk6.12
  • Verify contents of that file are %clk_version 6.12
  • Verify debug-devel and 64k-devel do NOT own the macros file

🤖 Generated with Claude Code

@PlaidCat PlaidCat requested review from a team, bmastbergen and Copilot and removed request for bmastbergen May 21, 2026 22:08
@PlaidCat PlaidCat requested a review from jdieter May 21, 2026 22:08
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the kernel-clk6.12 RPM spec to ensure the CLK kernel pulls in the patched ciq-kmod package automatically, and to ship a %clk_version RPM macro via the kernel-clk6.12-devel package for downstream kmod specs to reference.

Changes:

  • Add Requires: ciq-kmod to the kernel-clk6.12 metapackage.
  • Install a macros drop-in file that defines %clk_version %{kernel_major_minor} during the base (non-variant) kernel-devel build.
  • Add the macros file to the %files ... devel list only for the base (non-variant) devel package.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ciq/SPECS/kernel-clk6.12.spec Outdated
Comment thread ciq/SPECS/kernel-clk6.12.spec Outdated
@ciq-kernel-automation ciq-kernel-automation Bot deleted the branch ciq-6.12.y June 4, 2026 15:06
@bmastbergen bmastbergen reopened this Jun 4, 2026
@bmastbergen bmastbergen force-pushed the jtate/clk-kmod-requires-6.12 branch from 5355c80 to 3fd25bf Compare June 4, 2026 17:10
@josephtate josephtate force-pushed the jtate/clk-kmod-requires-6.12 branch from 3fd25bf to b036514 Compare June 5, 2026 16:54
@josephtate josephtate force-pushed the jtate/clk-kmod-requires-6.12 branch from b036514 to b78c55d Compare June 5, 2026 17:02
Copy link
Copy Markdown
Collaborator

@bmastbergen bmastbergen left a comment

Choose a reason for hiding this comment

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

🥌

@bmastbergen bmastbergen requested a review from a team June 5, 2026 17:14
Copy link
Copy Markdown
Collaborator

@PlaidCat PlaidCat left a comment

Choose a reason for hiding this comment

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

:shipit:

@bmastbergen bmastbergen merged commit 946af30 into ciq-6.12.y Jun 5, 2026
8 checks passed
@bmastbergen bmastbergen deleted the jtate/clk-kmod-requires-6.12 branch June 5, 2026 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants