Skip Menu |

This queue is for tickets about the Time-HiRes CPAN distribution.

Report information
The Basics
Id: 120032
Status: resolved
Priority: 0/
Queue: Time-HiRes

People
Owner: Nobody in particular
Requestors: NHORNE [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 1.29_00
Fixed in: (no value)



Subject: Doesn't compile with Perl 5.24.1 on OS/X
cc -c -fno-common -DPERL_DARWIN -O2 -W -Wformat=2 -Wswitch -Wshadow -Wwrite-strings -Wuninitialized -Wall -pipe -mtune=native -march=native -fno-strict-aliasing -fstack-protector-strong -I/usr/local/include -I/opt/local/include -Werror=declaration-after-statement -Wextra -Wc++-compat -Wthread-safety -O3 -DVERSION=\"1.9733\" -DXS_VERSION=\"1.9733\" "-I../.." -DTIME_HIRES_NANOSLEEP -DTIME_HIRES_CLOCK_GETTIME -DTIME_HIRES_CLOCK_GETRES -DTIME_HIRES_CLOCK_NANOSLEEP -DTIME_HIRES_CLOCK -DTIME_HIRES_STAT=1 -DATLEASTFIVEOHOHFIVE HiRes.c In file included from HiRes.xs:19: In file included from ../../perl.h:5859: ../../inline.h:526:5: warning: declaration shadows a local variable [-Wshadow] CX_POP_SAVEARRAY(cx); ^ ../../cop.h:612:13: note: expanded from macro 'CX_POP_SAVEARRAY' AV *av = GvAV(PL_defgv); \ ^ ../../inline.h:518:9: note: previous declaration is here AV *av; ^ HiRes.c:2151:14: warning: declaration shadows a local variable [-Wshadow] CV * cv; ^ HiRes.c:2126:1: note: previous declaration is here XS_EXTERNAL(boot_Time__HiRes) ^ ../../XSUB.h:142:51: note: expanded from macro 'XS_EXTERNAL' # define XS_EXTERNAL(name) void name(pTHX_ CV* cv __attribute__unused__) ^ HiRes.xs:945:15: error: use of undeclared identifier 'darwin_time_mutex' MUTEX_INIT(&darwin_time_mutex); ^ 2 warnings and 1 error generated. make[1]: *** [HiRes.o] Error 1
From: jhi [...] iki.fi
Thanks, yes, this is a known problem, fixed in the most recent CPAN releases of Time-HiRes, and in the bleadperl. I will file a ticket to get the fixes to 5.24.2.
For reference, this also affects 5.24.0 under Sierra (or the latest XCode or some combination of the two). I've filed a PR for Devel::PatchPerl: https://github.com/bingos/devel-patchperl/pull/26 Hopefully a more minimal change can be found for 5.24.2, but I ran out of tuits trying to just fix the compilation errors and still pass tests.
An upgrade to 1.9741 is now in maint-5.24, so this should be fixed in perl-5.24.3 when that gets released.
On Tue Aug 08 09:03:44 2017, SHAY wrote: Show quoted text
> An upgrade to 1.9741 is now in maint-5.24, so this should be fixed in > perl-5.24.3 when that gets released.
perl-5.24.3 was released 2017-09-22, closing.