Skip to content
This repository has been archived by the owner on Aug 21, 2023. It is now read-only.

Clicking OK in sets ladishd setting for terminal emulator to x-terminal-emulator even if it is not present #361

Open
nedko opened this issue Jun 1, 2023 · 0 comments

Comments

@nedko
Copy link

nedko commented Jun 1, 2023

Clicking OK in "Configure Claudia" sets ladishd setting for terminal emulator to x-terminal-emulator even if it is not present.
This causes ladish applications not being able start in terminal if x-terminal-emulator executable is not available.

Changing default value that works (if one has xterm) to one that does not work if one does not have dpkg as package manager
does not work on Gentoo at least.

As ladiconf daemon is shared between gladish and Claudia, this bug affects both frontends.

There are two fixes that come to mind

  • The setting should not be set (thus leaving it in a sane default of "xterm") without explicit request from user.
  • When set, the executable should be checked to exist, and if not at least warn the clueless user.
nedko added a commit to LADI/Cadence that referenced this issue Jun 1, 2023
x-terminal-emulator, while good in theory is debian specific.
xterm, OTOH is ubiquitous and minimal.

Fix for falkTX#361
nedko added a commit to LADI/os-gentoo-overlay that referenced this issue Jun 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant