-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Default pager breaks colours in Windows Terminal #860
Comments
Thank you for the detailed bug report! We can keep this open until it has been fixed in the Chocolatey package. Apart from that, I don't think there is anything we can do here except to suggest an (obvious) workaround, which would be to put # Disable paging
--paging=never |
less 557 has been recently released as a beta: http://greenwoodsoftware.com/less/download.html Chocolatey:
Scoop:
|
In the meantime, @gwsw released |
@sharkdp |
I have built https://github.com/jftuga/less-Windows As a workaround, I have had good luck with really old versions of
Please note that this workaround is unnecessary for binary I am distributing in my GitHub repo. |
I updated my binary version of |
@jftuga Thank you very much for verifying this! In this case, we only need to wait for the new releases to make it into Chocolatey/Scoop. |
In addition to the Chocolatey information, under the Paging section, you could also link to my repo. This could possibly be a resolution to this issue. |
@jftuga I'm using your builds, thanks! |
Oh, I missed the reference to the comment section of the Chocolatey package. That is unfortunate (not the fact that
I'd be okay with that, but I would much rather like this to be solved on the Chocolatey side. Maybe they could use the binary (binaries) from your repo? |
I have messaged the Chocolatey |
@jftuga Instead of (or in addition to) Chocolatey, can you please update Scoop? https://github.com/ScoopInstaller/Main/blob/master/bucket/less.json |
Chocolatey is now using my version of less: |
@jftuga Yay! Would you mind doing Scoop too? I (& many I know) avoid Chocolatey. |
@ofek I made a PR for scoop |
@jftuga Awesome work, thank you very much! Can someone please confirm that the original bug has been fixed when freshly installing |
I can confirm that this is fixed with |
@jftuga Thank you!!! |
This has been merged as well 🎉. If anyone can confirm that everything works as expected when installing via Scoop, we can finally close this ticket. |
Anyone here using the Scoop-installed version of |
@sharkdp Yes I have |
Ok, I'm going to close this for now. Let me know if this should re-appear in some setting. Thank you all for your help. |
Nvm, I wasn't running the latest version of less. |
Hi, I've run into a bug in
less
that effectively breaksbat
in the new Windows Terminal. I appreciate that this is notbat
's fault, but I'm reporting it here to save other people the investigation time.Bug
When calling
bat
in PowerShell in Windows Terminal, it breaks subsequent colouring in the terminal and makes it unusable (at least with my colour scheme) until the terminal is relaunched:Details
This occurs for me in PowerShell (7 and 5.1) in Windows Terminal, it does not occur in the old Windows PowerShell application.
I believe this is due to a bug in version 551 of the
less
pager, which is installed by default with thebat
Chocolatey package. The issue has been fixed inless
source, but it's not available via Windows package managers yet.Short-term workaround
I was able to fix the issue by compiling version 554 of
less
, using the MinGW compilation instructions here.Long-term fix
Wait for a newer version of
less
to be released via Chocolatey and Scoop.What version of
bat
are you using?bat 0.12.1 and less 551
How did you install
bat
?Chocolatey
Windows 10 1909
The text was updated successfully, but these errors were encountered: