Bug 1216026

Summary: kitty should require kitty-terminfo
Product: [openSUSE] openSUSE Tumbleweed Reporter: Marcus Rückert <mrueckert>
Component: X11 ApplicationsAssignee: Scott Bradnick <scott.bradnick>
Status: RESOLVED WORKSFORME QA Contact: E-mail List <qa-bugs>
Severity: Normal    
Priority: P5 - None    
Version: Current   
Target Milestone: ---   
Hardware: Other   
OS: Other   
Whiteboard:
Found By: --- Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---

Description Marcus Rückert 2023-10-07 20:26:24 UTC
just to make sure it has the terminfo files when launching.
Comment 1 Scott Bradnick 2023-10-08 03:31:36 UTC
It's a 'recommends', if someone goes out of their way to disable those - not sure how many people do, I don't - then _maybe_ that argument can be made.

$ rpm -q --recommends kitty
kitty-shell-integration
kitty-terminfo
python3-importlib-resources
Comment 2 Marcus Rückert 2023-10-11 13:03:57 UTC
recommends can be disabled. and yes I do. having the proper terminfo file is probably required for kitty to work properly no?

so that should probably be a require.
Comment 3 Scott Bradnick 2023-10-11 14:02:41 UTC
If you chose to disable 'recommends', I feel you've chosen to live with the concept of 'I should check what's recommended and decide if I feel it should be installed' - that's the point of doing it that way, I'd assume.

If 'kitty-terminfo' wasn't mentioned at all, I'd be inclined to simply add it as a 'requires' - but making a change (albeit a simple one) to accomodate those that have disabled recommends doesn't seem necessary, if it _always_ was, then what's the point in the distinction between 'recommends' and 'requires'?

Revisiting the existence of 'kitty-terminfo' - which didn't exist as a separate package until the not-too-distant past (prior 'kitty' was an all-in-one package) - it isn't 'required' if all you want is to use /usr/bin/kitty on the machine you're physically in front of. 'kitty' (the package) includes /usr/lib64/kitty/terminfo/x/xterm-kitty, so functionality shouldn't be an issue. I believe the main point of 'kitty-terminfo' is to provide /usr/share/terminfo/x/xterm-kitty for systems that aren't being used in the 'I'm sitting right in front of it' sense, i.e. ssh'ing into a remote host.
Comment 4 Marcus Rückert 2023-10-12 00:24:14 UTC
this is not where ncurses looks for the file /usr/lib64/kitty/terminfo/x/xterm-kitty

so this can probably be deleted.

you want the requires.
Comment 5 Scott Bradnick 2023-10-12 00:47:19 UTC
(In reply to Marcus Rückert from comment #4)
> this is not where ncurses looks for the file
> /usr/lib64/kitty/terminfo/x/xterm-kitty
> 
> so this can probably be deleted.
> 

Can you provide some type of evidence of this (and its implications, honestly I'm not sure what ncurses directly has to do w/ this RPM for kitty)? If I move /usr/lib64/kitty/terminfo/x/xterm-kitty -> /usr/lib64/kitty/terminfo/x/xterm-kitty.orig, I get this message at the next kitty launch:

warning: Could not set up terminal.
warning: TERM environment variable set to 'xterm-kitty'.
warning: Check that this terminal type is supported on this system.
warning: Using fallback terminal type 'xterm-256color'.

> you want the requires.

In your opinion; I'm not seeing _proof_ why that would be so other than your preference for disabling recommends. I'm not morally opposed to adding it, I just don't see the need. I can have 'kitty' & 'kitty-shell-integration' on the host I'm sitting in front of and just 'kitty-terminfo' on a remote host w/o issue - both hosts report `TERM=xterm-kitty` and function fine.