Terminal cannot be opened after adjusting the language #22079

Closed
opened 2026-01-31 08:02:52 +00:00 by claunia · 10 comments
Owner

Originally created by @Goo-aw233 on GitHub (Aug 10, 2024).

Windows Terminal version

1.22.2221.0

Windows build number

10.0.26257.5000

Other Software

No response

Steps to reproduce

Open Settings > Appearance > Language > 中文(简体) > Save
Afterwards, close the Settings tab and reopen it. Changing any function and clicking Save will cause the program to crash.
Any language can cause this problem.

Expected Behavior

No response

Actual Behavior

After saving the above changes, Terminal Canary will crash, and after stopping, repairing, or resetting the program, it will not be able to open the program again, and will need to be uninstalled and reinstalled.

Terminal Preview 1.21.1772.0 does not have this problem.

In addition, this is the DMP that is automatically generated after the program crashes.

WindowsTerminal.exe.18664.dmp

Originally created by @Goo-aw233 on GitHub (Aug 10, 2024). ### Windows Terminal version 1.22.2221.0 ### Windows build number 10.0.26257.5000 ### Other Software _No response_ ### Steps to reproduce Open Settings > Appearance > Language > 中文(简体) > Save Afterwards, close the Settings tab and reopen it. Changing any function and clicking Save will cause the program to crash. Any language can cause this problem. ### Expected Behavior _No response_ ### Actual Behavior After saving the above changes, Terminal Canary will crash, and after stopping, repairing, or resetting the program, it will not be able to open the program again, and will need to be uninstalled and reinstalled. Terminal Preview 1.21.1772.0 does not have this problem. In addition, this is the DMP that is automatically generated after the program crashes. [WindowsTerminal.exe.18664.dmp](https://github.com/user-attachments/files/16570297/WindowsTerminal.exe.18664.dmp)
Author
Owner

@lhecker commented on GitHub (Aug 12, 2024):

@DHowett How would you like your bad MSVC news today?

@lhecker commented on GitHub (Aug 12, 2024): @DHowett How would you like your bad MSVC news today?
Author
Owner

@DHowett commented on GitHub (Aug 12, 2024):

Preferably “actionable so we stop getting knocked over by compiler bugs every eight minutes”

@DHowett commented on GitHub (Aug 12, 2024): Preferably “actionable so we stop getting knocked over by compiler bugs every eight minutes”
Author
Owner

@zadjii-msft commented on GitHub (Aug 20, 2024):

@lhecker scale of 1-10, how bad is this? This sounds like it's ship blocking

@zadjii-msft commented on GitHub (Aug 20, 2024): @lhecker scale of 1-10, how bad is this? This sounds like it's ship blocking
Author
Owner

@lhecker commented on GitHub (Aug 20, 2024):

I would try disabling PGO as a test. It's possible that it's because I'm using VS 17.11 (and now .12), but this issue doesn't repro for me in local Release builds. Alternatively, we could find a way to get a OneBranch 17.11 image.

@lhecker commented on GitHub (Aug 20, 2024): I would try disabling PGO as a test. It's possible that it's because I'm using VS 17.11 (and now .12), but this issue doesn't repro for me in local Release builds. Alternatively, we could find a way to get a OneBranch 17.11 image.
Author
Owner

@lhecker commented on GitHub (Aug 21, 2024):

Disabling PGO worked: The crash is now fixed in 1.22.2334.0.

@lhecker commented on GitHub (Aug 21, 2024): Disabling PGO worked: The crash is now fixed in 1.22.2334.0.
Author
Owner

@zadjii-msft commented on GitHub (Aug 21, 2024):

Okay, so what's the next step here? We can't disable PGO for a real release. Do we just need to roll back the PGO CI to an older VS version?

@zadjii-msft commented on GitHub (Aug 21, 2024): Okay, so what's the next step here? We can't disable PGO for a real release. Do we just need to roll back the PGO CI to an older VS version?
Author
Owner

@DHowett commented on GitHub (Aug 21, 2024):

We can't disable PGO for a real release.

Why not?

Leonard had this to say:

Is there a reason why PGO must be enabled? I thought it's just an optimization.


FWIW, the bug is not in the generation of PGO data (which as of today is on VS 17.11) but in the consumption of it (which is in a onebranch pipeline that is stuck on VS 17.10 -- the known bad compiler.)

@DHowett commented on GitHub (Aug 21, 2024): > We can't disable PGO for a real release. Why not? Leonard had this to say: > Is there a reason why PGO must be enabled? I thought it's just an optimization. --- FWIW, the bug is not in the generation of PGO data (which as of today is on VS 17.11) but in the consumption of it (which is in a onebranch pipeline that is stuck on VS 17.10 -- the known bad compiler.)
Author
Owner

@lhecker commented on GitHub (Aug 21, 2024):

Disabling PGO caused a ~13% regression in my testing. It's still magnitudes faster than what it used to be though, so I think we'll be fine even if we never enabled it again. (I think we should though, because PGO is technically really nice.)

@lhecker commented on GitHub (Aug 21, 2024): Disabling PGO caused a ~13% regression in my testing. It's still magnitudes faster than what it used to be though, so I think we'll be fine even if we never enabled it again. (I think we should though, because PGO is technically really nice.)
Author
Owner

@carlos-zamora commented on GitHub (Aug 21, 2024):

Since this is now fixed on Canary, and we have a plan moving forward, I'm closing this. Thanks all!

@carlos-zamora commented on GitHub (Aug 21, 2024): Since this is now fixed on Canary, and we have a plan moving forward, I'm closing this. Thanks all!
Author
Owner

@Andarwinux commented on GitHub (Aug 22, 2024):

Maybe consider trying to use clang-cl.

@Andarwinux commented on GitHub (Aug 22, 2024): Maybe consider trying to use clang-cl.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#22079