Skip to content

incus/usage: Fix edge case#3080

Merged
stgraber merged 1 commit intolxc:mainfrom
bensmrs:fix-diagnosis
Mar 24, 2026
Merged

incus/usage: Fix edge case#3080
stgraber merged 1 commit intolxc:mainfrom
bensmrs:fix-diagnosis

Conversation

@bensmrs
Copy link
Copy Markdown
Contributor

@bensmrs bensmrs commented Mar 24, 2026

This fixes an off-by-n error when diagnosing incomplete RTL-parsed commands. This is a pretty specific edge case.

Was:

Usage: incus file pull [<remote>:]<instance>/<path>... <target path>
                       ┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅ └──────────────┬──────────────┘
  ┌───────────────────────────────────────────────────────────────────┘
  └─ “trixie/foo”

Now:

Usage: incus file pull [<remote>:]<instance>/<path>... <target path>
                       ┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅┅ └─────┬─────┘
  ┌──────────────────────────────────────────────────────────┘
  └─ “trixie/foo

@bensmrs
incus/usage: Fix edge case
26c1c65 
This fixes an off-by-n error when diagnosing incomplete RTL-parsed
commands.

Signed-off-by: Benjamin Somers <[email protected]>
@bensmrs bensmrs requested a review from stgraber as a code owner March 24, 2026 22:01
@stgraber stgraber enabled auto-merge March 24, 2026 22:12
@stgraber stgraber merged commit 49336cd into lxc:main Mar 24, 2026
106 of 108 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stgraber stgraber stgraber approved these changes

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bensmrs @stgraber