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.

Sign up for GitHub

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

Jump to bottom

Treat base conda environment as though it was a system Python #7137

Open
notatallshaw opened this issue Sep 6, 2024 · 0 comments · Fixed by #7691
Open

Treat base conda environment as though it was a system Python #7137

notatallshaw opened this issue Sep 6, 2024 · 0 comments · Fixed by #7691
Labels
breaking A breaking change needs-decision Undecided if this should be done

Comments

@notatallshaw
Copy link
Contributor

notatallshaw commented Sep 6, 2024 •
edited
Loading

The base environment of conda is often not something that should be installed into as it can break conda.

It appears you can detect if the base environment is currently activated if CONDA_PREFIX_1 is not set and/or CONDA_DEFAULT_ENV is equal to base (on very old versions of conda this was named root) : https://conda.io/projects/conda/en/latest/dev-guide/deep-dives/activation.html#conda-activate

Perhaps worth requiring users to use --system to install into this base conda environment?
@notatallshaw notatallshaw mentioned this issue Sep 6, 2024
Open
@zanieb zanieb added needs-decision Undecided if this should be done breaking A breaking change labels Sep 7, 2024
@zanieb zanieb mentioned this issue Sep 25, 2024
Merged
zanieb added a commit that referenced this issue Oct 21, 2024
@zanieb
Treat the base Conda environment as a system environment (#7691)
2777324 
Closes #7124
Closes #7137
zanieb added a commit that referenced this issue Oct 22, 2024
@zanieb
Treat the base Conda environment as a system environment (#7691)
0a985f4 
Closes #7124
Closes #7137
zanieb added a commit that referenced this issue Oct 24, 2024
@zanieb
Treat the base Conda environment as a system environment (#7691)
248e9f7 
Closes #7124
Closes #7137
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
breaking A breaking change needs-decision Undecided if this should be done
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Treat the base Conda environment as a system environment astral-sh/uv
2 participants
@zanieb @notatallshaw