Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cygwin: give cygperl*.dll an explicit base address #22853

Open
wants to merge 3 commits into
base: blead
Choose a base branch
from
Open

cygwin: give cygperl*.dll an explicit base address #22853

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
790b690
cygwin: give cygperl*.dll an explicit base address
tonycoz Dec 9, 2024
c3c96b7
Revert "cygwin: workaround DLL load address conflict"
tonycoz Dec 9, 2024
4f3b845
perldelta for setting cygperl*.dll image base
tonycoz Dec 9, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/testsuite.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -605,7 +605,7 @@ jobs:
run: |
cd ~/work
set +e
./Configure -des -Dusedevel -Doptimize=-g -DDEBUGGING -Astatic_ext=I18N/Langinfo || exit 1
./Configure -des -Dusedevel -Doptimize=-g -DDEBUGGING || exit 1
- name: Build
shell: sh
env:
Expand Down
8 changes: 8 additions & 0 deletions Makefile.SH
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,14 @@ true)
;;
cygwin*)
shrpldflags="$shrpldflags -Wl,--out-implib=libperl.dll.a"
# The cygperl dll can be large and has conflicted with
# other DLLs as its name has changed from release to release.
# Give it its own space (and a lot of it) to work in.
# 32-bit (untested) gets less space since it is 32-bit.
case "$myarchname" in
"x86_64-cygwin") shrpldflags="$shrpldflags -Wl,--image-base=0x00380000000" ;;
*) shrpldflags="$shrpldflags -Wl,--image-base=0x60500000" ;;
esac
linklibperl="-L. -lperl"
;;
sunos*)
Expand Down
6 changes: 4 additions & 2 deletions pod/perldelta.pod
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -387,9 +387,11 @@ source tree.

=over 4

=item XXX-some-platform
=item Cygwin

XXX
Supply an explicit base address for C<cygperl*.dll> that cannot
conflict with those generated by C<--enable-auto-image-base>. [GH
#22695][GH #22104]

=back

Expand Down
Loading