From: Junio C Hamano <gitster@pobox.com>
To: git@vger.kernel.org
Subject: [PATCH] compat: drop inclusion of <git-compat-util.h>
Date: Sat, 24 Feb 2024 12:32:51 -0800 [thread overview]
Message-ID: <xmqqwmqtli18.fsf@gitster.g> (raw)
These two header files are included from ordinary source files that
already include <git-compat-util.h> as the first header file as they
should. There is no need to include the compat-util in these
headers.
"make hdr-check" is not affected, as it is designed to assume that
what <git-compat-util.h> offers is available to everybody without
being included.
Signed-off-by: Junio C Hamano <gitster@pobox.com>
---
* There is an obvious alternative that goes in the complete
opposite direction possible, to update "make hdr-check" to ensure
that things that are depended upon in each header file
(e.g. pager.h refers to uintmax_t) are brought in by the header
file to include the compat-util in it, i.e.
diff --git c/Makefile w/Makefile
index 78e874099d..d7b360f15e 100644
--- c/Makefile
+++ w/Makefile
@@ -3259,7 +3259,7 @@ HCO = $(patsubst %.h,%.hco,$(CHK_HDRS))
HCC = $(HCO:hco=hcc)
%.hcc: %.h
- @echo '#include "git-compat-util.h"' >$@
+ @echo '/* #include "git-compat-util.h" */' >$@
@echo '#include "$<"' >>$@
$(HCO): %.hco: %.hcc FORCE
which would require in a noisy diff to add inclusion of
git-compat-util.h to many header files. For purposes of folks
who may want to carve out only pieces of our source tree, such an
approach might work better, but for that to happen and yield any
useful result, I suspect that compat-util header needs to be
split into "compatibility essentials" and other "it is convenient
if these are available everywhere, even though they do not have
much to do with hiding system dependencies from the sources"
parts first.
compat/compiler.h | 1 -
compat/disk.h | 1 -
2 files changed, 2 deletions(-)
diff --git c/compat/compiler.h w/compat/compiler.h
index 10dbb65937..e9ad9db84f 100644
--- c/compat/compiler.h
+++ w/compat/compiler.h
@@ -1,7 +1,6 @@
#ifndef COMPILER_H
#define COMPILER_H
-#include "git-compat-util.h"
#include "strbuf.h"
#ifdef __GLIBC__
diff --git c/compat/disk.h w/compat/disk.h
index 6c979c27d8..23bc1bef86 100644
--- c/compat/disk.h
+++ w/compat/disk.h
@@ -1,7 +1,6 @@
#ifndef COMPAT_DISK_H
#define COMPAT_DISK_H
-#include "git-compat-util.h"
#include "abspath.h"
#include "gettext.h"
next reply other threads:[~2024-02-24 20:32 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-02-24 20:32 Junio C Hamano [this message]
2024-02-24 22:25 ` [PATCH] compat: drop inclusion of <git-compat-util.h> Kyle Lippincott
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=xmqqwmqtli18.fsf@gitster.g \
--to=gitster@pobox.com \
--cc=git@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox