-
Notifications
You must be signed in to change notification settings - Fork 3
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
Update Poetry re rpmbuild & normalise on Python 3.11 #50 #52
Update Poetry re rpmbuild & normalise on Python 3.11 #50 #52
Conversation
Align with rockstor-core `build.sh` re Poetry update and install via OS provided pipx under Python 3.11. ## Includes - Path changes regarding new Poetry install location. - All targets move from Python 3.9 to Python 3.11. - Additional python311-pipx BuildRequires and Requires. - Additional update-alternatives to enforce Py3.11 pipx.
TestingWe have the intended new build dependencies doing their thing:
|
rpmbuild - legacy Poetry in-place%build scriptlet:
%check scriptlet
|
Resulting rpm installThe resulting test rpm, installed on the build host, has the following of note in: %posttrans scriptletHere we do our
We have our new "... i.e. NOT via RPM:" instruction and an apparent pickup of the installed during rpmbuild global poetry. poetry-install.txt
systemd servicesrockstor-pre
rockstor
rockstor-bootstrap
|
Poetry showIn the resulting rpm install we have a working poetry re:
&
&
|
The rpm build against this PR and its referenced rockstor-core partner, also successfully rebooted with all services starting as expected. |
rpmbuild - pipx installed Poetry in-placeTesting for issues with a repeat rpmbuild, where an existing instance of the new 1.7.1 pipx installer Poetry already exists. %build scriptlet:The exact same rpm was re-build as per comment: (#52 (comment))
|
@FroggyFlox & @Hooverdan96 Merging as-is as we clear up three outstanding issues (rockstor-core partner PR), with a hefty technical dept assistance; and we have clear-cut positive test results re the indicated issues there and here in rockstor-rpmbuild where we proven rpmbuild and repeat rpmbuild in concert with the rockstor-core PR. Plus these multi repo partnered issues are a little cumbersome to have hanging open. These changes may come back to haunt us be we are still mid to mid-late testing phase for some more field tests to this end - i.e. updating from prior rpms and future updates newer rpms to follow. |
Align with rockstor-core
build.sh
re Poetry update and install via OS provided pipx under Python 3.11.Thanks to @FroggyFlox for assisting with development.
Includes
Fixes #50
See also partner:
Issue re rockstor-core: rockstor/rockstor-core#2703
Pull request re rockstor-core: rockstor/rockstor-core#2755