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

chore: merge fedora and upstream spec file #74

Merged
merged 1 commit into from
Oct 25, 2023

Conversation

nullr0ute
Copy link
Member

The upstream splits out things that don't really need to be, but have things the Fedora spec doesn't. Merge the two for the best of both worlds so we can use things like packit to automate as much as possible.

@nullr0ute
Copy link
Member Author

@coiby can you review these changes, it will align Fedora and here, I think overall it simplifies it somewhat.

@coiby
Copy link
Contributor

coiby commented Oct 25, 2023

While looking at the changes, I notice python2-{rpm,cryptography,koji,rpm-head-signing} doesn't exist for RHEL8 even after enabling CRB and EPEL. So I think we can greatly simplify the the SPEC file by removing python2-related lines.

@nullr0ute
Copy link
Member Author

While looking at the changes, I notice python2-{rpm,cryptography,koji,rpm-head-signing} doesn't exist for RHEL8 even after enabling CRB and EPEL. So I think we can greatly simplify the the SPEC file by removing python2-related lines.

It builds for EPEL-8 but TBH I don't think it makes sense to support py2 so I agree, I had left it because the previous upstream spec had it. Will remove.

The upstream splits out things that don't really need
to be, but have things the Fedora spec doesn't. Merge
the two for the best of both worlds so we can use things
like packit to automate as much as possible. We also
drop the Python2 builds as it's long EOL.

Signed-off-by: Peter Robinson <[email protected]>
@nullr0ute nullr0ute merged commit 9616778 into fedora-iot:main Oct 25, 2023
16 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.

2 participants