The bug was related to trying to split a string with unstript spaces in
the action parameter. It resulted in a unbound error as there was no
previous raise when getting the key from args_funcs. The fix is twofold,
it makes sure that we raise the keyerror if we're trying to grab a non
existing key and we're stripping the spaces from the action before
splitting.
This should fix both the unbound error and the not being able to resolve
correct actions.
Fixes `new_os_window`:
scancode: 0x39 release: 0 clean_sym: n composed_sym: N mods: ctrl+shift glfw_key: N
Traceback (most recent call last):
File "…/Vcs/kitty/kitty/boss.py", line 431, in dispatch_special_key
return self.dispatch_action(key_action)
File "…/Vcs/kitty/kitty/boss.py", line 487, in dispatch_action
passthrough = f(*key_action.args)
File "…/Vcs/kitty/kitty/boss.py", line 199, in new_os_window
self._new_os_window(args)
File "…/Vcs/kitty/kitty/boss.py", line 195, in _new_os_window
startup_session = create_session(self.opts, special_window=sw, cwd_from=cwd_from)
File "…/Vcs/kitty/kitty/session.py", line 127, in create_session
if args.hold:
Thank you for developing Kitty.
Currently kitty is not available on the list of "alternatives" for a
terminal emulator on Debian GNU/Linux and derivatives. This is because
Debian Policy states that packages that provide a terminal emulator must
provide, inter alia, support the command-line option -T title, which
creates a new terminal window with the specified window title [0].
This PR therefore adds -T as an alias for --title, thus bringing kitty into
compliance and allowing cleaner integration into the Debian desktop.
This issue was originally filed as #900704.
[0] https://www.debian.org/doc/debian-policy/#packages-providing-a-terminal-emulator
[1] https://bugs.debian.org/900704
Signed-off-by: Chris Lamb <lamby@debian.org>
