Bug 8371 - tl-config completion depth differs when completing folders and parameters
Summary: tl-config completion depth differs when completing folders and parameters
Status: NEW
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Misc (show other bugs)
Version: trunk
Hardware: PC Unknown
: P2 Normal
Target Milestone: LowPrio
Assignee: Bugzilla mail exporter
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-06-19 15:15 CEST by Tobias
Modified: 2024-09-10 17:41 CEST (History)
1 user (show)

See Also:
Acceptance Criteria:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Tobias cendio 2024-06-19 15:15:34 CEST 
For instance with this entry,
> [/vsmagent/level1/level2/level3/level4/level5/level6/level7/level8]
> 
> testvalue=999
in vsmagent.hconf, attempting to tab complete directories with the --all-entries option as in
> $ tl-config --all-entries /vsmagent/lev
won't complete the full unique directory path in one completion. Instead, one has to press tab for all the levels.

Conversely, with no options given we tab complete for parameters, and this completer only returns a maximum of two levels down for some reason. Attempting to tab complete this line
> $ tl-config /vsmagent/lev
yields
> $ tl-config /vsmagent/level1/level2/
> /vsmagent/level1/level2/         /vsmagent/level1/level2/level3/
where the first suggestion is simply a repetition, and one has to assist the completer into the next level (every two levels).

Unsure if tl-session-param exhibits similar problems with similar completers.

Seen with server build #3632 on RHEL9.
Comment 1 Tobias cendio 2024-06-19 15:39:55 CEST 
Missed when testing tab completion in bug 7867 probably due to lack of sufficiently deep entries while testing.
Comment 2 Samuel Mannehed cendio 2024-06-25 13:39:05 CEST 
It might not be a bug that we only get one folder at a time when completing a unique path with multiple levels. This is how 'ls' behaves when completing folders.
Comment 3 Samuel Mannehed cendio 2024-09-04 14:35:39 CEST 
Attachment 1234 [details] on bug 8384 removes a lot of complexity for the --all-entries, I think the correct way forward is to only complete one level at a time.
Note You need to log in before you can comment on or make changes to this bug.