-
Notifications
You must be signed in to change notification settings - Fork 2k
Fixes PowerShell 7.5.0 environment activation #14517
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
Conversation
CodSpeed Performance ReportMerging #14517 will not alter performanceComparing Summary
|
91b9bba to
bbea40b
Compare
bbea40b to
dfbf566
Compare
cad7310 to
22ad933
Compare
|
Confirming that installing PowerShell 7.5.0 and parameterizing the applicable tests replicates the issue: https://github.com/conda/conda/actions/runs/12999890671/job/36256153948?pr=14517 |
e97762e to
ed8692b
Compare
ed8692b to
97e835d
Compare
| if [ -n "${_CE_M:+x}" ] && [ -n "${_CE_CONDA:+x}" ]; then | ||
| # only use _CE_M/_CE_CONDA when defined to avoid issues when run with `set -u` | ||
| "$CONDA_EXE" $_CE_M $_CE_CONDA "$@" | ||
| else | ||
| "$CONDA_EXE" "$@" | ||
| fi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This makes sense I think, looks awfully something like I've seen before in conda, can't put my finger on it though
|
When will this ship? |
Description
Fixes: #14514
Fixes: #14292
This is a suggested proposal for addressing the two issue above. I have tested this locally so far with PowerShell 7.5.0 and it appears to be working. Needs more extensive testing though.
Checklist - did you ...
newsdirectory (using the template) for the next release's release notes?Add / update outdated documentation?