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

Add "N" to the version model #4222

Merged
merged 1 commit into from
Sep 12, 2021
Merged

Add "N" to the version model #4222

merged 1 commit into from
Sep 12, 2021

Conversation

nirik
Copy link
Member

@nirik nirik commented May 21, 2021

Fedora's bodhi just added a "EPEL-8N" release that has
epel8-next-updates and epel8-next-updates-testing. This is to build
against CentOS stream in those cases where normal EPEL packages are no
longer compatible due to stream changes. The bodhi module doesn't
understand the 'N' here and can't figure out the version. If we add it
into the model it's fine. :)

Signed-off-by: Kevin Fenzi kevin@scrye.com

@nirik nirik requested a review from a team as a code owner May 21, 2021 16:53
@mattiaverga mattiaverga added this to the 5.7.1 milestone May 25, 2021
Copy link
Contributor

@mattiaverga mattiaverga left a comment

Choose a reason for hiding this comment

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

LGTM

Just let's announce this in release notes with a news fragment in /news, shall we?
https://bodhi.fedoraproject.org/docs/developer/#release-notes

Fedora's bodhi just added a "EPEL-8N" release that has
epel8-next-updates and epel8-next-updates-testing. This is to build
against CentOS stream in those cases where normal EPEL packages are no
longer compatible due to stream changes. The bodhi module doesn't
understand the 'N' here and can't figure out the version. If we add it
into the model it's fine. :)

Signed-off-by: Kevin Fenzi <kevin@scrye.com>
Copy link
Contributor

@mattiaverga mattiaverga left a comment

Choose a reason for hiding this comment

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

I've added the news fragment, so I think we can now merge this small change.

@mergify mergify bot merged commit 8f62be6 into fedora-infra:develop Sep 12, 2021
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