-
-
Notifications
You must be signed in to change notification settings - Fork 532
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
Installation guide: Do not recommend --prefix=$CONDA_PREFIX
#38017
base: develop
Are you sure you want to change the base?
Conversation
Documentation preview for this PR (built with commit 45e2591; changes) is ready! 🎉 |
While we are at it, would you unindent the surrounding block (which is shown as quoted text, unintentionally I think) here and below? |
…to put SAGE_LOCAL in $CONDA_PREFIX/libexec/sage-local
c508988
to
743d2da
Compare
Sorry but would you explain how "clobber" happens and why the change in this PR solves the problem? |
Now
seems reasonable to me. Is there an argument to prefer
than other choices, such as |
I think I can answer this. Previously sage installs files into
Then this also could be problematic... |
We may just create a new directory instead of treading into conda areas. How about
? |
Sage packages can clobber installed non-Python conda packages.
Instead we change it to recommend
--with-sage-venv=$CONDA_PREFIX
and to put SAGE_LOCAL in a subdirectory of the CONDA_PREFIX.📝 Checklist
⌛ Dependencies