git/list[1] front-page[2] threads[3] people[4] search[5] about
wed 2026-10-07 16:50 UTC

Re: [PATCH v2 10/10] hook: show disabled hooks in "git hook list"

From
Patrick Steinhardt <ps@pks.im>
Date
Mar 24, 2026, 08:38 UTC
Message-ID
<acJNdKyz-ISsq4fJ@pks.im>
In-Reply-To
<20260320115211.177351-11-adrian.ratiu@collabora.com>
On Fri, Mar 20, 2026 at 01:52:11PM +0200, Adrian Ratiu wrote:
Show 12 quoted lines
> diff --git a/hook.c b/hook.c
> index aa08c38c27..0e09b9a2bb 100644
> --- a/hook.c
> +++ b/hook.c
> @@ -119,6 +119,7 @@ static void list_hooks_add_default(struct repository *r, const char *hookname,
>  struct hook_config_cache_entry {
>  	char *command;
>  	enum config_scope scope;
> +	unsigned int disabled:1;
>  };
>  
>  /*

Curious, this is now a single-bit int. I still would have expected a proper bool here :)

Show 11 quoted lines
> diff --git a/hook.h b/hook.h
> index 92e9faf9bb..7c8c3d471e 100644
> --- a/hook.h
> +++ b/hook.h
> @@ -31,6 +31,7 @@ struct hook {
>  			const char *friendly_name;
>  			const char *command;
>  			enum config_scope scope;
> +			unsigned int disabled:1;
>  		} configured;
>  	} u;
Same here, I would expect a proper bool.
Patrick
Previous: Patrick SteinhardtNext: Junio C Hamano
Message 47 of 71 in “config-hook cleanups and two small 'git hook list' features”
  1. 00/10 config-hook cleanups and two small 'git hook list' featuresAdrian Ratiu, Mar 9, 2026
  2. 01/10 hook: move unsorted_string_list_remove() to string-list.[ch]Adrian Ratiu, Mar 9, 2026
  3. 02/10 hook: fix minor style issuesAdrian Ratiu, Mar 9, 2026
  4. 03/10 hook: rename cb_data_free/alloc -> hook_data_free/allocAdrian Ratiu, Mar 9, 2026
  5. 04/10 hook: detect & emit two more bugsAdrian Ratiu, Mar 9, 2026
  6. 05/10 hook: replace hook_list_clear() -> string_list_clear_func()Adrian Ratiu, Mar 9, 2026
  7. 06/10 hook: make consistent use of friendly-name in docsAdrian Ratiu, Mar 9, 2026
  8. 08/10 hook: refactor hook_config_cache from strmap to named structAdrian Ratiu, Mar 9, 2026
  9. 07/10 t1800: add test to verify hook execution orderingAdrian Ratiu, Mar 9, 2026
  10. 09/10 hook: show config scope in git hook listAdrian Ratiu, Mar 9, 2026
  11. 10/10 hook: show disabled hooks in "git hook list"Adrian Ratiu, Mar 9, 2026
  12. Eric SunshineMar 9, 2026
  13. Eric SunshineMar 9, 2026
  14. Junio C HamanoMar 9, 2026
  15. Junio C HamanoMar 9, 2026
  16. Junio C HamanoMar 9, 2026
  17. Junio C HamanoMar 9, 2026
  18. Adrian RatiuMar 10, 2026
  19. Adrian RatiuMar 10, 2026
  20. Adrian RatiuMar 10, 2026
  21. Adrian RatiuMar 10, 2026
  22. SZEDER GáborMar 10, 2026
  23. Patrick SteinhardtMar 11, 2026
  24. Patrick SteinhardtMar 11, 2026
  25. Patrick SteinhardtMar 11, 2026
  26. Adrian RatiuMar 11, 2026
  27. Adrian RatiuMar 11, 2026
  28. Adrian RatiuMar 11, 2026
  29. Adrian RatiuMar 11, 2026
  30. Patrick SteinhardtMar 11, 2026
  31. 00/10 config-hook cleanups and two small 'git hook list' featuresAdrian Ratiu, Mar 20, 2026
  32. 01/10 hook: move unsorted_string_list_remove() to string-list.[ch]Adrian Ratiu, Mar 20, 2026
  33. 03/10 hook: rename cb_data_free/alloc -> hook_data_free/allocAdrian Ratiu, Mar 20, 2026
  34. 04/10 hook: detect & emit two more bugsAdrian Ratiu, Mar 20, 2026
  35. 05/10 hook: replace hook_list_clear() -> string_list_clear_func()Adrian Ratiu, Mar 20, 2026
  36. 06/10 hook: make consistent use of friendly-name in docsAdrian Ratiu, Mar 20, 2026
  37. 07/10 t1800: add test to verify hook execution orderingAdrian Ratiu, Mar 20, 2026
  38. 08/10 hook: introduce hook_config_cache_entry for per-hook dataAdrian Ratiu, Mar 20, 2026
  39. 02/10 hook: fix minor style issuesAdrian Ratiu, Mar 20, 2026
  40. 09/10 hook: show config scope in git hook listAdrian Ratiu, Mar 20, 2026
  41. 10/10 hook: show disabled hooks in "git hook list"Adrian Ratiu, Mar 20, 2026
  42. Junio C HamanoMar 23, 2026
  43. Patrick SteinhardtMar 24, 2026
  44. Patrick SteinhardtMar 24, 2026
  45. Patrick SteinhardtMar 24, 2026
  46. Patrick SteinhardtMar 24, 2026
  47. Patrick SteinhardtMar 24, 2026
  48. Junio C HamanoMar 24, 2026
  49. Adrian RatiuMar 24, 2026
  50. Adrian RatiuMar 24, 2026
  51. Adrian RatiuMar 24, 2026
  52. Adrian RatiuMar 24, 2026
  53. Patrick SteinhardtMar 25, 2026
  54. Adrian RatiuMar 25, 2026
  55. 00/12 config-hook cleanups and three small git-hook featuresAdrian Ratiu, Mar 25, 2026
  56. 01/12 hook: move unsorted_string_list_remove() to string-list.[ch]Adrian Ratiu, Mar 25, 2026
  57. 02/12 builtin/receive-pack: properly init receive_hook strbufAdrian Ratiu, Mar 25, 2026
  58. 04/12 hook: rename cb_data_free/alloc -> hook_data_free/allocAdrian Ratiu, Mar 25, 2026
  59. 03/12 hook: fix minor style issuesAdrian Ratiu, Mar 25, 2026
  60. 05/12 hook: detect & emit two more bugsAdrian Ratiu, Mar 25, 2026
  61. 06/12 hook: replace hook_list_clear() -> string_list_clear_func()Adrian Ratiu, Mar 25, 2026
  62. 07/12 hook: make consistent use of friendly-name in docsAdrian Ratiu, Mar 25, 2026
  63. 08/12 t1800: add test to verify hook execution orderingAdrian Ratiu, Mar 25, 2026
  64. 09/12 hook: introduce hook_config_cache_entry for per-hook dataAdrian Ratiu, Mar 25, 2026
  65. 10/12 hook: show config scope in git hook listAdrian Ratiu, Mar 25, 2026
  66. 11/12 hook: show disabled hooks in "git hook list"Adrian Ratiu, Mar 25, 2026
  67. 12/12 hook: reject unknown hook names in git-hook(1)Adrian Ratiu, Mar 25, 2026
  68. Junio C HamanoMar 25, 2026
  69. Adrian RatiuMar 26, 2026
  70. Patrick SteinhardtMar 27, 2026
  71. Junio C HamanoMar 27, 2026

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.