Subject: Re: [PATCH v3 1/6] credential-cache{,--daemon}: don't build under NO_UNIX_SOCKETS

From: Junio C Hamano <[email-masked]>
Date: Sun, 12 Sep 2021 14:48:27 -0700

Ævar Arnfjörð Bjarmason  <[email-masked]> writes:

> @@ -1693,6 +1695,7 @@ ifdef NO_INET_PTON
>  endif
>  ifdef NO_UNIX_SOCKETS
>  	BASIC_CFLAGS += -DNO_UNIX_SOCKETS
> +	EXCLUDED_PROGRAMS += git-credential-cache git-credential-cache--daemon

Yah, I forgot about EXCLUDED_PROGRAMS ;-)  

Sounds like a sensible direction to go.

> diff --git a/git.c b/git.c
> index 18bed9a9964..7c696e06ecf 100644
> --- a/git.c
> +++ b/git.c
> @@ -513,8 +513,10 @@ 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 },
> +#ifndef NO_UNIX_SOCKETS
>  	{ "credential-cache", cmd_credential_cache },
>  	{ "credential-cache--daemon", cmd_credential_cache_daemon },
> +#endif
>  	{ "credential-store", cmd_credential_store },
>  	{ "describe", cmd_describe, RUN_SETUP },
>  	{ "diff", cmd_diff, NO_PARSEOPT },

OK.  Looks good.