Bug #128004 for ExtUtils-MakeMaker: gcc AIX builds broken

Wed Dec 12 18:44:56 2018 TONYC [...] cpan.org - Ticket created

Subject:

gcc AIX builds broken

Reported to perl as https://rt.perl.org/Ticket/Display.html?id=133596 This appears to have been broken by https://github.com/Perl-Toolchain-Gang/ExtUtils-MakeMaker/commit/982f991c7f564f3e40e3f57c1f7090a5d1763611 In the report above, gcc fails to build .so files: gcc -maix32 -Wl,-bhalt:4 -Wl,-bM:SRE -Wl,-bI:../../perl.exp -Wl,-bnoentry -lpt hreads -L/usr/lib -bE:B.exp B.o -o ../../lib/auto/B/B.so \ \ gcc: error: unrecognized command line option '-bE:B.exp' This appears to be from stripping out the "-Wl,-bE:$(BASEEXT).exp" from lddlflags and replacing it unconditionally with "-bE:$(BASEEXT).exp". lddlflags=' -Wl,-b64 -Wl,-bhalt:4 -Wl,-bM:SRE -Wl,-bI:$(PERL_INC)/perl.exp -Wl,-bE:$(BASEEXT).exp -Wl,-bnoentry -lpthreads -L/opt/freeware/lib -L/usr/lib' I suspect author of this change appears to have missed this code in hints/aix.sh that adds -Wl, to -b options when the compiler is gcc: lddlflags="`echo ' '$lddlflags | sed -e 's@ -b@ -Wl,-b@g'`" Unfortunately the reporter of the original issue has been unresponsive. Tony (I see a lot of activity in github issues, but ExtUtils::MakeMaker and META.json still points bug reports to rt.cpan.org, hence the report here.)

Sun Dec 16 13:46:25 2018 ETJ [...] cpan.org - Correspondence added

Following discussion on #toolchain, I believe MM_AIX::xs_dlsyms_arg should be made to read: my($self, $file) = @_; my $arg = qq{-bE:${file}}; $arg = '-Wl,'.$arg if $Config{lddlflags} =~ /-Wl,-bE:/; return $arg;

Sun Dec 16 13:46:27 2018 The RT System itself - Status changed from 'new' to 'open'

Thu Apr 25 09:33:18 2019 BINGOS [...] cpan.org - Correspondence added 30 min

On Sun Dec 16 13:46:25 2018, ETJ wrote: Show quoted text

> Following discussion on #toolchain, I believe MM_AIX::xs_dlsyms_arg > should be made to read: > > my($self, $file) = @_; > my $arg = qq{-bE:${file}}; > $arg = '-Wl,'.$arg if $Config{lddlflags} =~ /-Wl,-bE:/; > return $arg;

I have applied this and released v7.35_11

Thu Apr 25 09:33:19 2019 BINGOS [...] cpan.org - Taken

Thu Apr 25 09:33:20 2019 BINGOS [...] cpan.org - Fixed in 7.35_11 added

Thu Apr 25 09:34:11 2019 BINGOS [...] cpan.org - Correspondence added 5 min

Closing

Thu Apr 25 09:34:13 2019 BINGOS [...] cpan.org - Status changed from 'open' to 'resolved'