-
-
Notifications
You must be signed in to change notification settings - Fork 480
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
tox.ini: Update voidlinux, update installation guide #36957
Conversation
574d229
to
ad0531d
Compare
For any void linux users out there:
is in general quite up to date (10.2 right now). Feel free to report any issues in https://github.com/void-linux/void-packages/issues. We also have a draft PR tracking beta/rc sagemath prereleases (currently void-linux/void-packages#47712 tracking 10.3 prereleases) Installing sagemath in any other way is unsupported. The main problem you could have is that, void being a rolling distro, the system packages will change and your locally compiled sage might break. |
…commend Arch, Void downstream packages
58be8bb
to
2b3874f
Compare
… up-to-date versions of SageMath
I'll defer other work on the installation guide to a follow up PR. |
Documentation lgtm. As explained above I'm not really interested, but just to see how it works I tried running This is futile, why are you trying to build python? BTW, I think You are also missing a bunch of packages (of course everything required is available as packages; nothing is vendored in in our sagemath package). Most notable:
For the names of required packages, see: https://github.com/void-linux/void-packages/blob/master/srcpkgs/sagemath/template |
There's definitely something wrong with fflas_ffpack, we also see random hangs like you describe on other platforms in our CI. Hoping to get #35148 unstuck soon.
It's probably rejecting python 3.12. |
Yes, using system site packages is still only an experimental option @orlitzky. To test it with tox, one can use
Thanks for the pointer! You added many of these packages in I'll check what's new. |
|
According to https://docs.python.org/3/library/crypt.html: Deprecated since version 3.11, will be removed in version 3.13 I'll remove the check for it |
tox is broken on voidlinux
(Also unusable because it is outdated) |
Isn't it just running a (very, very) long test suite? Note that the new versions of givaro, fflas, and linbox are supported since 10.1 (#35612) so even without #35148 it should be possible to use them just fine. All the upper bounds on unreleased package versions are really bad. Should just get rid of all of them.
For no reason. Same as above. BTW, I changed |
Cherry-picked for velocity in #36983, please review |
@tornaria Let's merge it? |
Thanks! |
By the way, I had trouble to get voidlinux to work in a devcontainer. I don't know who the target user base of voidlinux is, but I think that's a showstopper for adoption by developers who use VS Code or want to use Codespaces or anything like that. |
I have no idea what is a "devcontainer".
Can scratch its own itch.
Some might say this is a feature. But in fact there's actually a vscode package and it's up to date, so I don't see what is the problem. |
Since you didn't say "... and I don't want to know": https://code.visualstudio.com/docs/devcontainers/containers
It means for Sage that -- despite the competent packaging of Sage on voidlinux -- it's hard to recommend it as a deployment option for Sage for users who have to use Windows. |
Ok, but what is the trouble you had, and what can be done on void linux side to improve your experience?
Still no idea what went wrong. |
Let's take this to #36995 |
@tornaria Did you say you also package the current beta? What would I need to do to install it? |
I do have a draft PR void-linux/void-packages#47712 It works perfectly fine, it's a draft only because void policy is to stick to main releases, and I wouldn't want to be updating sagemath every week anyways. However, there are no binary packages for the beta release: you'd have to build it from source using If you find this could be useful, I can do it from scratch and record step by steps (this would be 3-4 easy automatic steps). I guess one could also put the repo with the beta packages built in some public place, then just configure xbps to use that repo, but I don't do this. |
Yes, that would be great, thanks!
No need for this. |
Something like this should work: git clone --depth=200 -b sagemath https://github.com/tornaria/void-packages void-sage
cd void-sage
git fetch --depth=500 https://github.com/void-linux/void-packages
git rebase FETCH_HEAD
./xbps-src binary-bootstrap
./xbps-src pkg sagemath
sudo xbps-install --repository=hostdir/binpkgs/sagemath sagemath It's important that you rebase on top of the main repo to avoid rebuilding packages that have changed. The pkg step you can change for Once the package is built, all you need to share with other boxes to install the pkg should be in the local repo in |
Done in |
Is this usable from the command line, or only via vscode? If so, how? Does it work with podman or only with docker? |
You can probably use https://github.com/devcontainers/cli but I have no experience with it |
<!-- ^^^^^ Please provide a concise, informative and self-explanatory title. Don't put issue numbers in there, do this in the PR body below. For example, instead of "Fixes sagemath#1234" use "Introduce new method to calculate 1+1" --> <!-- Describe your changes here in detail --> To test: Use `export DOCKER_TARGETS="with-system-packages configured with-targets-pre with-targets"`, then one of: - `tox -e docker-voidlinux-standard-python3.11` - `tox -e docker-voidlinux-musl-standard-python3.11` Support for `tox -e docker-voidlinux-standard-sitepackages` will have to wait for: - sagemath#36181 This - Fixes sagemath#33084 <!-- Why is this change required? What problem does it solve? --> <!-- If this PR resolves an open issue, please link to it here. For example "Fixes sagemath#12345". --> <!-- If your change requires a documentation PR, please link it appropriately. --> ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> <!-- If your change requires a documentation PR, please link it appropriately --> <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> <!-- Feel free to remove irrelevant items. --> - [ ] The title is concise, informative, and self-explanatory. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation accordingly. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on - sagemath#12345: short description why this is a dependency - sagemath#34567: ... --> <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> URL: sagemath#36957 Reported by: Matthias Köppe Reviewer(s):
…ing sphinx-inline-tabs <!-- ^^^^^ Please provide a concise, informative and self-explanatory title. Don't put issue numbers in there, do this in the PR body below. For example, instead of "Fixes sagemath#1234" use "Introduce new method to calculate 1+1" --> <!-- Describe your changes here in detail --> Preview: https://deploy-preview-37056-- sagemath.netlify.app/html/en/installation/source#linux-system-package- installation Same for the SPKG pages in the reference manual. https://deploy- preview-37056-- sagemath.netlify.app/html/en/reference/spkg/gfortran#equivalent-system- packages Same for the git instructions in the developer guide, using 2 tabs for HTTPS and SSH: https://deploy-preview-37056-- sagemath.netlify.app/html/en/developer/github#forking-the-sage- repository <!-- Why is this change required? What problem does it solve? --> <!-- If this PR resolves an open issue, please link to it here. For example "Fixes sagemath#12345". --> <!-- If your change requires a documentation PR, please link it appropriately. --> ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> <!-- If your change requires a documentation PR, please link it appropriately --> <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> <!-- Feel free to remove irrelevant items. --> - [x] The title is concise, informative, and self-explanatory. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation accordingly. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on - sagemath#12345: short description why this is a dependency - sagemath#34567: ... --> - Depends on sagemath#36957 (merged here) <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> URL: sagemath#37056 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee
…ing sphinx-inline-tabs <!-- ^^^^^ Please provide a concise, informative and self-explanatory title. Don't put issue numbers in there, do this in the PR body below. For example, instead of "Fixes sagemath#1234" use "Introduce new method to calculate 1+1" --> <!-- Describe your changes here in detail --> Preview: https://deploy-preview-37056-- sagemath.netlify.app/html/en/installation/source#linux-system-package- installation Same for the SPKG pages in the reference manual. https://deploy- preview-37056-- sagemath.netlify.app/html/en/reference/spkg/gfortran#equivalent-system- packages Same for the git instructions in the developer guide, using 2 tabs for HTTPS and SSH: https://deploy-preview-37056-- sagemath.netlify.app/html/en/developer/github#forking-the-sage- repository <!-- Why is this change required? What problem does it solve? --> <!-- If this PR resolves an open issue, please link to it here. For example "Fixes sagemath#12345". --> <!-- If your change requires a documentation PR, please link it appropriately. --> ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> <!-- If your change requires a documentation PR, please link it appropriately --> <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> <!-- Feel free to remove irrelevant items. --> - [x] The title is concise, informative, and self-explanatory. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation accordingly. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on - sagemath#12345: short description why this is a dependency - sagemath#34567: ... --> - Depends on sagemath#36957 (merged here) <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> URL: sagemath#37056 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee
…ing sphinx-inline-tabs <!-- ^^^^^ Please provide a concise, informative and self-explanatory title. Don't put issue numbers in there, do this in the PR body below. For example, instead of "Fixes sagemath#1234" use "Introduce new method to calculate 1+1" --> <!-- Describe your changes here in detail --> Preview: https://deploy-preview-37056-- sagemath.netlify.app/html/en/installation/source#linux-system-package- installation Same for the SPKG pages in the reference manual. https://deploy- preview-37056-- sagemath.netlify.app/html/en/reference/spkg/gfortran#equivalent-system- packages Same for the git instructions in the developer guide, using 2 tabs for HTTPS and SSH: https://deploy-preview-37056-- sagemath.netlify.app/html/en/developer/github#forking-the-sage- repository <!-- Why is this change required? What problem does it solve? --> <!-- If this PR resolves an open issue, please link to it here. For example "Fixes sagemath#12345". --> <!-- If your change requires a documentation PR, please link it appropriately. --> ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> <!-- If your change requires a documentation PR, please link it appropriately --> <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> <!-- Feel free to remove irrelevant items. --> - [x] The title is concise, informative, and self-explanatory. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation accordingly. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on - sagemath#12345: short description why this is a dependency - sagemath#34567: ... --> - Depends on sagemath#36957 (merged here) <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> URL: sagemath#37056 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee
…ing sphinx-inline-tabs <!-- ^^^^^ Please provide a concise, informative and self-explanatory title. Don't put issue numbers in there, do this in the PR body below. For example, instead of "Fixes sagemath#1234" use "Introduce new method to calculate 1+1" --> <!-- Describe your changes here in detail --> Preview: https://deploy-preview-37056-- sagemath.netlify.app/html/en/installation/source#linux-system-package- installation Same for the SPKG pages in the reference manual. https://deploy- preview-37056-- sagemath.netlify.app/html/en/reference/spkg/gfortran#equivalent-system- packages Same for the git instructions in the developer guide, using 2 tabs for HTTPS and SSH: https://deploy-preview-37056-- sagemath.netlify.app/html/en/developer/github#forking-the-sage- repository <!-- Why is this change required? What problem does it solve? --> <!-- If this PR resolves an open issue, please link to it here. For example "Fixes sagemath#12345". --> <!-- If your change requires a documentation PR, please link it appropriately. --> ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> <!-- If your change requires a documentation PR, please link it appropriately --> <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> <!-- Feel free to remove irrelevant items. --> - [x] The title is concise, informative, and self-explanatory. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation accordingly. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on - sagemath#12345: short description why this is a dependency - sagemath#34567: ... --> - Depends on sagemath#36957 (merged here) <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> URL: sagemath#37056 Reported by: Matthias Köppe Reviewer(s): Kwankyu Lee
… (setuptools instead of distutils) <!-- ^^^^^ Please provide a concise, informative and self-explanatory title. Don't put issue numbers in there, do this in the PR body below. For example, instead of "Fixes sagemath#1234" use "Introduce new method to calculate 1+1" --> <!-- Describe your changes here in detail --> - Cherry-picked from sagemath#36181 This PR does not change what versions are accepted! To test with Python 3.12, still need to adjust the version bound set in `build/pkgs/python3/spkg-configure.m4`. <!-- Why is this change required? What problem does it solve? --> <!-- If this PR resolves an open issue, please link to it here. For example "Fixes sagemath#12345". --> <!-- If your change requires a documentation PR, please link it appropriately. --> ### 📝 Checklist <!-- Put an `x` in all the boxes that apply. --> <!-- If your change requires a documentation PR, please link it appropriately --> <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> <!-- Feel free to remove irrelevant items. --> - [x] The title is concise, informative, and self-explanatory. - [ ] The description explains in detail what this PR is about. - [ ] I have linked a relevant issue or discussion. - [ ] I have created tests covering the changes. - [ ] I have updated the documentation accordingly. ### ⌛ Dependencies <!-- List all open PRs that this PR logically depends on - sagemath#12345: short description why this is a dependency - sagemath#34567: ... --> - Depends on sagemath#36957 (merged here to resolve merge conflict) <!-- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> URL: sagemath#36983 Reported by: Matthias Köppe Reviewer(s): Gonzalo Tornaría, Matthias Köppe
To test: Use
export DOCKER_TARGETS="with-system-packages configured with-targets-pre with-targets"
, then one of:tox -e docker-voidlinux-standard-python3.11
tox -e docker-voidlinux-musl-standard-python3.11
Support for
tox -e docker-voidlinux-standard-sitepackages
will have to wait for:This
📝 Checklist
⌛ Dependencies