-
-
Notifications
You must be signed in to change notification settings - Fork 481
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
System package information for MacPorts #30504
Comments
comment:1
Inspired by a report on sage-support: |
comment:3
Another request: |
comment:4
You know what to do... add system package information, https://repology.org/ |
Dependencies: #31114 |
Changed dependencies from #31114 to none |
Author: Samuel Lelièvre |
Last 10 new commits:
|
Commit: |
Changed keywords from none to macports |
Branch: u/slelievre/30504 |
comment:7
Not ready yet. |
comment:8
Wow, thanks a lot for this work. Some comments:
|
comment:9
See also #30124, in which I started to add package info for packages related to the Jupyter notebook |
comment:10
The necessary changes to |
comment:11
Replying to @mkoeppe:
Thanks for warning about this, I'll drop pypi and raspbian. Are the distros we write files for listed somewhere, |
comment:12
Answering my own question: from the Sage root directory:
|
comment:13
docutils.txt must be a typo centos.txt - we actually use fedora.txt for CentOS and hopefully centos.txt can be removed |
comment:14
Indeed, could you just remove this centos.txt as part of this ticket? It's not needed |
comment:15
This |
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
|
comment:33
I fixed |
comment:34
Is it a problem that MacPorts uses |
comment:35
Some reading about gfortran and MacPorts: |
comment:36
Thanks for the pointer! |
comment:37
For Feel free to change anything if it's useful. |
comment:38
Yes, the build was successful with these changes:
|
comment:39
With latest #31505 merged, I am now getting:
|
comment:40
Replying to @slel:
Our scripts don't check for |
Changed branch from u/slelievre/30504 to u/mkoeppe/30504 |
Reviewer: Matthias Koeppe |
comment:42
I'd say this is good enough. Thanks a lot for this work. Further refinements can be done in follow up tickets. New commits:
|
Branch pushed to git repo; I updated commit sha1 and set ticket back to needs_review. This was a forced push. New commits:
|
comment:44
(fixed up the merge) |
This comment has been minimized.
This comment has been minimized.
comment:46
Thanks! |
Changed branch from u/mkoeppe/30504 to |
This comment has been minimized.
This comment has been minimized.
comment:48
Thanks for merging! |
Changed commit from |
This ticket adds system packages information for MacPorts,
a common macOS package manager.
The follow-up ticket #31505 extends the package scripts
to make use of them.
Related:
CC: @mkoeppe @slel
Component: build
Keywords: macports
Author: Samuel Lelièvre
Branch:
b8b7092
Reviewer: Matthias Koeppe
Issue created by migration from https://trac.sagemath.org/ticket/30504
The text was updated successfully, but these errors were encountered: