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

Strange Logcat output #178

Open
CreatorB opened this issue Feb 21, 2022 · 4 comments
Open

Strange Logcat output #178

CreatorB opened this issue Feb 21, 2022 · 4 comments

Comments

@CreatorB
Copy link

image

I'm using latest lubuntu 21.10, 5.13.0-28-generic, x86_64 GNU/Linux

also have been followed to install coreutils on README.md

@JakeWharton
Copy link
Owner

Those are the ANSI control sequences which cause color to be displayed. It's not clear why it works on some lines but not others. Probably some obscure Python quirk I'm not aware of.

@ozziefallick
Copy link

I suspect that this would be fixed by the change in #97 (comment). I'm going to confirm that it's backwards-compatible with python2, and if so, send a PR.

ozziefallick added a commit to ozziefallick/pidcat that referenced this issue Mar 8, 2022
Removed utf-8 encoding step as recommended in JakeWharton#97 (comment). This is backwards-compatible with Python 2.
ozziefallick added a commit to ozziefallick/pidcat that referenced this issue Mar 8, 2022
Fixed color encoding problem causing Issue JakeWharton#178
ozziefallick added a commit to ozziefallick/pidcat that referenced this issue Mar 8, 2022
Version increases to 2.1.1 to cover fixes for Issues JakeWharton#147 and JakeWharton#178.
@kmansoft
Copy link

Same here on Ubuntu 22.04, and the fix in #97 helped.

Maybe it's also time to update the #! to python3 :)

1fexd pushed a commit to 1fexd/pidcat that referenced this issue Apr 7, 2024
Removed utf-8 encoding step as recommended in JakeWharton#97 (comment). This is backwards-compatible with Python 2.
1fexd pushed a commit to 1fexd/pidcat that referenced this issue Apr 7, 2024
1fexd pushed a commit to 1fexd/pidcat that referenced this issue Apr 7, 2024
Version increases to 2.1.1 to cover fixes for Issues JakeWharton#147 and JakeWharton#178.
@pforhan
Copy link

pforhan commented Apr 23, 2024

so this change was to fix a unicode issue in the past... probably what's needed is to utf-encode the actual logcat content, but not the formatting pieces

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

No branches or pull requests

5 participants