Hi Peff, On Sun, 3 Mar 2019, Jeff King wrote:
> By the way, "make hdr-check" already fails for me on master, as I do not > have libgcrypt installed, and it unconditionally checks sha256/gcrypt.h. Oh? I thought it'd fail only on Windows, so I have not submitted https://github.com/dscho/git/commit/fix-hdr-check yet: -- snipsnap -- >From c13d9985284d4b452db0d95b6949e02c533db634 Mon Sep 17 00:00:00 2001 From: Johannes Schindelin <johannes.schinde...@gmx.de> Date: Sat, 23 Feb 2019 20:38:40 +0100 Subject: [PATCH] hdr-check: make it work on Windows On Windows, we define a specific set of pre-processor macros, among other reasons: to avoid including syslog.h (which is not available on Windows). The hdr-check target did not use those definitions, resulting in a failure to include said syslog.h. To fix that, let's let the hdr-check target make use of ALL_CFLAGS, which does include the pre-processor macros that would e.g. skip the conditional block including syslog.h. Signed-off-by: Johannes Schindelin <johannes.schinde...@gmx.de> --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index c5240942f2..b9f00f0ea0 100644 --- a/Makefile +++ b/Makefile @@ -2741,7 +2741,7 @@ CHK_HDRS = $(filter-out $(EXCEPT_HDRS),$(patsubst ./%,%,$(LIB_H))) HCO = $(patsubst %.h,%.hco,$(CHK_HDRS)) $(HCO): %.hco: %.h FORCE - $(QUIET_HDR)$(CC) -include git-compat-util.h -I. -o /dev/null -c -xc $< + $(QUIET_HDR)$(CC) $(ALL_CFLAGS) -include git-compat-util.h -I. -o /dev/null -c -xc $< .PHONY: hdr-check $(HCO) hdr-check: $(HCO) -- 2.20.1.windows.1.1576.g37ac248661.dirty
