diff mbox series

[v5,03/36] Makefile: remove an out-of-date comment

Message ID	patch-v5-03.36-ffef1d3257e-20210902T125110Z-avarab@gmail.com (mailing list archive)
State	New, archived
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>, Emily Shaffer <emilyshaffer@google.com>, Jeff King <peff@peff.net>, Taylor Blau <me@ttaylorr.com>, Felipe Contreras <felipe.contreras@gmail.com>, Eric Sunshine <sunshine@sunshineco.com>, "brian m . carlson" <sandals@crustytoothpaste.net>, Josh Steadmon <steadmon@google.com>, Jonathan Tan <jonathantanmy@google.com>, Derrick Stolee <stolee@gmail.com>, =?utf-8?b?w4Z2YXIgQXJuZmrDtnLDsCBCamFy?= =?utf-8?b?bWFzb24=?= <avarab@gmail.com> Subject: [PATCH v5 03/36] Makefile: remove an out-of-date comment Date: Thu, 2 Sep 2021 15:11:03 +0200 Message-Id: <patch-v5-03.36-ffef1d3257e-20210902T125110Z-avarab@gmail.com> In-Reply-To: <cover-v5-00.36-00000000000-20210902T125110Z-avarab@gmail.com> References: <cover-v4-00.36-00000000000-20210803T191505Z-avarab@gmail.com> <cover-v5-00.36-00000000000-20210902T125110Z-avarab@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk
Series	Run hooks via "git run hook" & hook library \| expand [v5,00/36] Run hooks via "git run hook" & hook library [v5,01/36] Makefile: mark "check" target as .PHONY [v5,02/36] Makefile: stop hardcoding {command,config}-list.h [v5,03/36] Makefile: remove an out-of-date comment [v5,04/36] hook.[ch]: move find_hook() from run-command.c to hook.c [v5,05/36] hook.c: add a hook_exists() wrapper and use it in bugreport.c [v5,06/36] hook.c users: use "hook_exists()" instead of "find_hook()" [v5,07/36] hook-list.h: add a generated list of hooks, like config-list.h [v5,08/36] hook: add 'run' subcommand [v5,09/36] gc: use hook library for pre-auto-gc hook [v5,10/36] rebase: convert pre-rebase to use hook.h [v5,11/36] am: convert applypatch to use hook.h [v5,12/36] hooks: convert 'post-checkout' hook to hook library [v5,13/36] merge: convert post-merge to use hook.h [v5,14/36] git hook run: add an --ignore-missing flag [v5,15/36] send-email: use 'git hook run' for 'sendemail-validate' [v5,16/36] git-p4: use 'git hook' to run hooks [v5,17/36] commit: convert {pre-commit,prepare-commit-msg} hook to hook.h [v5,18/36] read-cache: convert post-index-change to use hook.h [v5,19/36] receive-pack: convert push-to-checkout hook to hook.h [v5,20/36] run-command: remove old run_hook_{le,ve}() hook API [v5,21/36] run-command: allow stdin for run_processes_parallel [v5,22/36] hook: support passing stdin to hooks [v5,23/36] am: convert 'post-rewrite' hook to hook.h [v5,24/36] run-command: add stdin callback for parallelization [v5,25/36] hook: provide stdin by string_list or callback [v5,26/36] hook: convert 'post-rewrite' hook in sequencer.c to hook.h [v5,27/36] transport: convert pre-push hook to hook.h [v5,28/36] hook tests: test for exact "pre-push" hook input [v5,29/36] hook tests: use a modern style for "pre-push" tests [v5,30/36] reference-transaction: use hook.h to run hooks [v5,31/36] run-command: allow capturing of collated output [v5,32/36] hooks: allow callers to capture output [v5,33/36] receive-pack: convert 'update' hook to hook.h [v5,34/36] post-update: use hook.h library [v5,35/36] receive-pack: convert receive hooks to hook.h [v5,36/36] hooks: fix a TOCTOU in "did we run a hook?" heuristic

Commit Message

Ævar Arnfjörð Bjarmason Sept. 2, 2021, 1:11 p.m. UTC

This comment added in dfea575017 (Makefile: lazily compute header
dependencies, 2010-01-26) has been out of date since
92b88eba9f (Makefile: use `git ls-files` to list header files, if
possible, 2019-03-04), when we did exactly what it tells us not to do
and added $(GENERATED_H) to $(OBJECTS) dependencies.

The rest of it was also somewhere between inaccurate and outdated,
since as of b8ba629264 (Makefile: fold MISC_H into LIB_H, 2012-06-20)
it's not followed by a list of header files, that got moved earlier in
the file into LIB_H in 60d24dd255 (Makefile: fold XDIFF_H and VCSSVN_H
into LIB_H, 2012-07-06).

Let's just remove it entirely, to the extent that we have anything
useful to say here the comment on the
"USE_COMPUTED_HEADER_DEPENDENCIES" variable a few lines above this
change does the job for us.

Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
---
 Makefile | 7 -------
 1 file changed, 7 deletions(-)

diff mbox series

Patch

diff --git a/Makefile b/Makefile
index 4f1f13b5f0a..53c67f4a726 100644
--- a/Makefile
+++ b/Makefile
@@ -2524,13 +2524,6 @@  ifneq ($(dep_files_present),)
 include $(dep_files_present)
 endif
 else
-# Dependencies on header files, for platforms that do not support
-# the gcc -MMD option.
-#
-# Dependencies on automatically generated headers such as command-list.h
-# should _not_ be included here, since they are necessary even when
-# building an object for the first time.
-
 $(OBJECTS): $(LIB_H) $(GENERATED_H)
 endif