RE: [PATCH v2 15/30] run-command: remove old run_hook_{le,ve}() hook API

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: RE: [PATCH v2 15/30] run-command: remove old run_hook_{le,ve}() hook API
From: Felipe Contreras <felipe.contreras@xxxxxxxxx>
Date: Fri, 25 Jun 2021 13:34:26 -0500
Cc: Junio C Hamano <gitster@xxxxxxxxx>, Emily Shaffer <emilyshaffer@xxxxxxxxxx>, Jeff King <peff@xxxxxxxx>, Taylor Blau <me@xxxxxxxxxxxx>, Felipe Contreras <felipe.contreras@xxxxxxxxx>, Eric Sunshine <sunshine@xxxxxxxxxxxxxx>, "brian m . carlson" <sandals@xxxxxxxxxxxxxxxxxxxx>, Josh Steadmon <steadmon@xxxxxxxxxx>, Jonathan Tan <jonathantanmy@xxxxxxxxxx>, Derrick Stolee <stolee@xxxxxxxxx>, Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx>
In-reply-to: <patch-15.30-477eb2245c3-20210614T101920Z-avarab@gmail.com>
References: <cover-00.31-00000000000-20210528T110515Z-avarab@gmail.com> <cover-00.30-00000000000-20210614T101920Z-avarab@gmail.com> <patch-15.30-477eb2245c3-20210614T101920Z-avarab@gmail.com>

Ævar Arnfjörð Bjarmason wrote:
> From: Emily Shaffer <emilyshaffer@xxxxxxxxxx>
> 
> The new hook.h library has replaced all run-command.h hook-related
> functionality. So let's delete this dead code.

Ahhh, much less mental load of the reviewer ;)

-- 
Felipe Contreras

References:
- [PATCH 00/31] minimal restart of "config-based-hooks"
  - From: Ævar Arnfjörð Bjarmason
- [PATCH v2 00/30] Minimal restart of "config-based-hooks"
  - From: Ævar Arnfjörð Bjarmason
- [PATCH v2 15/30] run-command: remove old run_hook_{le,ve}() hook API
  - From: Ævar Arnfjörð Bjarmason

Prev by Date: RE: [PATCH v2 13/30] read-cache: convert post-index-change hook to use config
Next by Date: Re: [PATCH v2 01/30] hook: add 'run' subcommand
Previous by thread: [PATCH v2 15/30] run-command: remove old run_hook_{le,ve}() hook API
Next by thread: [PATCH v2 29/30] hooks: fix a TOCTOU in "did we run a hook?" heuristic
Index(es):
- Date
- Thread

[Index of Archives] [Linux Kernel Development] [Gcc Help] [IETF Annouce] [DCCP] [Netdev] [Networking] [Security] [V4L] [Bugtraq] [Yosemite] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux SCSI] [Fedora Users]