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

fix: use centos-stream-9 target instead of epel-9 #687

Merged
merged 2 commits into from
Oct 10, 2024

Conversation

mmartinv
Copy link
Contributor

@mmartinv mmartinv commented Oct 10, 2024

Use the centos-stream-9 target as the epel-9 repos
don't seem to be up to date.

Remove the preparation steps enabling the copr repo
and installing the copr packages because it's not needed
and it's causing problems: Packit is supposed to enable the
copr repo and install all the packages contained in
that repo.

Use pipx instead of pip3 as suggested by the error
message to fix the postgres tests.

@mmartinv mmartinv force-pushed the use-centos-stream-9-instead-of-epel-9 branch from 58d36f9 to 31c2180 Compare October 10, 2024 06:51
Use the centos-stream-9 target as the epel-9 repos
don't seem to be up to date.

Remove the preparation steps enabling the copr repo
and installing the copr packages because it's not needed
and it's causing problems: Packit is supposed to enable the
copr repo and install all the packages contained in
that repo.

Signed-off-by: Miguel Martín <mmartinv@redhat.com>
Use pipx instead of pip3 as suggested by the error
message to fix the postgres tests.

Signed-off-by: Miguel Martín <mmartinv@redhat.com>
@mmartinv mmartinv force-pushed the use-centos-stream-9-instead-of-epel-9 branch from 16d6f2c to ae037db Compare October 10, 2024 10:02
@mmartinv mmartinv changed the title fix: build for centos-stream-9 instead of epel-9 fix: use centos-stream-9 target instead of epel-9 Oct 10, 2024
Copy link
Contributor

@nullr0ute nullr0ute left a comment

Choose a reason for hiding this comment

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

LGTM

@mergify mergify bot merged commit f0bd963 into fdo-rs:main Oct 10, 2024
26 checks passed
@nullr0ute
Copy link
Contributor

Use the centos-stream-9 target as the epel-9 repos don't seem to be up to date.

For future reference EPEL uses the current release of RHEL as the base repo (so currently 9.4+updates) where as stream is the next release of RHEL so it's likely the version that users would expect, not out of date.

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