Skip Menu |

This queue is for tickets about the threads-lite CPAN distribution.

Report information
The Basics
Id: 68111
Status: resolved
Priority: 0/
Queue: threads-lite
People
Owner: LEONT [...] cpan.org
Requestors: RURBAN [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Critical
Broken in: 0.030
Fixed in: (no value)

History Show all quoted text
  Wed May 11 06:07:14 2011 RURBAN [...] cpan.org - Ticket created
Subject: end_locker dllexport windows patch
Attached patch fixes the windows/cygwin linker error src/resources.c:64: error: external linkage required for symbol 'end_locker' because of 'dllexport' attribute -- Reini Urban
Subject: threads-lite-windows.patch
To fix cygwin and probably windows also: src/resources.c:64: error: external linkage required for symbol 'end_locker' because of 'dllexport' attribute diff -u ./src/resources.c~ ./src/resources.c --- ./src/resources.c~ 2011-03-05 14:45:35.000000000 +0100 +++ ./src/resources.c 2011-05-11 09:56:10.983500000 +0200 @@ -61,7 +61,7 @@ COND_DESTROY(&counter.condvar); } -static XS(end_locker) { +XS(end_locker) { dVAR; dXSARGS; perl_mutex* mutex;
  Fri May 13 07:09:56 2011 LEONT [...] cpan.org - Correspondence added
On Wed May 11 06:07:14 2011, RURBAN wrote: Show quoted text
> Attached patch fixes the windows/cygwin linker error > > src/resources.c:64: error: external linkage required for symbol > 'end_locker' because of 'dllexport' attribute
Hi Reini, Thanks for the report. I solved it in a slightly different way: instead of removing the static I removed the 'dllexport' (by using XSPROTO() instead of XS()). Leon
  Fri May 13 07:09:57 2011 The RT System itself - Status changed from 'new' to 'open'
  Fri May 13 07:09:57 2011 LEONT [...] cpan.org - Status changed from 'open' to 'patched'
  Fri May 13 07:35:49 2011 LEONT [...] cpan.org - Taken
  Tue May 01 06:42:15 2012 LEONT [...] cpan.org - Status changed from 'patched' to 'resolved'