Fix csh path #10410
Fix csh path #10410
Conversation
…rs or default commands
|
We require contributors to sign our Contributor License Agreement, and we don't have one on file for @bgobbi. In order for us to review and merge your code, please e-sign the PDF at https://conda.io/en/latest/contributing.html#conda-contributor-license-agreement. We then need to manually verify your signature. We will ping the bot to refresh the PR status when we have confirmed your signature. |
|
I have signed the cla multiple times now. Even so this is still marked as "failed" here. I do not know what to do. |
|
@anaconda-issue-bot check |
|
Thank you @bgobbi! |
|
Hi there, thank you for your contribution! This pull request has been automatically locked because it has not had recent activity after being closed. Please open a new issue or pull request if needed. Thanks! |
Current Behavior
When using csh or tcsh only
After using "conda activate env" for the first time issuing "conda env list" adds the conda/bin directory as a prefix to the PATH variable causeing future python execution to use a non-intended python version.
Even worse: every time "conda env list is issued a new "conda/bin" is prefixed to the PATH variable.
The same issue happens when an conda command is executed the results in an error e.g "conda activate non_existing".
Steps to Reproduce
Expected Behavior
"conda env list" or any informational or invalid conda com and should not affect the PATH variable
Cause
This problem is caused by conda.csh line 29
conda/conda/shell/etc/profile.d/conda.csh
Line 29 in aed799b
It modifies the path environment variable. That change is reversed in cases when the
eval statements are executed (line 33,38 or 44) but if an error happens in lines (32, 37 or 43)
or if the "default:" statement is executed the path variable is left unchanged with the altered value.
Solution
The proposed solution stores the value of the Path variable and restores it in case of error or in the default branch of the switch statement when no "eval" statement is executed that would set the PATH variable correctly.