Skip to content

Revert "Implement OSC 7 for setting the CWD (#20019)"#20116

Merged
DHowett merged 1 commit intorelease-1.25from
dev/duhowett/revert-osc-7-125
Apr 15, 2026
Merged

Revert "Implement OSC 7 for setting the CWD (#20019)"#20116
DHowett merged 1 commit intorelease-1.25from
dev/duhowett/revert-osc-7-125

Conversation

@DHowett
Copy link
Copy Markdown
Member

@DHowett DHowett commented Apr 14, 2026

This reverts commit 7f5185e.

I have some new concerns about the fix in #20094, so I'm considering reverting OSC 7 from 1.25.

Refs #20108

@DHowett
Copy link
Copy Markdown
Member Author

DHowett commented Apr 14, 2026

#20094 (comment)

Copy link
Copy Markdown
Member

@carlos-zamora carlos-zamora left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clean revert. Hopefully we can figure out a way to implement it still. Thanks to you and Leonard for working on this :)

Comment on lines +868 to +869
// TODO: Add support for OSC 7 = CWD sequences?
// In GH#8214 it was decided that it's a bad idea due to WSL compat.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

funny that we're reverting it due to WSL compat 😆

@DHowett DHowett merged commit 88a236c into release-1.25 Apr 15, 2026
12 checks passed
@DHowett DHowett deleted the dev/duhowett/revert-osc-7-125 branch April 15, 2026 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants