On Sat, Sep 14, 2013 at 12:06 AM, Junio C Hamano <gits...@pobox.com> wrote:
> You can explicitly include the system header from your compatibility
> layer, i.e.
> === compat/mingw/string.h ===
> #define __NO_INLINE__
> #ifdef SYSTEM_STRING_H_HEADER
> #include SYSTEM_STRING_H_HEADER
> #include_next <string.h>
> and then in config.mak.uname, do something like this:
> ifneq (,$(findstring MINGW,$(uname_S)))
> ifndef SYSTEM_STRING_H_HEADER
> SYSTEM_STRING_H_HEADER = "C:\\llvm\include\string.h"
> COMPAT_CFLAGS += -DSYSTEM_STRING_H_HEADER=$(SYSTEM_STRING_H_HEADER)
> People who have the system header file at different paths can
> further override SYSTEM_STRING_H_HEADER in their config.mak.
> That would help compilers targetting mingw that do not support
> "#include_next" without spreading the damage to other people's
> systems, I think.
I think this is less favorable compared to my last proposed solution.
While my work-around in git-compat-util.h from  already is quite
ugly, it's at least in a single place. You solution spreads the code
it multiple place, making it even more ugly and less comprehensible,
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html