Skip to content

fix(ci): RPM buildroot — resolve macros, install LICENSE/README#145

Merged
Jesssullivan merged 1 commit intomainfrom
sid/fix-rpm-buildroot
Apr 3, 2026
Merged

fix(ci): RPM buildroot — resolve macros, install LICENSE/README#145
Jesssullivan merged 1 commit intomainfrom
sid/fix-rpm-buildroot

Conversation

@Jesssullivan
Copy link
Copy Markdown
Owner

Summary

Fixes the RPM build failure in release-linux.yml:

  • Use rpm --eval '%{_udevrulesdir}' and '%{_libdir}' instead of hardcoded paths
  • Install LICENSE and README.md to buildroot (required by %license and %doc in spec)

Second fix after #143 (version hyphen). DEB and tarball already pass.

Test plan

  • Merge, then gh workflow run release-linux.yml -f tag=lab-v0.73.0-test2
  • Both DEB and RPM jobs should succeed
  • Upload job runs

The RPM build failed because:
1. Udev rules installed to wrong path (hardcoded vs %{_udevrulesdir})
2. %license LICENSE and %doc README.md missing from buildroot
3. %{_libdir} differs between distros (lib vs lib64)

Fix: resolve RPM macros at build time with rpm --eval, install all
files the spec %files section expects.
@Jesssullivan Jesssullivan merged commit d943c7f into main Apr 3, 2026
6 of 8 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.

1 participant