Hi Ramsay, Ramsay Jones wrote: > LINK : warning LNK4044: unrecognized option '/Zi'; ignored > > In order to suppress the warning, we refrain from passing the > $(ALL_CFLAGS) macro to the linker. > > Note that, should it be necessary in the future, an option > intended for both the (front-end) compiler and the linker can > be included in both CFLAGS and LDFLAGS. I think traditionally CPPFLAGS is meant to be used for the purpose you are describing (see [1] for example). I realize that the Makefile does not currently use the terms this way: making it consistent would require . s/BASIC_CFLAGS/BASIC_CPPFLAGS/, except that the BASIC_CFLAGS += -Kthread settings should probably stay as-is . Windows BASIC_CFLAGS would probably need to be split: BASIC_CFLAGS = -nologo BASIC_CPPFLAGS = -I. -I../zlib ... -DWIN32 ... . s/COMPAT_CFLAGS/COMPAT_CPPFLAGS What do you think? Jonathan [1] http://www.gnu.org/software/autoconf/manual/html_node/Preset-Output-Variables.html#index-CFLAGS-80 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html