search

rgwood / systemctl-tui

A fast, simple TUI for interacting with systemd services and their logs
MIT License
235 stars 5 forks source link

Broken TUI under WSL2 + `systemctl: invalid option -- 'P'` #17

Closed fadeevab closed 4 months ago

fadeevab commented 4 months ago

systemctl: invalid option -- 'P' makes a noise in the TUI rendering. Scrolling is affected.

image

rgwood commented 4 months ago

Thanks for the bug report. I haven’t observed this on WSL. What distro (name+version) are you using?

rgwood commented 4 months ago

Also, what version of systemctl-tui?

honnorat commented 4 months ago

Same problem here with version 0.3.4 on Ubuntu 20.04.

fadeevab commented 4 months ago

@rgwood Ubuntu 20.04, WSL2, systemctl-tui 0.3.4

$ systemctl --version
systemd 245 (245.4-4ubuntu3.22)
+PAM +AUDIT +SELINUX +IMA +APPARMOR +SMACK +SYSVINIT +UTMP +LIBCRYPTSETUP +GCRYPT +GNUTLS +ACL +XZ +LZ4 +SECCOMP +BLKID +ELFUTILS +KMOD +IDN2 -IDN +PCRE2 default-hierarchy=hybrid
rgwood commented 4 months ago

Thanks! I think there are a few issues here:

  1. I'm using a systemctl CLI flag that is not present on older versions of systemd (it works on Ubuntu 22.04, systemd v249)
  2. I'm not handling errors well in this particular path.

Not much I can do about 1 but I should be able to fix up 2. I'm a little busy this week+weekend but will aim to fix this up next week.

kranurag7 commented 4 months ago

Please take your time @rgwood

I started using it today and love the project so far.

Adding my details from Fedora system attached to an external monitor.

Details

![image](https://github.com/rgwood/systemctl-tui/assets/81210977/9e780298-4bcb-45e5-a4a9-c076970fdcd3)

rgwood commented 4 months ago

Fixed in v0.3.5.