Thank you for the follow ups.

Am 20.01.2026 um 08:56 schrieb LIU Hao:
diff --git a/mingw-w64-headers/Makefile.am b/mingw-w64-headers/Makefile.am
index 6ebeda477..2e7d8eaf8 100644
--- a/mingw-w64-headers/Makefile.am
+++ b/mingw-w64-headers/Makefile.am
@@ -21,8 +21,7 @@ gdiplushead_HEADERS = @GDIPLUSHEAD_LIST@
 wrlhead_HEADERS = @WRLHEAD_LIST@
 wrlwrappershead_HEADERS = @WRLWRAPPERSHEAD_LIST@
 mingwhelperhead_HEADERS = @MINGWHELPERHEAD_LIST@
-nodist_sdkshead_HEADERS = _mingw_ddk.h
-noinst_HEADERS = crt/sdks/_mingw_ddk.h.in
+sdkshead_HEADERS = crt/sdks/_mingw_ddk.h
  ddkheaddir = $(baseheaddir)/ddk
 idlheaddir = $(baseheaddir)


This hunk should probably be changed.

The file crt/sdks/_mingw_ddk.h.in file should not be installed. But it should be part of the dist tarball (created with `make dist`).

The file crt/sdks/_mingw_ddk.h is generated by the configure script. So, it should not (and cannot) be part of a dist tarball. But it should be installed. So, the following is probably correct:

nodist_sdkshead_HEADERS =crt/sdks/_mingw_ddk.h
noinst_HEADERS = crt/sdks/_mingw_ddk.h.in


I (erroneously) assumed that `autoconf` would be executed when I added `Makefile.am` to `CONFIGURE_DEPENDENCIES`. But when I realized that that triggered running `automake` instead, I changed that to `Makefile.in` instead of dropping it entirely.
That was wrong.

It would probably still make sense to add `crt/_mingw.h.in` and `crt/sdks/_mingw_ddk.h.in` to `CONFIG_STATUS_DEPENDENCIES` because the corresponding generated headers should be regenerated by the `configure` script in case these files are changed.

But that could also be done in a follow-up script.


Markus




_______________________________________________
Mingw-w64-public mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/mingw-w64-public

Reply via email to