Bug #25589 for Audio-XMMSClient: Audio-XMMSClient-0.02: XS warnings/errors on AMD64 platform

Wed Mar 21 07:00:06 2007 asackett [...] artsackett.com - Ticket created

Subject:	Audio-XMMSClient-0.02: XS warnings/errors on AMD64 platform
Date:	Wed, 21 Mar 2007 04:48:52 -0600
To:	bug-Audio-XMMSClient [...] rt.cpan.org
From:	Art Sackett <asackett [...] artsackett.com>

Greetings, Florian: Attempting to install Audio-XMMSClient-0.02 on an AMD64 Debian etch (testing) system: $ uname -a Linux skunk 2.6.18-4-amd64 #1 SMP Wed Feb 21 14:29:38 UTC 2007 x86_64 GNU/Linux $ perl -v This is perl, v5.8.8 built for x86_64-linux-gnu-thread-multi The result: cc1: warnings being treated as errors XMMSClientResult.xs: In function ‘perl_xmmsclient_xmms_result_cast_value’: XMMSClientResult.xs:71: warning: cast from pointer to integer of different size XMMSClientResult.xs:74: warning: cast from pointer to integer of different size make: *** [xs/XMMSClientResult.o] Error 1 -- Art Sackett http://www.artsackett.com/

Sun Apr 15 09:02:46 2007 mbarbon [...] users.sourceforge.net - Correspondence added

From:

MBARBON [...] cpan.org

Hi, the following patch fixes the error. HTH Mattia --- xs/XMMSClientResult.xs.orig 2007-04-15 14:52:03.000000000 +0200 +++ xs/XMMSClientResult.xs 2007-04-15 14:52:24.000000000 +0200 @@ -68,10 +68,10 @@ perl_value = newSVpv((const char*)value, 0); break; case XMMSC_RESULT_VALUE_TYPE_INT32: - perl_value = newSViv((int)value); + perl_value = newSViv((IV)value); break; case XMMSC_RESULT_VALUE_TYPE_UINT32: - perl_value = newSVuv((unsigned int)value); + perl_value = newSVuv((UV)value); break; default: perl_value = &PL_sv_undef;

Sun Apr 15 09:02:47 2007 The RT System itself - Status changed from 'new' to 'open'

Mon Apr 16 10:37:38 2007 asackett [...] artsackett.com - Correspondence added

Subject:	Re: [rt.cpan.org #25589] Audio-XMMSClient-0.02: XS warnings/errors on AMD64 platform
Date:	Mon, 16 Apr 2007 08:36:41 -0600
To:	via RT <bug-Audio-XMMSClient [...] rt.cpan.org>
From:	Art Sackett <asackett [...] artsackett.com>

On Sun, Apr 15, 2007 at 09:02:47AM -0400, via RT wrote: Show quoted text

> the following patch fixes the error.

Indeed it does, works like a charm. Thanks! -- Art Sackett http://www.artsackett.com/

Sun Apr 29 16:35:09 2007 rafl [...] debian.org - Correspondence added

Thanks for the patch. This, and lots of other things, have been updated in the xmms2 git repository as the perl bindings are now bundled with it. I'm not sure what to do with the cpan distribution of that module. Give me some time to either think of a good way how to maintain both of them or to remove this dist from cpan. -Flo