Skip to content

Commit cd2cc44

Browse files
committed
Merge branch 'ab/darwin-default-to-sha1dc'
Use the SHA1DC implementation on macOS, just like other platforms, by default. * ab/darwin-default-to-sha1dc: Makefile: use sha1collisiondetection by default on OSX and Darwin
2 parents 3613ab5 + 35898ad commit cd2cc44

File tree

2 files changed

+5
-8
lines changed

2 files changed

+5
-8
lines changed

Makefile

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -511,10 +511,8 @@ include shared.mak
511511
# Define BLK_SHA1 to make use of optimized C SHA-1 routines bundled
512512
# with git (in the block-sha1/ directory).
513513
#
514-
# Define NO_APPLE_COMMON_CRYPTO on OSX to opt-out of using the
515-
# "APPLE_COMMON_CRYPTO" backend for SHA-1, which is currently the
516-
# default on that OS. On macOS 01.4 (Tiger) or older,
517-
# NO_APPLE_COMMON_CRYPTO is defined by default.
514+
# Define APPLE_COMMON_CRYPTO_SHA1 to use Apple's CommonCrypto for
515+
# SHA-1.
518516
#
519517
# If don't enable any of the *_SHA1 settings in this section, Git will
520518
# default to its built-in sha1collisiondetection library, which is a
@@ -1911,7 +1909,7 @@ ifdef NO_POSIX_GOODIES
19111909
BASIC_CFLAGS += -DNO_POSIX_GOODIES
19121910
endif
19131911

1914-
ifdef APPLE_COMMON_CRYPTO
1912+
ifdef APPLE_COMMON_CRYPTO_SHA1
19151913
# Apple CommonCrypto requires chunking
19161914
SHA1_MAX_BLOCK_SIZE = 1024L*1024L*1024L
19171915
endif
@@ -1928,7 +1926,7 @@ ifdef BLK_SHA1
19281926
LIB_OBJS += block-sha1/sha1.o
19291927
BASIC_CFLAGS += -DSHA1_BLK
19301928
else
1931-
ifdef APPLE_COMMON_CRYPTO
1929+
ifdef APPLE_COMMON_CRYPTO_SHA1
19321930
COMPAT_CFLAGS += -DCOMMON_DIGEST_FOR_OPENSSL
19331931
BASIC_CFLAGS += -DSHA1_APPLE
19341932
else

ci/lib.sh

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -258,8 +258,7 @@ macos-*)
258258
MAKEFLAGS="$MAKEFLAGS PYTHON_PATH=$(which python3)"
259259
else
260260
MAKEFLAGS="$MAKEFLAGS PYTHON_PATH=$(which python2)"
261-
MAKEFLAGS="$MAKEFLAGS NO_APPLE_COMMON_CRYPTO=NoThanks"
262-
MAKEFLAGS="$MAKEFLAGS NO_OPENSSL=NoThanks"
261+
MAKEFLAGS="$MAKEFLAGS APPLE_COMMON_CRYPTO_SHA1=Yes"
263262
fi
264263
;;
265264
esac

0 commit comments

Comments
 (0)