[v2,2/6] git.c: add a NEED_UNIX_SOCKETS option for built-ins

Message ID	patch-v2-2.6-d6c44402715-20210910T153147Z-avarab@gmail.com (mailing list archive)
State	Superseded
Headers	show Return-Path: <git-owner@kernel.org> From: =?utf-8?b?w4Z2YXIgQXJuZmrDtnLDsCBCamFybWFzb24=?= <avarab@gmail.com> To: git@vger.kernel.org Cc: Junio C Hamano <gitster@pobox.com>, Jeff King <peff@peff.net>, Carlo Arenas <carenas@gmail.com>, Eric Sunshine <sunshine@sunshineco.com>, =?utf-8?b?w4Z2YXIgQXJuZmrDtnLDsCBCamFybWFzb24=?= <avarab@gmail.com> Subject: [PATCH v2 2/6] git.c: add a NEED_UNIX_SOCKETS option for built-ins Date: Fri, 10 Sep 2021 17:38:32 +0200 Message-Id: <patch-v2-2.6-d6c44402715-20210910T153147Z-avarab@gmail.com> In-Reply-To: <cover-v2-0.6-00000000000-20210910T153146Z-avarab@gmail.com> References: <cover-0.2-00000000000-20210901T110917Z-avarab@gmail.com> <cover-v2-0.6-00000000000-20210910T153146Z-avarab@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk
Series	parse-options: properly align continued usage output & related \| expand [v2,0/6] parse-options: properly align continued usage output & related [v2,1/6] test-lib.sh: add a UNIX_SOCKETS prerequisite [v2,2/6] git.c: add a NEED_UNIX_SOCKETS option for built-ins [v2,3/6] parse-options: stop supporting "" in the usagestr array [v2,4/6] built-ins: "properly" align continued usage output [v2,5/6] send-pack: properly use parse_options() API for usage string [v2,6/6] parse-options: properly align continued usage output

diff --git a/builtin.h b/builtin.h index 16ecd5586f0..66713da6a02 100644 --- a/builtin.h +++ b/builtin.h @@ -63,6 +63,12 @@ * more informed decision, e.g., by ignoring `pager.<cmd>` for * certain subcommands. * + * `NEED_UNIX_SOCKETS`: + * + * This built-in will not work if NO_UNIX_SOCKETS is defined. It + * will be recognized for emitting error messages, but won't be + * listed in --list-cmds=builtins. + * * . Add `builtin/foo.o` to `BUILTIN_OBJS` in `Makefile`. * * Additionally, if `foo` is a new command, there are 4 more things to do: diff --git a/builtin/credential-cache--daemon.c b/builtin/credential-cache--daemon.c index 4c6c89ab0de..d9863287a4d 100644 --- a/builtin/credential-cache--daemon.c +++ b/builtin/credential-cache--daemon.c @@ -304,16 +304,7 @@ int cmd_credential_cache_daemon(int argc, const char **argv, const char *prefix) int cmd_credential_cache_daemon(int argc, const char **argv, const char *prefix) { - const char * const usage[] = { - "git credential-cache--daemon [options] <action>", - "", - "credential-cache--daemon is disabled in this build of Git", - NULL - }; - struct option options[] = { OPT_END() }; - - argc = parse_options(argc, argv, prefix, options, usage, 0); - die(_("credential-cache--daemon unavailable; no unix socket support")); + BUG("should not be called under NO_UNIX_SOCKETS"); } #endif /* NO_UNIX_SOCKET */ diff --git a/builtin/credential-cache.c b/builtin/credential-cache.c index e8a74157471..22b49b265bf 100644 --- a/builtin/credential-cache.c +++ b/builtin/credential-cache.c @@ -142,16 +142,7 @@ int cmd_credential_cache(int argc, const char **argv, const char *prefix) int cmd_credential_cache(int argc, const char **argv, const char *prefix) { - const char * const usage[] = { - "git credential-cache [options] <action>", - "", - "credential-cache is disabled in this build of Git", - NULL - }; - struct option options[] = { OPT_END() }; - - argc = parse_options(argc, argv, prefix, options, usage, 0); - die(_("credential-cache unavailable; no unix socket support")); + BUG("should not be called under NO_UNIX_SOCKETS"); } #endif /* NO_UNIX_SOCKETS */ diff --git a/git.c b/git.c index 18bed9a9964..6b0248841db 100644 --- a/git.c +++ b/git.c @@ -17,6 +17,7 @@ #define SUPPORT_SUPER_PREFIX (1<<4) #define DELAY_PAGER_CONFIG (1<<5) #define NO_PARSEOPT (1<<6) /* parse-options is not used */ +#define NEED_UNIX_SOCKETS (1<<7) /* Works unless -DNO_UNIX_SOCKETS */ struct cmd_struct { const char *cmd; @@ -66,6 +67,10 @@ static int list_cmds(const char *spec) struct string_list list = STRING_LIST_INIT_DUP; int i; int nongit; + unsigned int exclude_option = 0; +#ifdef NO_UNIX_SOCKETS + exclude_option |= NEED_UNIX_SOCKETS; +#endif /* * Set up the repository so we can pick up any repo-level config (like @@ -78,7 +83,7 @@ static int list_cmds(const char *spec) int len = sep - spec; if (match_token(spec, len, "builtins")) - list_builtins(&list, 0); + list_builtins(&list, exclude_option); else if (match_token(spec, len, "main")) list_all_main_cmds(&list); else if (match_token(spec, len, "others")) @@ -423,6 +428,10 @@ static int run_builtin(struct cmd_struct *p, int argc, const char **argv) const char *prefix; prefix = NULL; +#ifdef NO_UNIX_SOCKETS + if (p->option & NEED_UNIX_SOCKETS) + die(_("%s is unavailable; there is no UNIX socket support in this build of Git"), p->cmd); +#endif help = argc == 2 && !strcmp(argv[1], "-h"); if (!help) { if (p->option & RUN_SETUP) @@ -513,8 +522,8 @@ static struct cmd_struct commands[] = { { "config", cmd_config, RUN_SETUP_GENTLY | DELAY_PAGER_CONFIG }, { "count-objects", cmd_count_objects, RUN_SETUP }, { "credential", cmd_credential, RUN_SETUP_GENTLY | NO_PARSEOPT }, - { "credential-cache", cmd_credential_cache }, - { "credential-cache--daemon", cmd_credential_cache_daemon }, + { "credential-cache", cmd_credential_cache, NEED_UNIX_SOCKETS }, + { "credential-cache--daemon", cmd_credential_cache_daemon, NEED_UNIX_SOCKETS }, { "credential-store", cmd_credential_store }, { "describe", cmd_describe, RUN_SETUP }, { "diff", cmd_diff, NO_PARSEOPT }, diff --git a/t/t0012-help.sh b/t/t0012-help.sh index 5679e29c624..2d05dde4b90 100755 --- a/t/t0012-help.sh +++ b/t/t0012-help.sh @@ -85,4 +85,14 @@ do ' done <builtins +test_expect_success UNIX_SOCKETS 'builtin list includes NEED_UNIX_SOCKETS under UNIX_SOCKETS' ' + grep ^credential-cache$ builtins && + grep ^credential-cache--daemon$ builtins +' + +test_expect_success !UNIX_SOCKETS 'builtin list excludes NEED_UNIX_SOCKETS under !UNIX_SOCKETS' ' + ! grep ^credential-cache$ builtins && + ! grep ^credential-cache--daemon$ builtins +' + test_done

[v2,2/6] git.c: add a NEED_UNIX_SOCKETS option for built-ins

Commit Message

Comments

Patch