Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@travishathaway
Copy link
Contributor

@travishathaway travishathaway commented Jan 27, 2025

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 ...

  • Add a file to the news directory (using the template) for the next release's release notes?
  • Add / update necessary tests?
  • Add / update outdated documentation?

@travishathaway travishathaway requested a review from a team as a code owner January 27, 2025 14:07
@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label Jan 27, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Jan 27, 2025

CodSpeed Performance Report

Merging #14517 will not alter performance

Comparing travishathaway:issue-14292 (97e835d) with 25.1.x (5acd0dd)

Summary

✅ 21 untouched benchmarks

@kenodegard kenodegard changed the base branch from main to 25.1.x January 27, 2025 18:10
@kenodegard
Copy link
Contributor

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

@kenodegard kenodegard force-pushed the issue-14292 branch 2 times, most recently from e97762e to ed8692b Compare January 28, 2025 03:54
Comment on lines +4 to +9
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
Copy link
Member

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

@Gravifer
Copy link

When will this ship?

@kenodegard
Copy link
Contributor

@Gravifer packaging and release to main & conda-forge is actively in the works, please refer to the release epic: #14474

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

Labels

cla-signed [bot] added once the contributor has signed the CLA

Projects

Archived in project

5 participants