Skip to content

Commit

Permalink
Revert "Cygwin: Enable dynamicbase on the Cygwin DLL by default"
Browse files Browse the repository at this point in the history
This reverts commit 943433b.

This seems to fix fork errors under Docker, see
https://cygwin.com/pipermail/cygwin/2022-December/252711.html
  • Loading branch information
lazka authored and dscho committed Apr 6, 2024
1 parent cdd9e8d commit ae4c9c3
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions winsup/cygwin/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -600,8 +600,7 @@ $(NEW_DLL_NAME): $(LDSCRIPT) libdll.a $(VERSION_OFILES) $(LIBSERVER)\
$(newlib_build)/libm.a $(newlib_build)/libc.a
$(AM_V_CXXLD)$(CXX) $(CXXFLAGS) \
-mno-use-libstdc-wrappers \
-Wl,--gc-sections -nostdlib -Wl,-T$(LDSCRIPT) \
-Wl,--dynamicbase -static \
-Wl,--gc-sections -nostdlib -Wl,-T$(LDSCRIPT) -static \
$${SOURCE_DATE_EPOCH:+-Wl,--no-insert-timestamp} \
-Wl,--heap=0 -Wl,--out-implib,msysdll.a -shared -o $@ \
-e @DLL_ENTRY@ $(DEF_FILE) \
Expand Down

0 comments on commit ae4c9c3

Please sign in to comment.