Configure packit to use caret notation for postrelease snapshots#638
Configure packit to use caret notation for postrelease snapshots#638kontura wants to merge 1 commit intofedora-modularity:mainfrom
Conversation
|
The caret character is not supported on RHEL 7 and thus the CI fails there. libmodulemd needs to support RHEL 7. Is it possible to change the configuration file so that it keeps using the old schema for epel-7 target? |
This ensures the build of that specific version is considered newer than any other regular release of it. For rpm-software-management/ci-dnf-stack#1843 For epel7 keep old style since rpm doesn't support `^` in version there.
|
|
I have separated the |
ppisar
left a comment
ppisar
left a comment
There was a problem hiding this comment.
Various tests on Fedora, e.g. "Installed RPM Tests" failed:
Run $GITHUB_WORKSPACE/.ci/retry-command.sh dnf -y install --nogpgcheck --allowerasing --repofrompath libmodulemd-ci,x86_64 x86_64/python3-libmodulemd*.rpm x86_64/libmodulemd-devel*.rpm
[...]
Failed to resolve the transaction:
Problem 1: cannot install both python3-libmodulemd-2.15.3^20260407074247132574.0fea74c7c0ba16895f0691dab7bbd3db5c1b2718-0.20260407074247130049.0fea74c7c0ba16895f0691dab7bbd3db5c1b2718.fc45.x86_64 from @commandline and python3-libmodulemd-2.15.3-0.20260407074427604704.0fea74c7c0ba16895f0691dab7bbd3db5c1b2718.fc45.x86_64 from @commandline
- conflicting requests
I have not idea why it did not failed before, but obviously the dnf commands needs to explicitly install "$arch/libmodulemd*.rpm" package to allow DNF to replace a libmodulemd binary package with the matching version.
Could you please amend that command in .ci/fedora/ci-tasks.sh?
Since it's difficult to glob local "libmodulemd" package with unknown version without including libmodulemd-debuginfo package, try a different approach like dnf5 do --from-repo libmodulemd.
| version_suffix: "^{PACKIT_PROJECT_SNAPSHOTID}" | ||
| update_release: false | ||
|
|
||
| # rpmbuild doesn't allow '^' in version on rhel 7, don't use caret notatio |
There was a problem hiding this comment.
Add the missing "n" in the "notatio" word.
| - epel-8 | ||
| - centos-stream-9 | ||
| - centos-stream-10 | ||
|
|
There was a problem hiding this comment.
One fewer empty line, please.
2 participants
This ensures the build of that specific version is considered newer than any other regular release of it.
For rpm-software-management/ci-dnf-stack#1843
Example build: https://copr.fedorainfracloud.org/coprs/amatej/dnf-nightly/build/10282557/