On Aug 14, 2014, at 16:18, Junio C Hamano wrote: > Markus Hitter <mah@xxxxxxxxxxx> writes: > >>> The <CommonCrypto/CommonHMAC.h> is in Mac OS X 10.6 .. 10.9, >>> but not in 10.4 (I don't know about 10.5). That header is new with 10.5 > Is this about platform dependency, or what the end user happens to > choose to install (in other words, is there an add-on users of 10.4 > can choose to add, which allows them to use that header)? Nope, it's a platform dependency. Not available prior to 10.5. The below patch does the right thing. Conveniently there's already a test for 10.4 and earlier so only a single line need be added. --Kyle ---- 8< ---- Subject: [PATCH] config.mak.uname: set NO_APPLE_COMMON_CRYPTO on older systems Older MacOS systems prior to 10.5 do not have the CommonCrypto support Git uses so set NO_APPLE_COMMON_CRYPTO on those systems. Signed-off-by: Kyle J. McKay <mackyle@xxxxxxxxx> --- config.mak.uname | 1 + 1 file changed, 1 insertion(+) diff --git a/config.mak.uname b/config.mak.uname index 7846bd76..f8e12c96 100644 --- a/config.mak.uname +++ b/config.mak.uname @@ -88,6 +88,7 @@ ifeq ($(uname_S),Darwin) NEEDS_LIBICONV = YesPlease ifeq ($(shell expr "$(uname_R)" : '[15678]\.'),2) OLD_ICONV = UnfortunatelyYes + NO_APPLE_COMMON_CRYPTO = YesPlease endif ifeq ($(shell expr "$(uname_R)" : '[15]\.'),2) NO_STRLCPY = YesPlease -- 1.8.5 -- 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