Skip to content
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

Replace source activate by conda activate in conda install guide #27950

Closed
slel opened this issue Jun 8, 2019 · 22 comments
Closed

Replace source activate by conda activate in conda install guide #27950

slel opened this issue Jun 8, 2019 · 22 comments

Comments

@slel
Copy link
Member

slel commented Jun 8, 2019

As reported by "dantetante"
on sage-devel,
the SageMath installation guide for conda
currently suggests

source activate sage

but this should be changed to

conda activate sage

For reference, this changed with the release of Conda 4.4.0, see the release notes:

Depends on #27817

CC: @saraedum @slel

Component: documentation

Keywords: conda

Author: Dima Pasechnik

Branch: 50c84da

Reviewer: Volker Braun

Issue created by migration from https://trac.sagemath.org/ticket/27950

@slel slel added this to the sage-8.8 milestone Jun 8, 2019
@dimpase
Copy link
Member

dimpase commented Jun 8, 2019

comment:1

Thanks. I meant to add it to #27817 but forgot...

@dimpase
Copy link
Member

dimpase commented Jun 8, 2019

Branch: #27817

@dimpase
Copy link
Member

dimpase commented Jun 8, 2019

Changed branch from #27817 to none

@dimpase
Copy link
Member

dimpase commented Jun 8, 2019

Dependencies: #27817

@dimpase
Copy link
Member

dimpase commented Jun 11, 2019

comment:4

please review.


New commits:

0e329c9updated conda installation details

@dimpase
Copy link
Member

dimpase commented Jun 11, 2019

Commit: 0e329c9

@dimpase
Copy link
Member

dimpase commented Jun 11, 2019

Branch: public/documentation/condainst

@dimpase
Copy link
Member

dimpase commented Jun 11, 2019

Author: Dima Pasechnik

@slel
Copy link
Member Author

slel commented Jun 12, 2019

comment:5
- dependicies resolution
+ dependency resolution

@dimpase
Copy link
Member

dimpase commented Jun 12, 2019

comment:6

you're welcome do add a commit, it's a public branch :-)

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 12, 2019

Changed commit from 0e329c9 to ef292fe

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 12, 2019

Branch pushed to git repo; I updated commit sha1. New commits:

ef292fefixed typo

@dimpase
Copy link
Member

dimpase commented Jun 12, 2019

comment:8

fixed.

@slel
Copy link
Member Author

slel commented Jun 12, 2019

comment:9

After doing

$ conda install mamba -c conda-forge # installs mamba

does something need to be done to make mamba work?
Running this directly

$ mamba create -n sage sage -c conda-forge # replaces "conda create..."

gives an error:

-bash: mamba: command not found

@slel
Copy link
Member Author

slel commented Jun 12, 2019

comment:10

It seems to be a matter of letting conda tweak one's PATH vs not.

Running the following works for me (I installed Miniconda in /opt):

$ /opt/miniconda3/bin/mamba create -n sage sage -c conda-forge

Besides that: maybe add some actual date instead of |today| in:

At the time of this edit (|today|)

@dimpase
Copy link
Member

dimpase commented Jun 12, 2019

comment:11

Replying to @slel:

After doing

$ conda install mamba -c conda-forge # installs mamba

does something need to be done to make mamba work?
Running this directly

$ mamba create -n sage sage -c conda-forge # replaces "conda create..."

gives an error:

-bash: mamba: command not found

it seems that your setup is non-standard (wrong conda is called ?). What I see that conda and mamba are installed into the same directory, and so whenever you have one in PATH, you have the other, too.

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 12, 2019

Branch pushed to git repo; I updated commit sha1. New commits:

50c84dawrote the actual date in

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Jun 12, 2019

Changed commit from ef292fe to 50c84da

@vbraun
Copy link
Member

vbraun commented Jun 14, 2019

Reviewer: Volker Braun

@vbraun
Copy link
Member

vbraun commented Jun 15, 2019

Changed branch from public/documentation/condainst to 50c84da

@slel
Copy link
Member Author

slel commented Aug 31, 2019

Changed commit from 50c84da to none

@slel

This comment has been minimized.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants