-
-
Notifications
You must be signed in to change notification settings - Fork 504
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
Install sage-env-config as part of sage_conf #29850
Comments
This comment has been minimized.
This comment has been minimized.
Dependencies: #29825 |
comment:4
Branch on this ticket does NOT work because pip/setuptools do not handle the symlink well. Last 10 new commits:
|
Commit: |
comment:5
Perhaps it's easier to get rid of sage-env-config altogether (#29852). |
comment:9
Replying to @mkoeppe:
It would probably suffice to just replace the use of |
This comment has been minimized.
This comment has been minimized.
Work Issues: use setup.py install, add spkg-src |
Changed work issues from use setup.py install, add spkg-src to make sage-env-config optional for sage-env |
Changed work issues from make sage-env-config optional for sage-env to none |
This comment has been minimized.
This comment has been minimized.
Author: Matthias Koeppe |
Work Issues: Move sage_conf.py to SAGE_ROOT/src/, symlink to there |
comment:17
(comment was meant for a different ticket) |
Changed work issues from Move sage_conf.py to SAGE_ROOT/src/, symlink to there to none |
Branch pushed to git repo; I updated commit sha1. This was a forced push. Last 10 new commits:
|
Work Issues: Rebase on top of #22731 |
Branch pushed to git repo; I updated commit sha1. This was a forced push. Last 10 new commits:
|
Changed work issues from Rebase on top of #22731 to none |
This comment has been minimized.
This comment has been minimized.
Changed keywords from none to sd111 |
Work Issues: use new function from #31002 |
Branch pushed to git repo; I updated commit sha1. New commits:
|
comment:45
Still needs review... |
comment:46
OK. |
comment:47
Thanks! |
Changed branch from u/mkoeppe/install_sage_env_config_with_sage_conf to |
As a follow-up to #21559 and #30128, we reassign
sage-env-config
from the packagesagelib
to the packagesage_conf
.(After #22731,
sage-env-config
is no longer required by thesage
script.)(This is preparation for allowing installation of sagelib for example in a conda environment without having to run ./configure first - see #28745, #30845.)
Part of Meta-ticket #21707: Split sage-env into 5 to clean up sage configuration
Depends on #31002
CC: @jhpalmieri @kiwifb @orlitzky @dimpase @isuruf @tobiasdiez
Component: build
Keywords: sd111
Author: Matthias Koeppe
Branch/Commit:
4b52dcf
Reviewer: Dima Pasechnik
Issue created by migration from https://trac.sagemath.org/ticket/29850
The text was updated successfully, but these errors were encountered: