-
Notifications
You must be signed in to change notification settings - Fork 149
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
ci: Add Fedora build job to GitHub Actions CI #1610
ci: Add Fedora build job to GitHub Actions CI #1610
Conversation
c087c74
to
8952b91
Compare
@simonmichal This is ready for review. |
@matthewfeickert : I just pushed a6a1fda that replaces the use of |
P. S. I think we don't need anymore the adjustments to the spec file (but please double check), only the test for the CI. |
8952b91
to
99a4b85
Compare
PYTHON_INSTALLED is no longer used anywhere in the codebase, so there is no need to use the --record option for deprecated `python setup.py install` installs.
@@ -81,8 +81,7 @@ if ( NOT ${VALID_PIP_EXIT_CODE} EQUAL 0 ) | |||
COMMAND ${PYTHON_EXECUTABLE} ${SETUP_PY} install \ | |||
${PIP_INSTALL_VERBOSE_FLAG} \ | |||
--prefix \$ENV{DESTDIR}/${CMAKE_INSTALL_PREFIX} \ | |||
${DEB_INSTALL_ARGS} \ | |||
--record PYTHON_INSTALLED | |||
${DEB_INSTALL_ARGS} |
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.
a6a1fda removed all use of PYTHON_INSTALLED
so no need to use --record
here anymore.
@simonmichal this is ready for review. |
Resolves #1609Following PR #1586, the use ofPYTHON_INSTALLED_FILES
is regulated to Python 2 builds only. To guard against Python 3 build looking forPYTHON_INSTALLED_FILES
when it doesn't exist, have the file only be created underpython2only
andpython2and3
builds. Forpython3only
builds usepython3_sitearch
.Superseded by a6a1fda.
Add a Fedora 35 build job to the CI.