Front page | perl.perl5.porters |
Postings from October 2013
On 10/30/13 03:16, bulk88 via RT wrote:
> To move this ticket along.
>
> As a 3rd party who understands very little of this ticket, I think you need to remove the double to single quotes change in Configure. Also your patch is a diff patch, not a git patch. Here at p5p, we prefer, but don't require, git patches with commit messages. Mention this ticket number in the commit message body. Also since dld was removed per http://www.xray.mpe.mpg.de/mailing-lists/perl5-porters/2013-09/msg00174.html , this patch seems uncommitable now. Note the OP diff patch is from 5.12, not 5.19.
As previous tickets have mentioned that dld will be added back with a
freemint renamed file. And as mentioned the Configure quotes can be
reverted.
> Also, some [of my] questions to answer either here in this ticket or in commit message.
>
> Why did the fixed stack size change in the OS?
Because the app needs more stack.
>
> ____________________________________________________
> diff -ur perl-5.12.3/ext/DynaLoader/dl_dld.xs perl-5.12.3-mint/ext/DynaLoader/dl_dld.xs
> --- perl-5.12.3/ext/DynaLoader/dl_dld.xs 2010-11-21 21:40:55.000000000 +0000
> +++ perl-5.12.3-mint/ext/DynaLoader/dl_dld.xs 2011-02-21 09:38:51.000000000 +0000
> @@ -116,6 +116,18 @@
> goto haverror;
> }
>
> + DLDEBUG(1,PerlIO_printf(Perl_debug_log, "dld_link(libm.a)\n"));
> + if (dlderr = dld_link("/usr/lib/libm.a")) {
> + SaveError(aTHX_ "dld_link(libm.a): %s", dld_strerror(dlderr));
> + goto haverror;
> + }
> +
> + DLDEBUG(1,PerlIO_printf(Perl_debug_log, "dld_link(libc.a)\n"));
> + if (dlderr = dld_link("/usr/lib/libc.a")) {
> + SaveError(aTHX_ "dld_link(libc.a): %s", dld_strerror(dlderr));
> + goto haverror;
> + }
> +
> max = AvFILL(dl_resolve_using);
> for (x = 0; x <= max; x++) {
> char *sym = SvPVX(*av_fetch(dl_resolve_using, x, 0));
> ____________________________________________________
> Why more error cases for dld?
>
Because dld needs to link with system libraries.
Alan.
Thread Next