| CC: | Perl 5 Porters <perl5-porters [...] perl.org>, bug-Clone [...] rt.cpan.org |
| Subject: | Re: Clone fails to build with blead@34780 |
| Date: | Mon, 10 Nov 2008 20:15:05 +0100 |
| To: | Michael G Schwern <schwern [...] pobox.com> |
| From: | Marcus Holland-Moritz <mhx-perl [...] gmx.net> |
On 2008-11-10, at 08:30:12 +0100, Marcus Holland-Moritz wrote:
Show quoted text
> On 2008-11-09, at 18:03:55 -0800, Michael G Schwern wrote:
>
Done as #34804.
Marcus
Show quoted text> > There's a regression with Clone 0.29 and blead@34780 on OS X 10.4.
> >
> > $ bleadperl Makefile.PL Checking if your kit is complete...
> > Looks good
> > Writing Makefile for Clone
> >
> > $ make
> > cp Clone.pm blib/lib/Clone.pm
> > AutoSplitting blib/lib/Clone.pm (blib/lib/auto/Clone)
> > /usr/local/bin/bleadperl /usr/local/perl/blead/lib/5.11.0/ExtUtils/xsubpp
> > -typemap /usr/local/perl/blead/lib/5.11.0/ExtUtils/typemap Clone.xs >
> > Clone.xsc && mv Clone.xsc Clone.c
> > cc -c -I/sw/include -DDEBUGGING -fno-strict-aliasing -pipe
> > -I/usr/local/include -I/opt/local/include -O3 -DVERSION=\"0.29\"
> > -DXS_VERSION=\"0.29\"
> > "-I/usr/local/perl/blead/lib/5.11.0/darwin-thread-multi-2level/CORE" Clone.c
> > Clone.xs: In function ‘sv_clone’:
> > Clone.xs:286: error: assignment of read-only location
> > make: *** [Clone.o] Error 1
>
> I think something similar to #34770 needs to be done for SvRV(),
> for compatibility reasons. (I just can't do it right now.)
> But ultimately, Clone should start using SvRV_set() instead
> of assigning to SvRV().
>
> Marcus
--
C makes it easy for you to shoot yourself in the foot. C++ makes that
harder, but when you do, it blows away your whole leg.
-- Bjarne Stroustrup
Message body not shown because it is not plain text.