Mirrors/fish-shell
2
0
Fork 0
mirror of https://github.com/fish-shell/fish-shell synced 2024-12-27 13:23:09 +00:00
Code Issues Projects Releases Packages Wiki Activity

complete: Print better error for -x -F

Browse source 
-x is a cheesy shortcut for `-rf`, so it conflicts with `-F`.

Fixes #8818.
This commit is contained in:
Fabian Homborg 2022-05-26 14:15:28 +02:00
parent be781e9144
commit 65b9c26fb4
2 changed files with 16 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
src/builtins/complete.cpp
View file

@ -170,6 +170,8 @@ maybe_t<int> builtin_complete(parser_t &parser, io_streams_t &streams, const wch
{L"escape", no_argument, nullptr, opt_escape},
{}};
bool have_x = false;
int opt;
wgetopter_t w;
while ((opt = w.wgetopt_long(argc, argv, short_options, long_options, nullptr)) != -1) {
@ -177,6 +179,8 @@ maybe_t<int> builtin_complete(parser_t &parser, io_streams_t &streams, const wch
case 'x': {
result_mode.no_files = true;
result_mode.requires_param = true;
// Needed to print an error later.
have_x = true;
break;
}
case 'f': {
@ -301,8 +305,15 @@ maybe_t<int> builtin_complete(parser_t &parser, io_streams_t &streams, const wch
}
if (result_mode.no_files && result_mode.force_files) {
streams.err.append_format(BUILTIN_ERR_COMBO2, L"complete",
L"'--no-files' and '--force-files'");
if (!have_x) {
streams.err.append_format(BUILTIN_ERR_COMBO2, L"complete",
L"'--no-files' and '--force-files'");
} else {
// The reason for us not wanting files is `-x`,
// which is short for `-rf`.
streams.err.append_format(BUILTIN_ERR_COMBO2, L"complete",
L"'--exclusive' and '--force-files'");
}
return STATUS_INVALID_ARGS;
}

3
tests/checks/complete.fish
View file

@ -463,3 +463,6 @@ complete -C"a=1 b=2 cmd_with_fancy_completion "
# CHECK: 1
complete -C"a=1 b=2 cmd_with_fancy_completion 1 "
# CHECK: 2
complete -c thing -x -F
# CHECKERR: complete: invalid option combination, '--exclusive' and '--force-files'