Skip to content

fix(rsync):Use the canonical command for alias completions. - #1748

Closed
cjhxmx wants to merge 1 commit into
scop:mainfrom
cjhxmx:main
Closed

cjhxmx wants to merge 1 commit into
scop:mainfrom
cjhxmx:main

Conversation

@cjhxmx

@cjhxmx cjhxmx commented Oct 13, 2026

Copy link
Copy Markdown

Fixes #1370

$1 can be a user alias, which is not expanded when used as a command name through a parameter expansion. Query the real command instead so that aliases produce the same options.

Fixes scop#1370

$1 can be a user alias, which is not expanded when used as a
command name through a parameter expansion.  Query the real
command instead so that aliases produce the same options.
@akinomyoga

akinomyoga commented Oct 13, 2026 •

Copy link
Copy Markdown
Collaborator

Allowing the user to define the completion for alias rrr=... by complete -F _comp_cmd_rsync rrr isn't an official feature we provide.

There is a reason that we use $1. This PR ignores it.

In addition, even if we should fix this, the fix shouldn't be applied to each command separately in each PR. The change should be applied consistently to all similar completion settings.

@cjhxmx

cjhxmx commented Oct 14, 2026

Copy link
Copy Markdown
Author

Now I know. Thank you for your reply.

@cjhxmx cjhxmx closed this Oct 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

issue with alias on rsync

2 participants