Skip to content
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

Fix for "erresc: unknown str ESC]8" stderr spam #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yaaama
Copy link

@yaaama yaaama commented May 14, 2024

This is a single line fix for the endless standard stderr IO spam generated by the fact that st has not implemented the xterm hyperlink extension.

Some context:

  • st does not properly support the hyperlink extension/proposal that some other xterm based terminal emulators support.
  • We currently print an error whenever the specific escape codes are encountered.
  • Due to this, whenever I open a terminal my log files are endlessly flooded with the message: erresc: unknown str ESC]8.

The fix seems to work on multiple different builds of st (I've tested two so far). I've tested the siduck/snazzy-terminal build, and the mdrdotx/st build.

I've been using this patch for two weeks or so and I haven't noticed any problems so I think it is okay.

Here is a short gif of the mayhem:

ezgif-6-56cbccb863

This is a single line fix for the endless standard stderr IO spam
generated by the fact that st has not implemented the xterm hyperlink
extension.

Some context:
- st does not properly support the hyperlink extension/proposal that some other
xterm based terminal emulators support.
- We currently print an error whenever the specific escape
codes are encountered.
- Due to this, whenever I open a terminal my log files are endlessly
flooded with the message: `erresc: unknown str ESC]8`.

The fix seems to work on multiple different builds of st (I've tested
two so far). I've tested the `siduck/snazzy-terminal` build, and the
`mdrdotx/st` build.

I've been using this patch for two weeks or so and I haven't noticed
any problems so I think it is okay.
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.

None yet

1 participant