Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Perl: porters

[perl #113050] Bleadperl v5.16.0-87-g50853fa breaks GFUJI/Mouse-0.97.tar.gz

 

 

Perl porters RSS feed   Index | Next | Previous | View Threaded


perlbug-followup at perl

May 22, 2012, 10:48 PM

Post #1 of 8 (220 views)
Permalink
[perl #113050] Bleadperl v5.16.0-87-g50853fa breaks GFUJI/Mouse-0.97.tar.gz

# New Ticket Created by (Andreas J. Koenig)
# Please include the string: [perl #113050]
# in the subject line of all future correspondence about this issue.
# <URL: https://rt.perl.org:443/rt3/Ticket/Display.html?id=113050 >


git bisect
----------
commit 50853fa94fa95a4fad70b61c9360709826bb8093
Author: Father Chrysostomos <sprout [at] cpan>
Date: Sat May 19 23:46:04 2012 -0700

Make overloaded classes inherit fallback

sample fail report
------------------
http://www.cpantesters.org/cpan/report/7d4efbec-a3ba-11e1-b045-c602f4b14d39

perl -V
-------
Summary of my perl5 (revision 5 version 17 subversion 0) configuration:
Commit id: 50853fa94fa95a4fad70b61c9360709826bb8093
Platform:
osname=linux, osvers=3.2.0-2-amd64, archname=x86_64-linux-thread-multi
uname='linux k83 3.2.0-2-amd64 #1 smp mon apr 30 05:20:23 utc 2012 x86_64 gnulinux '
config_args='-Dprefix=/home/src/perl/repoperls/installed-perls/perl/v5.16.0-87-g50853fa/9980 -Dmyhostname=k83 -Dinstallusrbinperl=n -Uversiononly -Dusedevel -des -Ui_db -Duseithreads -Uuselongdouble -DDEBUGGING=-g'
hint=recommended, useposix=true, d_sigaction=define
useithreads=define, usemultiplicity=define
useperlio=define, d_sfio=undef, uselargefiles=define, usesocks=undef
use64bitint=define, use64bitall=define, uselongdouble=undef
usemymalloc=n, bincompat5005=undef
Compiler:
cc='cc', ccflags ='-D_REENTRANT -D_GNU_SOURCE -fno-strict-aliasing -pipe -fstack-protector -I/usr/local/include -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64',
optimize='-O2 -g',
cppflags='-D_REENTRANT -D_GNU_SOURCE -fno-strict-aliasing -pipe -fstack-protector -I/usr/local/include'
ccversion='', gccversion='4.6.3', gccosandvers=''
intsize=4, longsize=8, ptrsize=8, doublesize=8, byteorder=12345678
d_longlong=define, longlongsize=8, d_longdbl=define, longdblsize=16
ivtype='long', ivsize=8, nvtype='double', nvsize=8, Off_t='off_t', lseeksize=8
alignbytes=8, prototype=define
Linker and Libraries:
ld='cc', ldflags =' -fstack-protector -L/usr/local/lib'
libpth=/usr/local/lib /lib/x86_64-linux-gnu /lib/../lib /usr/lib/x86_64-linux-gnu /usr/lib/../lib /lib /usr/lib
libs=-lnsl -ldb -ldl -lm -lcrypt -lutil -lpthread -lc
perllibs=-lnsl -ldl -lm -lcrypt -lutil -lpthread -lc
libc=, so=so, useshrplib=false, libperl=libperl.a
gnulibc_version='2.13'
Dynamic Linking:
dlsrc=dl_dlopen.xs, dlext=so, d_dlsymun=undef, ccdlflags='-Wl,-E'
cccdlflags='-fPIC', lddlflags='-shared -O2 -g -L/usr/local/lib -fstack-protector'


Characteristics of this binary (from libperl):
Compile-time options: HAS_TIMES MULTIPLICITY PERLIO_LAYERS
PERL_DONT_CREATE_GVSV PERL_IMPLICIT_CONTEXT
PERL_MALLOC_WRAP PERL_PRESERVE_IVUV PERL_USE_DEVEL
USE_64_BIT_ALL USE_64_BIT_INT USE_ITHREADS
USE_LARGE_FILES USE_LOCALE USE_LOCALE_COLLATE
USE_LOCALE_CTYPE USE_LOCALE_NUMERIC USE_PERLIO
USE_PERL_ATOF USE_REENTRANT_API
Built under linux
Compiled at May 23 2012 06:24:31
@INC:
/home/src/perl/repoperls/installed-perls/perl/v5.16.0-87-g50853fa/9980/lib/site_perl/5.17.0/x86_64-linux-thread-multi
/home/src/perl/repoperls/installed-perls/perl/v5.16.0-87-g50853fa/9980/lib/site_perl/5.17.0
/home/src/perl/repoperls/installed-perls/perl/v5.16.0-87-g50853fa/9980/lib/5.17.0/x86_64-linux-thread-multi
/home/src/perl/repoperls/installed-perls/perl/v5.16.0-87-g50853fa/9980/lib/5.17.0
.

--
andreas


perlbug-followup at perl

May 22, 2012, 10:53 PM

Post #2 of 8 (215 views)
Permalink
[perl #113050] Bleadperl v5.16.0-87-g50853fa breaks GFUJI/Mouse-0.97.tar.gz [In reply to]

Already? :-)

--

Father Chrysostomos


---
via perlbug: queue: perl5 status: new
https://rt.perl.org:443/rt3/Ticket/Display.html?id=113050


perlbug-followup at perl

May 23, 2012, 12:01 AM

Post #3 of 8 (216 views)
Permalink
[perl #113050] Bleadperl v5.16.0-87-g50853fa breaks GFUJI/Mouse-0.97.tar.gz [In reply to]

On Tue May 22 22:48:22 2012, andreas.koenig.7os6VVqR [at] franz wrote:
> git bisect
> ----------
> commit 50853fa94fa95a4fad70b61c9360709826bb8093
> Author: Father Chrysostomos <sprout [at] cpan>
> Date: Sat May 19 23:46:04 2012 -0700
>
> Make overloaded classes inherit fallback
>
> sample fail report
> ------------------
> http://www.cpantesters.org/cpan/report/7d4efbec-a3ba-11e1-b045-
> c602f4b14d39

Well, it turns out that most* of CPAN, not just Mouse, is breaking
overload.pm’s encapsulation, so I’ll have to work around it.

I also forgot to change ExtUtils::ParseXS to account.

I’ve just noticed that, whereas overload.pm says:

In the current
implementation, the value of C<fallback> in the first overloaded
ancestor is used, but this is accidental and subject to change.

perlxs.pod says:

If you do not set any FALLBACK value when using OVERLOAD,
it defaults to UNDEF.

So, what is described as an implementation artefact (something to be
fixed) in overload.pm is documented as a feature in perlxs.

So, should ExtUtils::ParseXS follow suit and allow fallback to be
inherited, or should it retain its documented behaviour?

I would prefer the former, but I could implement it either way. This
will affect how I fix this.

* Exaggeration.

--

Father Chrysostomos


---
via perlbug: queue: perl5 status: open
https://rt.perl.org:443/rt3/Ticket/Display.html?id=113050


perlbug-followup at perl

May 23, 2012, 6:03 AM

Post #4 of 8 (214 views)
Permalink
[perl #113050] Bleadperl v5.16.0-87-g50853fa breaks GFUJI/Mouse-0.97.tar.gz [In reply to]

On Wed May 23 00:01:46 2012, sprout wrote:
> On Tue May 22 22:48:22 2012, andreas.koenig.7os6VVqR [at] franz
wrote:
> > git bisect
> > ----------
> > commit 50853fa94fa95a4fad70b61c9360709826bb8093
> > Author: Father Chrysostomos <sprout [at] cpan>
> > Date: Sat May 19 23:46:04 2012 -0700
> >
> > Make overloaded classes inherit fallback
> >
> > sample fail report
> > ------------------
> > http://www.cpantesters.org/cpan/report/7d4efbec-a3ba-11e1-b045-
> > c602f4b14d39
>
> Well, it turns out that most* of CPAN, not just Mouse, is breaking
> overload.pm’s encapsulation, so I’ll have to work around it.

I’ve just fixed it in commit 3866ea3, but I don’t particularly like it.
The result of having parts of overload.pm and ParseXS.pm copied and
pasted all over the place is that perl can’t be as fast as it would be
otherwise. I’m wondering whether we should provide a C function for
registering overloading and force CPAN to switch (by reverting 3866ea3).

--

Father Chrysostomos


---
via perlbug: queue: perl5 status: open
https://rt.perl.org:443/rt3/Ticket/Display.html?id=113050


perlbug-followup at perl

May 23, 2012, 6:08 AM

Post #5 of 8 (208 views)
Permalink
[perl #113050] Bleadperl v5.16.0-87-g50853fa breaks GFUJI/Mouse-0.97.tar.gz [In reply to]

On Wed May 23 00:01:46 2012, sprout wrote:
> I’ve just noticed that, whereas overload.pm says:
>
> In the current
> implementation, the value of C<fallback> in the first overloaded
> ancestor is used, but this is accidental and subject to change.
>
> perlxs.pod says:
>
> If you do not set any FALLBACK value when using OVERLOAD,
> it defaults to UNDEF.
>
> So, what is described as an implementation artefact (something to be
> fixed) in overload.pm is documented as a feature in perlxs.
>
> So, should ExtUtils::ParseXS follow suit and allow fallback to be
> inherited, or should it retain its documented behaviour?
>
> I would prefer the former, but I could implement it either way. This
> will affect how I fix this.

Actually, it didn’t affect the fix, but the question remains.

--

Father Chrysostomos


---
via perlbug: queue: perl5 status: open
https://rt.perl.org:443/rt3/Ticket/Display.html?id=113050


rgs at consttype

May 23, 2012, 6:26 AM

Post #6 of 8 (208 views)
Permalink
Re: [perl #113050] Bleadperl v5.16.0-87-g50853fa breaks GFUJI/Mouse-0.97.tar.gz [In reply to]

On 23 May 2012 15:03, Father Chrysostomos via RT
<perlbug-followup [at] perl> wrote:
>> Well, it turns out that most* of CPAN, not just Mouse, is breaking
>> overload.pms encapsulation, so Ill have to work around it.
>
> Ive just fixed it in commit 3866ea3, but I dont particularly like it.
> The result of having parts of overload.pm and ParseXS.pm copied and
> pasted all over the place is that perl cant be as fast as it would be
> otherwise. Im wondering whether we should provide a C function for
> registering overloading and force CPAN to switch (by reverting 3866ea3).

2 functions probably: for registering, and for checking if there's
overloading in place (if I'm following you), like the Overloaded and
Method functions.


ikegami at adaelis

May 23, 2012, 8:34 AM

Post #7 of 8 (208 views)
Permalink
Re: [perl #113050] Bleadperl v5.16.0-87-g50853fa breaks GFUJI/Mouse-0.97.tar.gz [In reply to]

On Wed, May 23, 2012 at 3:01 AM, Father Chrysostomos via RT <
perlbug-followup [at] perl> wrote:

> Ive just noticed that, whereas overload.pm says:
>
> In the current
> implementation, the value of C<fallback> in the first overloaded
> ancestor is used, but this is accidental and subject to change.
>
> perlxs.pod says:
>
> If you do not set any FALLBACK value when using OVERLOAD,
> it defaults to UNDEF.
>
> So, what is described as an implementation artefact (something to be
> fixed) in overload.pm is documented as a feature in perlxs.
>

This is a false dichotomy. Set by the base class is still set.


perlbug-followup at perl

May 23, 2012, 1:41 PM

Post #8 of 8 (211 views)
Permalink
[perl #113050] Bleadperl v5.16.0-87-g50853fa breaks GFUJI/Mouse-0.97.tar.gz [In reply to]

On Wed May 23 06:26:40 2012, rgs [at] consttype wrote:
> On 23 May 2012 15:03, Father Chrysostomos via RT
> <perlbug-followup [at] perl> wrote:
> >> Well, it turns out that most* of CPAN, not just Mouse, is breaking
> >> overload.pm’s encapsulation, so I’ll have to work around it.
> >
> > I’ve just fixed it in commit 3866ea3, but I don’t particularly like it.
> >  The result of having parts of overload.pm and ParseXS.pm copied and
> > pasted all over the place is that perl can’t be as fast as it would be
> > otherwise.  I’m wondering whether we should provide a C function for
> > registering overloading and force CPAN to switch (by reverting 3866ea3).

Just to put this in perspective, these are affected:

PP:
Class-C3-0.23
Class::C3 as included by:
- Catalyst-Controller-Resources-0.08
- Lingua-EO-Orthography-0.03
- DBICx-Modeler-Generator-0.02
- MooseX-Types-Locale-Country-0.04
EntityModel::Support::Perl in EntityModel-0.014
AnyEvent-6.11
Lvalue-0.21
POEx::Role::SessionInstantiation::Meta::Session::Magic in
POEx-Role-SessionInstantiation-1.102610
Tak::STDIONode in Tak-0.001001
Pixie::Proxy in Pixie-2.06

XS:
version-0.95, but that is kept well in synch with core
ExtUtils::ParseXS as included by:
- Memcached-libmemcached (test suite)
Linux-CDROM-0.02
Mouse-0.97
Text-Xslate-1.5007
libapreq2-2.13

>
> 2 functions probably: for registering, and for checking if there's
> overloading in place (if I'm following you), like the Overloaded and
> Method functions.

I was only thinking in terms of what CPAN modules already do. All the
XS ones just register overloading. Most of the pure-Perl ones do the
same; some inspect the stash instead of calling overload'Overloaded;
some inspect the stash to find out the fallback value, for which we
currently have no interface. (I’ve just realised that I made
overload'Method("fallback") return the fallback value in 50853fa, and
then stopped that from working in 3866ea3.)

In any case, the registration interface could be:

hv_amagic_register(HV *stash, const char *op, SV *value)

with a NULL op for registering and a NULL value for the ‘no overload’
equivalent.

I’m suggesting amagic rather than overload in the name, because there
are already two API functions containing it. But maybe overload would
be better.

For enquiry, which is best?

hv_amagic(HV *stash, const char *op)
hv_amagic_get(...)
hv_amagic_method(...)

Maybe we should use get and set, for parallelism.

--

Father Chrysostomos


---
via perlbug: queue: perl5 status: open
https://rt.perl.org:443/rt3/Ticket/Display.html?id=113050

Perl porters RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.