Skip to content

Commit f593ef7

Browse files
mark987gitster
authored andcommitted
Cygwin 1.7 supports mmap
git has shipped for years with MMAP enabled in the stock distribution, there are no reports of problems / failures on the list relating to this. Leave the default as-is on v1.5 due to lack of knowlege of this working on earlier Cygwin. Signed-off-by: Mark Levedahl <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent 103d530 commit f593ef7

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

config.mak.uname

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -160,17 +160,17 @@ ifeq ($(uname_O),Cygwin)
160160
NO_IPV6 = YesPlease
161161
OLD_ICONV = UnfortunatelyYes
162162
NO_THREAD_SAFE_PREAD = YesPlease
163+
# There are conflicting reports about this.
164+
# On some boxes NO_MMAP is needed, and not so elsewhere.
165+
# Try commenting this out if you suspect MMAP is more efficient
166+
NO_MMAP = YesPlease
163167
else
164168
NO_REGEX = UnfortunatelyYes
165169
endif
166170
NEEDS_LIBICONV = YesPlease
167171
NO_FAST_WORKING_DIRECTORY = UnfortunatelyYes
168172
NO_TRUSTABLE_FILEMODE = UnfortunatelyYes
169173
NO_ST_BLOCKS_IN_STRUCT_STAT = YesPlease
170-
# There are conflicting reports about this.
171-
# On some boxes NO_MMAP is needed, and not so elsewhere.
172-
# Try commenting this out if you suspect MMAP is more efficient
173-
NO_MMAP = YesPlease
174174
X = .exe
175175
COMPAT_OBJS += compat/cygwin.o
176176
UNRELIABLE_FSTAT = UnfortunatelyYes

0 commit comments

Comments
 (0)