The "$prefix" variable has not been set since b3c5f5cb048 (submodule:
move core cmd_update() logic to C, 2022-03-15), so we'd never pass the
--super-prefix option to "git submodule--helper", and can therefore
remove the handling of it from builtin/submodule--helper.c as well.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx>
---
builtin/submodule--helper.c | 53 +++++++++++++++----------------------
1 file changed, 22 insertions(+), 31 deletions(-)
diff --git a/builtin/submodule--helper.c b/builtin/submodule--helper.c
index c6c2ba1b6dc..70693205764 100644
--- a/builtin/submodule--helper.c
+++ b/builtin/submodule--helper.c
@@ -3364,35 +3364,32 @@ static int module_add(int argc, const char **argv, const char *prefix)
return 0;
}
-#define SUPPORT_SUPER_PREFIX (1<<0)
-
struct cmd_struct {
const char *cmd;
int (*fn)(int, const char **, const char *);
- unsigned option;
};
static struct cmd_struct commands[] = {
- {"list", module_list, 0},
- {"name", module_name, 0},
- {"clone", module_clone, 0},
- {"add", module_add, SUPPORT_SUPER_PREFIX},
- {"update", module_update, 0},
- {"resolve-relative-url-test", resolve_relative_url_test, 0},
- {"foreach", module_foreach, SUPPORT_SUPER_PREFIX},
- {"init", module_init, SUPPORT_SUPER_PREFIX},
- {"status", module_status, SUPPORT_SUPER_PREFIX},
- {"sync", module_sync, SUPPORT_SUPER_PREFIX},
- {"deinit", module_deinit, 0},
- {"summary", module_summary, SUPPORT_SUPER_PREFIX},
- {"push-check", push_check, 0},
- {"absorb-git-dirs", absorb_git_dirs, SUPPORT_SUPER_PREFIX},
- {"is-active", is_active, 0},
- {"check-name", check_name, 0},
- {"config", module_config, 0},
- {"set-url", module_set_url, 0},
- {"set-branch", module_set_branch, 0},
- {"create-branch", module_create_branch, 0},
+ {"list", module_list},
+ {"name", module_name},
+ {"clone", module_clone},
+ {"add", module_add},
+ {"update", module_update},
+ {"resolve-relative-url-test", resolve_relative_url_test},
+ {"foreach", module_foreach},
+ {"init", module_init},
+ {"status", module_status},
+ {"sync", module_sync},
+ {"deinit", module_deinit},
+ {"summary", module_summary},
+ {"push-check", push_check},
+ {"absorb-git-dirs", absorb_git_dirs},
+ {"is-active", is_active},
+ {"check-name", check_name},
+ {"config", module_config},
+ {"set-url", module_set_url},
+ {"set-branch", module_set_branch},
+ {"create-branch", module_create_branch},
};
int cmd_submodule__helper(int argc, const char **argv, const char *prefix)
@@ -3401,15 +3398,9 @@ int cmd_submodule__helper(int argc, const char **argv, const char *prefix)
if (argc < 2 || !strcmp(argv[1], "-h"))
usage("git submodule--helper <command>");
- for (i = 0; i < ARRAY_SIZE(commands); i++) {
- if (!strcmp(argv[1], commands[i].cmd)) {
- if (get_super_prefix() &&
- !(commands[i].option & SUPPORT_SUPER_PREFIX))
- die(_("%s doesn't support --super-prefix"),
- commands[i].cmd);
+ for (i = 0; i < ARRAY_SIZE(commands); i++)
+ if (!strcmp(argv[1], commands[i].cmd))
return commands[i].fn(argc - 1, argv + 1, prefix);
- }
- }
die(_("'%s' is not a valid submodule--helper "
"subcommand"), argv[1]);
--
2.36.1.1178.gb5b1747c546
