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
Use packit to produce rpms for fedora-35 #1648
Conversation
0e92faf
to
4f95b37
Compare
96d0f3c
to
1e665ff
Compare
1e665ff
to
7196bd1
Compare
7196bd1
to
ef7eaee
Compare
Did you make any progress on the rpm packaging in the meantime? Is there anything we can help with? |
No progress as I got other priorities, not sure when I will be able to return to this one. Feel free to work on it. |
I can see that all builds on Fedora and el8 are passing: what's left to be done? |
.packit.yaml
Outdated
- job: copr_build | ||
metadata: | ||
targets: | ||
# Unable to build centos-stream due to missing python3-setuptools_scm_git_archive |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not patch it out in the spec file? It's not like there's nowhere to get the version from.
.packit.yaml
Outdated
- "python3 setup.py --version" | ||
post-upstream-clone: | ||
- | | ||
bash -c "sed -i -e 's/Version: VERSION_PLACEHOLDER/Version: $(python3 setup.py --version)/' ansible-lint.spec" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bash -c "sed -i -e 's/Version: VERSION_PLACEHOLDER/Version: $(python3 setup.py --version)/' ansible-lint.spec" | |
bash -c "sed -i -e 's/Version: VERSION_PLACEHOLDER/Version: $(python3 -m setuptools_scm | awk '{print $3}')/' ansible-lint.spec" |
0c6ae2a
to
dfc9885
Compare
af649c7
to
72b7e5c
Compare
This configures setuptools-scm to cache version when installing or building the package, so we no longer need to use pkg_resources to gather the version. This removes the need for setuptools at runtime and also improve performance as version is loaded directly from our module, when present. Related: ansible#1648
b0dfd80
to
e20131a
Compare
e20131a
to
dbb4bf7
Compare
dbb4bf7
to
c414f0e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM !
No description provided.