-
Notifications
You must be signed in to change notification settings - Fork 28
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
WIP: rpm building #316
WIP: rpm building #316
Conversation
@daaitudian As rpm building is currently not working at all, I cannot test your change about changelog generation. I will need you help to ensure we can build an rpm using |
Hi @ssbarnea : eg: python -I -m pip --default-timeout 1000 install PyYAML 'ansible-core>=2.12' coverage 'jsonschema>=4.6.0' packaging pip-tools pytest-mock pytest-plus 'pytest>=7.2.0' 'subprocess-tee>=0.4.1' 'typing-extensions>=4.5.0; python_version < "3.10"' Fedora RPM build failed due to the lack of "spec.in".
Then, I think we should be added "rpm" in the tox.ini "dnf install -y" and "allowlist_externals" content. In addition, Redhat and CentOS lack "packit" and related packages causes the "dnf install" to fail. I think this has also caused some impacts. Maybe we should replace "test -f /etc/redhat-release" with "uname -a|grep fedora". For Redhat and CentOS we can omit the build process or use other methods. |
On fedora I see this failing and I do not understand what is causing it:
PS. I am making the other changes you suggested, to make it usable on all 3 distros. |
I'm sorry I forgot this question. We should disable the debug package build. |
When I used If execute it manually, we can create a |
for more information, see https://pre-commit.ci
I will need some help here, so if someone can takeover the WIP PR and make it work at least on Fedora it would be awesome. |
/packit build |
Related: #315