-
-
Notifications
You must be signed in to change notification settings - Fork 544
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 manual: Update section "system-wide install" #33787
Comments
Author: Matthias Koeppe |
Commit: |
New commits:
|
Branch pushed to git repo; I updated commit sha1. New commits:
|
This comment has been minimized.
This comment has been minimized.
comment:6
The suggestion to use $ sudo ln -s $(sage -sh -c 'ls $SAGE_ROOT/venv/bin/sage') /usr/local/bin is not consistent with the top-level if [ -x "$SAGE_ROOT/src/bin/sage" ]; then
exec "$SAGE_ROOT/src/bin/sage" "$@"
elif [ -x "$SAGE_ROOT/local/bin/sage" ]; then # if in a stripped binary
# Note in this case we assume that SAGE_LOCAL is the "local" subdirectory
exec "$SAGE_ROOT/local/bin/sage" "$@"
else
echo >&2 "$0: no Sage installation found in \$SAGE_ROOT=$SAGE_ROOT"
exit 1
fi (a) Should the Sage script look in |
comment:7
The The "elif" branch is only relevant in binary builds that don't ship |
comment:8
Actually, this turns out to be irrelevant - it looks like Volker stopped making these binaries altogether, the last one is 9.4 -http://files.sagemath.org/linux/64bit/index.html |
comment:9
But for the record, http://files.sagemath.org/linux/64bit/sage-9.4-Ubuntu_20.04-x86_64.tar.bz2 does have |
comment:10
"stripped binary" might refer to what |
comment:12
So here's a simplification. |
comment:13
Replying to @jhpalmieri:
No. After a build of Sage, using the installed version is more canonical. |
comment:14
Follow-up = #33841 |
comment:15
How can you run
is |
comment:16
Thanks for catching this |
Branch pushed to git repo; I updated commit sha1. New commits:
|
comment:18
(I had copied this from similar invocations shown in |
comment:19
Looks good to me. |
Reviewer: John Palmieri |
comment:20
Thanks! |
Changed branch from u/mkoeppe/installation_manual__update_section__system_wide_install_ to |
(split out from #30207)
https://doc.sagemath.org/html/en/installation/source.html#installation-in-a-multiuser-environment should be updated:
sage
script in venvWe also add a link from the README to an optional step described only in the installation manual.
CC: @dimpase @jhpalmieri @slel
Component: documentation
Author: Matthias Koeppe
Branch/Commit:
7bc3394
Reviewer: John Palmieri
Issue created by migration from https://trac.sagemath.org/ticket/33787
The text was updated successfully, but these errors were encountered: