Skip to content

CLI pager fixes#827

Merged
troglobit merged 5 commits intomainfrom
cli-fixes
Nov 18, 2024
Merged

CLI pager fixes#827
troglobit merged 5 commits intomainfrom
cli-fixes

Conversation

@troglobit
Copy link
Contributor

Description

Pull in most of the upstream klish and klish-plugin-sysrepo changes (we need to fork!), mostly for getting the last known raw mode fixes so we can use an external pager for commands we know need paging.

Checklist

Tick relevant boxes, this PR is-a or has-a:

  • Bugfix
    • Regression tests
    • ChangeLog updates (for next release)
  • Feature
    • YANG model change => revision updated?
    • Regression tests added?
    • ChangeLog updates (for next release)
    • Documentation added?
  • Test changes
    • Checked in changed Readme.adoc (make test-spec)
    • Added new test to group Readme.adoc and yaml file
  • Code style update (formatting, renaming)
  • Refactoring (please detail in commit messages)
  • Build related changes
  • Documentation content changes
    • ChangeLog updated (for major changes)
  • Other (please describe):

This update include more backports from Klish 3.1.0, including, but not
limited to:

 - Syslog fixes
 - Return code fixes
 - Changed library and plugin paths (!)
 - TTY raw mode fixes
 - Use raw mode for interactive commands
 - Allow single quotes (') as alternative to regular quotes for commands

Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
These are basically the same flags as was passed to klish, but now we
use -R instead of -r to make sure only ANSI coloŕ escapes can affect
the behavior of less.

This change will make it posslbe to shop off too long lines and scroll
vertically on really long lines instead for them breaking output.

Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
With interactive mode (out="tty") klish puts the tty in raw mode which
fixes all weird issues reported around paged output in the CLI.

Fixes #659

Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
@troglobit troglobit added this to the Infix v24.11 milestone Nov 17, 2024
@troglobit troglobit requested a review from mattiaswal November 17, 2024 15:23
@troglobit troglobit self-assigned this Nov 17, 2024
@troglobit troglobit linked an issue Nov 17, 2024 that may be closed by this pull request
Copy link
Contributor

@mattiaswal mattiaswal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@troglobit troglobit merged commit 28c5f5e into main Nov 18, 2024
@troglobit troglobit deleted the cli-fixes branch November 18, 2024 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Console: CLI show interfaces does not always show all interfaces

2 participants