Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bleadperl v5.25.0-71-g5d32d26 breaks ZEFRAM/Array-Base-0.005.tar.gz #15335

Closed
p5pRT opened this issue May 18, 2016 · 36 comments
Closed

Bleadperl v5.25.0-71-g5d32d26 breaks ZEFRAM/Array-Base-0.005.tar.gz #15335

p5pRT opened this issue May 18, 2016 · 36 comments
Labels
BBC Blead Breaks CPAN - changes in blead broke a cpan module(s) Closable? We might be able to close this ticket, but we need to check with the reporter

Comments

@p5pRT
Copy link

p5pRT commented May 18, 2016

Migrated from rt.perl.org#128179 (status was 'open')

Searchable as RT128179$

@p5pRT
Copy link
Author

p5pRT commented May 18, 2016

From @andk

bisect


commit 5d32d26
Author​: David Mitchell <davem@​iabyn.com>
Date​: Sat May 14 15​:14​:28 2016 +0100

  enable PERL_OP_PARENT by default.

diagnostics


Compilation fails due missing op_sibling

also affected


  ZEFRAM/Array-Base-0.005.tar.gz
  CHOCOLATE/autobox-2.83.tar.gz
  ETHER/B-OPCheck-0.30.tar.gz
  TOKUHIROM/B-Tap-0.15.tar.gz
  CINDY/DestructAssign-0.002004.tar.gz
  PJCJ/Devel-Cover-1.23.tar.gz
  PEVANS/Devel-MAT-0.23.tar.gz
  TIMB/Devel-NYTProf-6.03.tar.gz
  ZEFRAM/Hash-SharedMem-0.004.tar.gz
  ADUITSIS/IPC-Mmap-0.21.tar.gz
  ETHER/List-Gather-0.12.tar.gz
  RURBAN/optimizer-0.08.tar.gz
  ZEFRAM/Params-Classify-0.013.tar.gz
  ZEFRAM/Scope-Cleanup-0.002.tar.gz
  YVES/Sereal-Decoder-3.014.tar.gz
  ZEFRAM/String-Base-0.001.tar.gz
  RUZ/Sub-StopCalls-0.02.tar.gz
  JJORE/UNIVERSAL-ref-0.14.tar.gz

perl -V


Summary of my perl5 (revision 5 version 25 subversion 1) configuration​:
  Commit id​: 5d32d26
  Platform​:
  osname=linux, osvers=4.3.0-1-amd64, archname=x86_64-linux-thread-multi-ld
  uname='linux k83 4.3.0-1-amd64 #1 smp debian 4.3.3-7 (2016-01-19) x86_64 gnulinux '
  config_args='-Dprefix=/home/sand/src/perl/repoperls/installed-perls/perl/v5.25.0-71-g5d32d26/109d -Dmyhostname=k83 -Dinstallusrbinperl=n -Uversiononly -Dusedevel -des -Ui_db -Dlibswanted=cl pthread socket inet nsl gdbm dbm malloc dl ld sun m crypt sec util c cposix posix ucb BSD gdbm_compat -Duseithreads -Duselongdouble -DDEBUGGING=-g'
  hint=recommended, useposix=true, d_sigaction=define
  useithreads=define, usemultiplicity=define
  use64bitint=define, use64bitall=define, uselongdouble=define
  usemymalloc=n, bincompat5005=undef
  Compiler​:
  cc='cc', ccflags ='-D_REENTRANT -D_GNU_SOURCE -fwrapv -fno-strict-aliasing -pipe -fstack-protector-strong -I/usr/local/include -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D_FORTIFY_SOURCE=2',
  optimize='-O2 -g',
  cppflags='-D_REENTRANT -D_GNU_SOURCE -fwrapv -fno-strict-aliasing -pipe -fstack-protector-strong -I/usr/local/include'
  ccversion='', gccversion='5.3.1 20160121', gccosandvers=''
  intsize=4, longsize=8, ptrsize=8, doublesize=8, byteorder=12345678, doublekind=3
  d_longlong=define, longlongsize=8, d_longdbl=define, longdblsize=16, longdblkind=3
  ivtype='long', ivsize=8, nvtype='long double', nvsize=16, Off_t='off_t', lseeksize=8
  alignbytes=16, prototype=define
  Linker and Libraries​:
  ld='cc', ldflags =' -fstack-protector-strong -L/usr/local/lib'
  libpth=/usr/local/lib /usr/lib/gcc/x86_64-linux-gnu/5/include-fixed /usr/include/x86_64-linux-gnu /usr/lib /lib/x86_64-linux-gnu /lib/../lib /usr/lib/x86_64-linux-gnu /usr/lib/../lib /lib
  libs=-lpthread -lnsl -lgdbm -ldl -lm -lcrypt -lutil -lc -lgdbm_compat
  perllibs=-lpthread -lnsl -ldl -lm -lcrypt -lutil -lc
  libc=libc-2.21.so, so=so, useshrplib=false, libperl=libperl.a
  gnulibc_version='2.21'
  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-strong'

Characteristics of this binary (from libperl)​:
  Compile-time options​: HAS_TIMES MULTIPLICITY PERLIO_LAYERS
  PERL_COPY_ON_WRITE PERL_DONT_CREATE_GVSV
  PERL_HASH_FUNC_ONE_AT_A_TIME_HARD
  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_LOCALE_TIME USE_LONG_DOUBLE
  USE_PERLIO USE_PERL_ATOF USE_REENTRANT_API
  Built under linux
  Compiled at May 18 2016 04​:18​:40
  @​INC​:
  /home/sand/src/perl/repoperls/installed-perls/perl/v5.25.0-71-g5d32d26/109d/lib/site_perl/5.25.1/x86_64-linux-thread-multi-ld
  /home/sand/src/perl/repoperls/installed-perls/perl/v5.25.0-71-g5d32d26/109d/lib/site_perl/5.25.1
  /home/sand/src/perl/repoperls/installed-perls/perl/v5.25.0-71-g5d32d26/109d/lib/5.25.1/x86_64-linux-thread-multi-ld
  /home/sand/src/perl/repoperls/installed-perls/perl/v5.25.0-71-g5d32d26/109d/lib/5.25.1
  .

--
andreas

@p5pRT
Copy link
Author

p5pRT commented May 20, 2016

From @jkeenan

On Wed May 18 12​:20​:25 2016, andreas.koenig.7os6VVqR@​franz.ak.mind.de wrote​:

bisect
------
commit 5d32d26
Author​: David Mitchell <davem@​iabyn.com>
Date​: Sat May 14 15​:14​:28 2016 +0100

enable PERL_OP_PARENT by default.

diagnostics
-----------
Compilation fails due missing op_sibling

Is this change to the internals the kind that we feel strongly enough to want to ask the authors below to revise their code?

also affected
-------------
ZEFRAM/Array-Base-0.005.tar.gz
CHOCOLATE/autobox-2.83.tar.gz
ETHER/B-OPCheck-0.30.tar.gz
TOKUHIROM/B-Tap-0.15.tar.gz
CINDY/DestructAssign-0.002004.tar.gz
PJCJ/Devel-Cover-1.23.tar.gz
PEVANS/Devel-MAT-0.23.tar.gz
TIMB/Devel-NYTProf-6.03.tar.gz
ZEFRAM/Hash-SharedMem-0.004.tar.gz
ADUITSIS/IPC-Mmap-0.21.tar.gz
ETHER/List-Gather-0.12.tar.gz
RURBAN/optimizer-0.08.tar.gz
ZEFRAM/Params-Classify-0.013.tar.gz
ZEFRAM/Scope-Cleanup-0.002.tar.gz
YVES/Sereal-Decoder-3.014.tar.gz
ZEFRAM/String-Base-0.001.tar.gz
RUZ/Sub-StopCalls-0.02.tar.gz
JJORE/UNIVERSAL-ref-0.14.tar.gz

perl -V
-------
Summary of my perl5 (revision 5 version 25 subversion 1)
configuration​:
Commit id​: 5d32d26
Platform​:
osname=linux, osvers=4.3.0-1-amd64, archname=x86_64-linux-thread-
multi-ld
uname='linux k83 4.3.0-1-amd64 #1 smp debian 4.3.3-7 (2016-01-19)
x86_64 gnulinux '
config_args='-Dprefix=/home/sand/src/perl/repoperls/installed-
perls/perl/v5.25.0-71-g5d32d26/109d -Dmyhostname=k83
-Dinstallusrbinperl=n -Uversiononly -Dusedevel -des -Ui_db
-Dlibswanted=cl pthread socket inet nsl gdbm dbm malloc dl ld sun m
crypt sec util c cposix posix ucb BSD gdbm_compat -Duseithreads
-Duselongdouble -DDEBUGGING=-g'
hint=recommended, useposix=true, d_sigaction=define
useithreads=define, usemultiplicity=define
use64bitint=define, use64bitall=define, uselongdouble=define
usemymalloc=n, bincompat5005=undef
Compiler​:
cc='cc', ccflags ='-D_REENTRANT -D_GNU_SOURCE -fwrapv -fno-strict-
aliasing -pipe -fstack-protector-strong -I/usr/local/include
-D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D_FORTIFY_SOURCE=2',
optimize='-O2 -g',
cppflags='-D_REENTRANT -D_GNU_SOURCE -fwrapv -fno-strict-aliasing
-pipe -fstack-protector-strong -I/usr/local/include'
ccversion='', gccversion='5.3.1 20160121', gccosandvers=''
intsize=4, longsize=8, ptrsize=8, doublesize=8,
byteorder=12345678, doublekind=3
d_longlong=define, longlongsize=8, d_longdbl=define,
longdblsize=16, longdblkind=3
ivtype='long', ivsize=8, nvtype='long double', nvsize=16,
Off_t='off_t', lseeksize=8
alignbytes=16, prototype=define
Linker and Libraries​:
ld='cc', ldflags =' -fstack-protector-strong -L/usr/local/lib'
libpth=/usr/local/lib /usr/lib/gcc/x86_64-linux-gnu/5/include-
fixed /usr/include/x86_64-linux-gnu /usr/lib /lib/x86_64-linux-gnu
/lib/../lib /usr/lib/x86_64-linux-gnu /usr/lib/../lib /lib
libs=-lpthread -lnsl -lgdbm -ldl -lm -lcrypt -lutil -lc
-lgdbm_compat
perllibs=-lpthread -lnsl -ldl -lm -lcrypt -lutil -lc
libc=libc-2.21.so, so=so, useshrplib=false, libperl=libperl.a
gnulibc_version='2.21'
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-strong'

Characteristics of this binary (from libperl)​:
Compile-time options​: HAS_TIMES MULTIPLICITY PERLIO_LAYERS
PERL_COPY_ON_WRITE PERL_DONT_CREATE_GVSV
PERL_HASH_FUNC_ONE_AT_A_TIME_HARD
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_LOCALE_TIME
USE_LONG_DOUBLE
USE_PERLIO USE_PERL_ATOF USE_REENTRANT_API
Built under linux
Compiled at May 18 2016 04​:18​:40
@​INC​:
/home/sand/src/perl/repoperls/installed-perls/perl/v5.25.0-71-
g5d32d26/109d/lib/site_perl/5.25.1/x86_64-linux-thread-multi-ld
/home/sand/src/perl/repoperls/installed-perls/perl/v5.25.0-71-
g5d32d26/109d/lib/site_perl/5.25.1
/home/sand/src/perl/repoperls/installed-perls/perl/v5.25.0-71-
g5d32d26/109d/lib/5.25.1/x86_64-linux-thread-multi-ld
/home/sand/src/perl/repoperls/installed-perls/perl/v5.25.0-71-
g5d32d26/109d/lib/5.25.1
.

--
James E Keenan (jkeenan@​cpan.org)

@p5pRT
Copy link
Author

p5pRT commented May 20, 2016

The RT System itself - Status changed from 'new' to 'open'

@p5pRT
Copy link
Author

p5pRT commented May 20, 2016

From @cpansprout

On Thu May 19 18​:01​:57 2016, jkeenan wrote​:

On Wed May 18 12​:20​:25 2016, andreas.koenig.7os6VVqR@​franz.ak.mind.de
wrote​:

bisect
------
commit 5d32d26
Author​: David Mitchell <davem@​iabyn.com>
Date​: Sat May 14 15​:14​:28 2016 +0100

enable PERL_OP_PARENT by default.

diagnostics
-----------
Compilation fails due missing op_sibling

Is this change to the internals the kind that we feel strongly enough
to want to ask the authors below to revise their code?

I certainly think so, but that is just one opinion.

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented May 20, 2016

From @ilmari

"Father Chrysostomos via RT" <perlbug-followup@​perl.org> writes​:

On Thu May 19 18​:01​:57 2016, jkeenan wrote​:

On Wed May 18 12​:20​:25 2016, andreas.koenig.7os6VVqR@​franz.ak.mind.de
wrote​:

bisect
------
commit 5d32d26
Author​: David Mitchell <davem@​iabyn.com>
Date​: Sat May 14 15​:14​:28 2016 +0100

enable PERL_OP_PARENT by default.

diagnostics
-----------
Compilation fails due missing op_sibling

Is this change to the internals the kind that we feel strongly enough
to want to ask the authors below to revise their code?

I certainly think so, but that is just one opinion.

I think so too. Conversion should be as trivial as adding conditional
defines of the non-PERL_OP_PARENT versions of OpSIBLING and friends
(which really sould be in Devel​::PPPort)​:

http​://perl5.git.perl.org/perl.git/blob/v5.25.0​:/op.h#l1005

And then replacing uses of ->op_sibling with calls to the appropriate
macro.

--
"The surreality of the universe tends towards a maximum" -- Skud's Law
"Never formulate a law or axiom that you're not prepared to live with
the consequences of." -- Skud's Meta-Law

@p5pRT
Copy link
Author

p5pRT commented Jun 16, 2016

From @andk

Also affected​: CLKAO/invoker-0.34.tar.gz
--
andreas

@p5pRT
Copy link
Author

p5pRT commented Jun 21, 2016

From @iabyn

On Fri, May 20, 2016 at 09​:12​:45AM +0100, Dagfinn Ilmari Mannsåker wrote​:

"Father Chrysostomos via RT" <perlbug-followup@​perl.org> writes​:

On Thu May 19 18​:01​:57 2016, jkeenan wrote​:

On Wed May 18 12​:20​:25 2016, andreas.koenig.7os6VVqR@​franz.ak.mind.de
wrote​:

bisect
------
commit 5d32d26
Author​: David Mitchell <davem@​iabyn.com>
Date​: Sat May 14 15​:14​:28 2016 +0100

enable PERL_OP_PARENT by default.

diagnostics
-----------
Compilation fails due missing op_sibling

Is this change to the internals the kind that we feel strongly enough
to want to ask the authors below to revise their code?

I certainly think so, but that is just one opinion.

I think so too. Conversion should be as trivial as adding conditional
defines of the non-PERL_OP_PARENT versions of OpSIBLING and friends
(which really sould be in Devel​::PPPort)​:

http​://perl5.git.perl.org/perl.git/blob/v5.25.0​:/op.h#l1005

And then replacing uses of ->op_sibling with calls to the appropriate
macro.

Note that I had earlier solicited opinions as to whether this should be
enabled by default in 5.25.x​:

  http​://nntp.perl.org/group/perl.perl5.porters/235635

--
"Procrastination grows to fill the available time"
  -- Mitchell's corollary to Parkinson's Law

@p5pRT
Copy link
Author

p5pRT commented Jun 21, 2016

From @demerphq

On 20 May 2016 at 10​:12, Dagfinn Ilmari Mannsåker <ilmari@​ilmari.org> wrote​:

"Father Chrysostomos via RT" <perlbug-followup@​perl.org> writes​:

On Thu May 19 18​:01​:57 2016, jkeenan wrote​:

On Wed May 18 12​:20​:25 2016, andreas.koenig.7os6VVqR@​franz.ak.mind.de
wrote​:

bisect
------
commit 5d32d26
Author​: David Mitchell <davem@​iabyn.com>
Date​: Sat May 14 15​:14​:28 2016 +0100

enable PERL_OP_PARENT by default.

diagnostics
-----------
Compilation fails due missing op_sibling

Is this change to the internals the kind that we feel strongly enough
to want to ask the authors below to revise their code?

I certainly think so, but that is just one opinion.

I think so too. Conversion should be as trivial as adding conditional
defines of the non-PERL_OP_PARENT versions of OpSIBLING and friends
(which really sould be in Devel​::PPPort)​:

http​://perl5.git.perl.org/perl.git/blob/v5.25.0​:/op.h#l1005

And then replacing uses of ->op_sibling with calls to the appropriate
macro.

I plan to do this to Sereal soon. Thanks for the instructions!

Yves

--
perl -Mre=debug -e "/just|another|perl|hacker/"

@p5pRT
Copy link
Author

p5pRT commented Jun 21, 2016

From @ilmari

demerphq <demerphq@​gmail.com> writes​:

On 20 May 2016 at 10​:12, Dagfinn Ilmari Mannsåker <ilmari@​ilmari.org> wrote​:

I think so too. Conversion should be as trivial as adding conditional
defines of the non-PERL_OP_PARENT versions of OpSIBLING and friends
(which really sould be in Devel​::PPPort)​:

http​://perl5.git.perl.org/perl.git/blob/v5.25.0​:/op.h#l1005

And then replacing uses of ->op_sibling with calls to the appropriate
macro.

I plan to do this to Sereal soon. Thanks for the instructions!

FYI​: These macros have been added to Devel​::PPPort as of version 3.33.

--
"I use RMS as a guide in the same way that a boat captain would use
a lighthouse. It's good to know where it is, but you generally
don't want to find yourself in the same spot." - Tollef Fog Heen

@p5pRT
Copy link
Author

p5pRT commented Dec 6, 2016

From @jkeenan

On Wed, 18 May 2016 19​:20​:25 GMT, andreas.koenig.7os6VVqR@​franz.ak.mind.de wrote​:

bisect
------
commit 5d32d26
Author​: David Mitchell <davem@​iabyn.com>
Date​: Sat May 14 15​:14​:28 2016 +0100

enable PERL_OP_PARENT by default.

diagnostics
-----------
Compilation fails due missing op_sibling

also affected
-------------
ZEFRAM/Array-Base-0.005.tar.gz
CHOCOLATE/autobox-2.83.tar.gz
ETHER/B-OPCheck-0.30.tar.gz
TOKUHIROM/B-Tap-0.15.tar.gz
CINDY/DestructAssign-0.002004.tar.gz
PJCJ/Devel-Cover-1.23.tar.gz
PEVANS/Devel-MAT-0.23.tar.gz
TIMB/Devel-NYTProf-6.03.tar.gz
ZEFRAM/Hash-SharedMem-0.004.tar.gz
ADUITSIS/IPC-Mmap-0.21.tar.gz
ETHER/List-Gather-0.12.tar.gz
RURBAN/optimizer-0.08.tar.gz
ZEFRAM/Params-Classify-0.013.tar.gz
ZEFRAM/Scope-Cleanup-0.002.tar.gz
YVES/Sereal-Decoder-3.014.tar.gz
ZEFRAM/String-Base-0.001.tar.gz
RUZ/Sub-StopCalls-0.02.tar.gz
JJORE/UNIVERSAL-ref-0.14.tar.gz

perl -V
-------
Summary of my perl5 (revision 5 version 25 subversion 1)
configuration​:
Commit id​: 5d32d26
Platform​:
osname=linux, osvers=4.3.0-1-amd64, archname=x86_64-linux-thread-
multi-ld
uname='linux k83 4.3.0-1-amd64 #1 smp debian 4.3.3-7 (2016-01-19)
x86_64 gnulinux '
config_args='-Dprefix=/home/sand/src/perl/repoperls/installed-
perls/perl/v5.25.0-71-g5d32d26/109d -Dmyhostname=k83
-Dinstallusrbinperl=n -Uversiononly -Dusedevel -des -Ui_db
-Dlibswanted=cl pthread socket inet nsl gdbm dbm malloc dl ld sun m
crypt sec util c cposix posix ucb BSD gdbm_compat -Duseithreads
-Duselongdouble -DDEBUGGING=-g'
hint=recommended, useposix=true, d_sigaction=define
useithreads=define, usemultiplicity=define
use64bitint=define, use64bitall=define, uselongdouble=define
usemymalloc=n, bincompat5005=undef
Compiler​:
cc='cc', ccflags ='-D_REENTRANT -D_GNU_SOURCE -fwrapv -fno-strict-
aliasing -pipe -fstack-protector-strong -I/usr/local/include
-D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D_FORTIFY_SOURCE=2',
optimize='-O2 -g',
cppflags='-D_REENTRANT -D_GNU_SOURCE -fwrapv -fno-strict-aliasing
-pipe -fstack-protector-strong -I/usr/local/include'
ccversion='', gccversion='5.3.1 20160121', gccosandvers=''
intsize=4, longsize=8, ptrsize=8, doublesize=8,
byteorder=12345678, doublekind=3
d_longlong=define, longlongsize=8, d_longdbl=define,
longdblsize=16, longdblkind=3
ivtype='long', ivsize=8, nvtype='long double', nvsize=16,
Off_t='off_t', lseeksize=8
alignbytes=16, prototype=define
Linker and Libraries​:
ld='cc', ldflags =' -fstack-protector-strong -L/usr/local/lib'
libpth=/usr/local/lib /usr/lib/gcc/x86_64-linux-gnu/5/include-
fixed /usr/include/x86_64-linux-gnu /usr/lib /lib/x86_64-linux-gnu
/lib/../lib /usr/lib/x86_64-linux-gnu /usr/lib/../lib /lib
libs=-lpthread -lnsl -lgdbm -ldl -lm -lcrypt -lutil -lc
-lgdbm_compat
perllibs=-lpthread -lnsl -ldl -lm -lcrypt -lutil -lc
libc=libc-2.21.so, so=so, useshrplib=false, libperl=libperl.a
gnulibc_version='2.21'
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-strong'

Characteristics of this binary (from libperl)​:
Compile-time options​: HAS_TIMES MULTIPLICITY PERLIO_LAYERS
PERL_COPY_ON_WRITE PERL_DONT_CREATE_GVSV
PERL_HASH_FUNC_ONE_AT_A_TIME_HARD
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_LOCALE_TIME
USE_LONG_DOUBLE
USE_PERLIO USE_PERL_ATOF USE_REENTRANT_API
Built under linux
Compiled at May 18 2016 04​:18​:40
@​INC​:
/home/sand/src/perl/repoperls/installed-perls/perl/v5.25.0-71-
g5d32d26/109d/lib/site_perl/5.25.1/x86_64-linux-thread-multi-ld
/home/sand/src/perl/repoperls/installed-perls/perl/v5.25.0-71-
g5d32d26/109d/lib/site_perl/5.25.1
/home/sand/src/perl/repoperls/installed-perls/perl/v5.25.0-71-
g5d32d26/109d/lib/5.25.1/x86_64-linux-thread-multi-ld
/home/sand/src/perl/repoperls/installed-perls/perl/v5.25.0-71-
g5d32d26/109d/lib/5.25.1
.

Contributors​:

Can we get an update on the status of this BBC ticket?

Thank you very much.
--
James E Keenan (jkeenan@​cpan.org)

@p5pRT
Copy link
Author

p5pRT commented Dec 7, 2016

From @iabyn

On Tue, Dec 06, 2016 at 01​:48​:15PM -0800, James E Keenan via RT wrote​:

On Wed, 18 May 2016 19​:20​:25 GMT, andreas.koenig.7os6VVqR@​franz.ak.mind.de wrote​:

bisect
------
commit 5d32d26
Author​: David Mitchell <davem@​iabyn.com>
Date​: Sat May 14 15​:14​:28 2016 +0100

enable PERL_OP_PARENT by default.

Contributors​:

Can we get an update on the status of this BBC ticket?

Of the distributions listed in this ticket, the following appear to have
been fixed​:

  CHOCOLATE/autobox-2.83.tar.gz
  ETHER/B-OPCheck-0.30.tar.gz
  CINDY/DestructAssign-0.002004.tar.gz
  TIMB/Devel-NYTProf-6.03.tar.gz
  YVES/Sereal-Decoder-3.014.tar.gz

  ADUITSIS/IPC-Mmap-0.21.tar.gz
  fixed in development release 0.22_02?

while the following haven't had a new release yet​:

columns are​:
  month and year of last release
  F = there is a fix available with the ticket
  rt.cpan or github ticket number

  02/2012 #117385 ZEFRAM/Array-Base-0.005.tar.gz
  05/2015 github #4 TOKUHIROM/B-Tap-0.15.tar.gz
  04/2016 F github #161 PJCJ/Devel-Cover-1.23.tar.gz
  06/2016 F #114581 PEVANS/Devel-MAT-0.23.tar.gz
  01/2015 #117255 ZEFRAM/Hash-SharedMem-0.004.tar.gz
  01/2015 #116694 ETHER/List-Gather-0.12.tar.gz
  02/2012 #117387 RURBAN/optimizer-0.08.tar.gz
  11/2010 F #114490 ZEFRAM/Params-Classify-0.013.tar.gz
  02/2012 #117409 ZEFRAM/Scope-Cleanup-0.002.tar.gz
  02/2012 #117410 ZEFRAM/String-Base-0.001.tar.gz
  08/2012 #117338 RUZ/Sub-StopCalls-0.02.tar.gz
  01/2011 #118008 JJORE/UNIVERSAL-ref-0.14.tar.gz
  02/2013 #117215 CLKAO/invoker-0.34.tar.gz

Several belong to list members, so would be good to hear whether they
intend to fix or is wontfix.

The original intention was to enable PERL_OP_PARENT by default early in the
5.25.x cycle, then decide nearer to 5.26.0 release time whether to back it
out or not.

--
It's not that I'm afraid to die, I just don't want to be there when it
happens.
  -- Woody Allen

@p5pRT
Copy link
Author

p5pRT commented Dec 7, 2016

From zefram@fysh.org

I have mine mostly updated in my private repos, such that they compile
and superficially work with PERL_OP_PARENT. Still need to check for
less obvious incompatibilities, and there are a bunch of other updates
that I'm bundling with this for release.

-zefram

@p5pRT
Copy link
Author

p5pRT commented Dec 7, 2016

From @karenetheridge

re List-Gather (and also Cond-Expr and Sub-Call-Tail, which are not listed
but are also broken since 5.25.1) -- I would be happy to ship fixes, but
the op tree parsing in these modules is complicated and I was unable to
make a translation to the ppport.h macros when I looked a few months ago.
Patches gratefully accepted!

On Wed, Dec 7, 2016 at 4​:07 AM, Dave Mitchell <davem@​iabyn.com> wrote​:

On Tue, Dec 06, 2016 at 01​:48​:15PM -0800, James E Keenan via RT wrote​:

On Wed, 18 May 2016 19​:20​:25 GMT, andreas.koenig.7os6VVqR@​franz.
ak.mind.de wrote​:

bisect
------
commit 5d32d26
Author​: David Mitchell <davem@​iabyn.com>
Date​: Sat May 14 15​:14​:28 2016 +0100

enable PERL_OP_PARENT by default.

Contributors​:

Can we get an update on the status of this BBC ticket?

Of the distributions listed in this ticket, the following appear to have
been fixed​:

CHOCOLATE/autobox\-2\.83\.tar\.gz
ETHER/B\-OPCheck\-0\.30\.tar\.gz
CINDY/DestructAssign\-0\.002004\.tar\.gz
TIMB/Devel\-NYTProf\-6\.03\.tar\.gz
YVES/Sereal\-Decoder\-3\.014\.tar\.gz

ADUITSIS/IPC\-Mmap\-0\.21\.tar\.gz
    fixed in development release 0\.22\_02?

while the following haven't had a new release yet​:

columns are​:
month and year of last release
F = there is a fix available with the ticket
rt.cpan or github ticket number

02/2012   \#117385      ZEFRAM/Array\-Base\-0\.005\.tar\.gz
05/2015   github \#4    TOKUHIROM/B\-Tap\-0\.15\.tar\.gz
04/2016 F github \#161  PJCJ/Devel\-Cover\-1\.23\.tar\.gz
06/2016 F \#114581      PEVANS/Devel\-MAT\-0\.23\.tar\.gz
01/2015   \#117255      ZEFRAM/Hash\-SharedMem\-0\.004\.tar\.gz
01/2015   \#116694      ETHER/List\-Gather\-0\.12\.tar\.gz
02/2012   \#117387      RURBAN/optimizer\-0\.08\.tar\.gz
11/2010 F \#114490      ZEFRAM/Params\-Classify\-0\.013\.tar\.gz
02/2012   \#117409      ZEFRAM/Scope\-Cleanup\-0\.002\.tar\.gz
02/2012   \#117410      ZEFRAM/String\-Base\-0\.001\.tar\.gz
08/2012   \#117338      RUZ/Sub\-StopCalls\-0\.02\.tar\.gz
01/2011   \#118008      JJORE/UNIVERSAL\-ref\-0\.14\.tar\.gz
02/2013   \#117215      CLKAO/invoker\-0\.34\.tar\.gz

Several belong to list members, so would be good to hear whether they
intend to fix or is wontfix.

The original intention was to enable PERL_OP_PARENT by default early in the
5.25.x cycle, then decide nearer to 5.26.0 release time whether to back it
out or not.

--
It's not that I'm afraid to die, I just don't want to be there when it
happens.
-- Woody Allen

@p5pRT
Copy link
Author

p5pRT commented Dec 9, 2016

From @iabyn

On Fri, Dec 09, 2016 at 12​:13​:12PM +0000, Dave Mitchell wrote​:

On Wed, Dec 07, 2016 at 12​:24​:41PM -0800, Karen Etheridge wrote​:

re List-Gather (and also Cond-Expr and Sub-Call-Tail, which are not listed
but are also broken since 5.25.1) -- I would be happy to ship fixes, but
the op tree parsing in these modules is complicated and I was unable to
make a translation to the ppport.h macros when I looked a few months ago.
Patches gratefully accepted!

Patches attached for List-Gather and Cond-Expr.

Tested on threaded blead and unthreaded 5.24.0.

I didn't fix Sub-Call-Tail, since it has a dependency on
B​::Hooks​::OP​::Check​::EntersubForCV, which appears not to have worked since
around 5.21.4.

I don't think this made it to the p5p list since the diffs included two
complete copies of ppport.h and made the email too big. Did you get a
private copy of my email ok?

--
All wight. I will give you one more chance. This time, I want to hear
no Wubens. No Weginalds. No Wudolf the wed-nosed weindeers.
  -- Life of Brian

@p5pRT
Copy link
Author

p5pRT commented Mar 13, 2017

From @iabyn

On Wed, Dec 07, 2016 at 12​:07​:25PM +0000, Dave Mitchell wrote​:

On Tue, Dec 06, 2016 at 01​:48​:15PM -0800, James E Keenan via RT wrote​:

On Wed, 18 May 2016 19​:20​:25 GMT, andreas.koenig.7os6VVqR@​franz.ak.mind.de wrote​:

bisect
------
commit 5d32d26
Author​: David Mitchell <davem@​iabyn.com>
Date​: Sat May 14 15​:14​:28 2016 +0100

enable PERL_OP_PARENT by default.

[snip]

while the following haven't had a new release yet​:

columns are​:
month and year of last release
F = there is a fix available with the ticket
rt.cpan or github ticket number

02/2012   \#117385      ZEFRAM/Array\-Base\-0\.005\.tar\.gz
05/2015   github \#4    TOKUHIROM/B\-Tap\-0\.15\.tar\.gz
04/2016 F github \#161  PJCJ/Devel\-Cover\-1\.23\.tar\.gz
06/2016 F \#114581      PEVANS/Devel\-MAT\-0\.23\.tar\.gz
01/2015   \#117255      ZEFRAM/Hash\-SharedMem\-0\.004\.tar\.gz
01/2015   \#116694      ETHER/List\-Gather\-0\.12\.tar\.gz
02/2012   \#117387      RURBAN/optimizer\-0\.08\.tar\.gz
11/2010 F \#114490      ZEFRAM/Params\-Classify\-0\.013\.tar\.gz
02/2012   \#117409      ZEFRAM/Scope\-Cleanup\-0\.002\.tar\.gz
02/2012   \#117410      ZEFRAM/String\-Base\-0\.001\.tar\.gz
08/2012   \#117338      RUZ/Sub\-StopCalls\-0\.02\.tar\.gz
01/2011   \#118008      JJORE/UNIVERSAL\-ref\-0\.14\.tar\.gz
02/2013   \#117215      CLKAO/invoker\-0\.34\.tar\.gz

  01/2015 #117254 ETHER/Cond-Expr-0.04.tar.gz

Several belong to list members, so would be good to hear whether they
intend to fix or is wontfix.

The original intention was to enable PERL_OP_PARENT by default early in the
5.25.x cycle, then decide nearer to 5.26.0 release time whether to back it
out or not.

If we're to back out PERL_OP_PARENT by default for 5.26.0, we need to do
it very soon. Of the list above, there have been *no* new fixed releases.

I privately emailed to Karen fixes for List-Gather and Cond-Expr, and
Zefram said that he had fixed his in his repositories, but there has been
no movement since.

--
Nothing ventured, nothing lost.

@p5pRT
Copy link
Author

p5pRT commented Mar 13, 2017

From @pjcj

On Mon, Mar 13, 2017 at 12​:29​:24PM +0000, Dave Mitchell wrote​:

04/2016 F github \#161  PJCJ/Devel\-Cover\-1\.23\.tar\.gz

Several belong to list members, so would be good to hear whether they
intend to fix or is wontfix.

The original intention was to enable PERL_OP_PARENT by default early in the
5.25.x cycle, then decide nearer to 5.26.0 release time whether to back it
out or not.

If we're to back out PERL_OP_PARENT by default for 5.26.0, we need to do
it very soon. Of the list above, there have been *no* new fixed releases.

I *will* get a new release of Devel​::Cover out soon incorporating this
fix.

--
Paul Johnson - paul@​pjcj.net
http​://www.pjcj.net

@p5pRT
Copy link
Author

p5pRT commented Mar 14, 2017

From @xsawyerx

On 03/13/2017 01​:29 PM, Dave Mitchell wrote​:

On Wed, Dec 07, 2016 at 12​:07​:25PM +0000, Dave Mitchell wrote​:

On Tue, Dec 06, 2016 at 01​:48​:15PM -0800, James E Keenan via RT wrote​:

On Wed, 18 May 2016 19​:20​:25 GMT, andreas.koenig.7os6VVqR@​franz.ak.mind.de wrote​:

bisect
------
commit 5d32d26
Author​: David Mitchell <davem@​iabyn.com>
Date​: Sat May 14 15​:14​:28 2016 +0100

enable PERL_OP_PARENT by default.
[snip]

while the following haven't had a new release yet​:

columns are​:
month and year of last release
F = there is a fix available with the ticket
rt.cpan or github ticket number

02/2012   \#117385      ZEFRAM/Array\-Base\-0\.005\.tar\.gz
05/2015   github \#4    TOKUHIROM/B\-Tap\-0\.15\.tar\.gz
04/2016 F github \#161  PJCJ/Devel\-Cover\-1\.23\.tar\.gz
06/2016 F \#114581      PEVANS/Devel\-MAT\-0\.23\.tar\.gz
01/2015   \#117255      ZEFRAM/Hash\-SharedMem\-0\.004\.tar\.gz
01/2015   \#116694      ETHER/List\-Gather\-0\.12\.tar\.gz
02/2012   \#117387      RURBAN/optimizer\-0\.08\.tar\.gz
11/2010 F \#114490      ZEFRAM/Params\-Classify\-0\.013\.tar\.gz
02/2012   \#117409      ZEFRAM/Scope\-Cleanup\-0\.002\.tar\.gz
02/2012   \#117410      ZEFRAM/String\-Base\-0\.001\.tar\.gz
08/2012   \#117338      RUZ/Sub\-StopCalls\-0\.02\.tar\.gz
01/2011   \#118008      JJORE/UNIVERSAL\-ref\-0\.14\.tar\.gz
02/2013   \#117215      CLKAO/invoker\-0\.34\.tar\.gz
  01/2015   \#117254      ETHER/Cond\-Expr\-0\.04\.tar\.gz

Several belong to list members, so would be good to hear whether they
intend to fix or is wontfix.

The original intention was to enable PERL_OP_PARENT by default early in the
5.25.x cycle, then decide nearer to 5.26.0 release time whether to back it
out or not.
If we're to back out PERL_OP_PARENT by default for 5.26.0, we need to do
it very soon. Of the list above, there have been *no* new fixed releases.

I privately emailed to Karen fixes for List-Gather and Cond-Expr, and
Zefram said that he had fixed his in his repositories, but there has been
no movement since.

Devel-MAT has a fix and just needs a merge and release.

UNIVERSAL-ref's last release is from 2011. (September 2016, half a year
ago.)
optimizer.pm's last release was 2012. No comment on ticket (August 2016,
half a year ago.)
Sub-StopCalls is the same. No comment on ticket. (August 2016, half a
year ago.)
invoker is 2013. No comment on ticket. (August 2016, half a year ago.)
B-Tap failing is not fun, but it declares itself alpha code. No comment
on ticket. (Almost a year ago.)

Would having Devel-MAT, Devel-Cover, and ZEFRAM and ETHER dists be
enough for us, or do we consider the rest absolutely required as well?

@p5pRT
Copy link
Author

p5pRT commented Mar 14, 2017

From @jkeenan

On Tue, 14 Mar 2017 12​:08​:05 GMT, xsawyerx@​gmail.com wrote​:

On 03/13/2017 01​:29 PM, Dave Mitchell wrote​:

On Wed, Dec 07, 2016 at 12​:07​:25PM +0000, Dave Mitchell wrote​:

On Tue, Dec 06, 2016 at 01​:48​:15PM -0800, James E Keenan via RT
wrote​:

On Wed, 18 May 2016 19​:20​:25 GMT,
andreas.koenig.7os6VVqR@​franz.ak.mind.de wrote​:

bisect
------
commit 5d32d26
Author​: David Mitchell <davem@​iabyn.com>
Date​: Sat May 14 15​:14​:28 2016 +0100

enable PERL_OP_PARENT by default.
[snip]

while the following haven't had a new release yet​:

columns are​:
month and year of last release
F = there is a fix available with the ticket
rt.cpan or github ticket number

02/2012 #117385 ZEFRAM/Array-Base-0.005.tar.gz
05/2015 github #4 TOKUHIROM/B-Tap-0.15.tar.gz
04/2016 F github #161 PJCJ/Devel-Cover-1.23.tar.gz
06/2016 F #114581 PEVANS/Devel-MAT-0.23.tar.gz
01/2015 #117255 ZEFRAM/Hash-SharedMem-0.004.tar.gz
01/2015 #116694 ETHER/List-Gather-0.12.tar.gz
02/2012 #117387 RURBAN/optimizer-0.08.tar.gz
11/2010 F #114490 ZEFRAM/Params-Classify-0.013.tar.gz
02/2012 #117409 ZEFRAM/Scope-Cleanup-0.002.tar.gz
02/2012 #117410 ZEFRAM/String-Base-0.001.tar.gz
08/2012 #117338 RUZ/Sub-StopCalls-0.02.tar.gz
01/2011 #118008 JJORE/UNIVERSAL-ref-0.14.tar.gz
02/2013 #117215 CLKAO/invoker-0.34.tar.gz
01/2015 #117254 ETHER/Cond-Expr-0.04.tar.gz
Several belong to list members, so would be good to hear whether
they
intend to fix or is wontfix.

The original intention was to enable PERL_OP_PARENT by default early
in the
5.25.x cycle, then decide nearer to 5.26.0 release time whether to
back it
out or not.
If we're to back out PERL_OP_PARENT by default for 5.26.0, we need to
do
it very soon. Of the list above, there have been *no* new fixed
releases.

I privately emailed to Karen fixes for List-Gather and Cond-Expr, and
Zefram said that he had fixed his in his repositories, but there has
been
no movement since.

Devel-MAT has a fix and just needs a merge and release.

UNIVERSAL-ref's last release is from 2011. (September 2016, half a
year
ago.)
optimizer.pm's last release was 2012. No comment on ticket (August
2016,
half a year ago.)
Sub-StopCalls is the same. No comment on ticket. (August 2016, half a
year ago.)
invoker is 2013. No comment on ticket. (August 2016, half a year ago.)
B-Tap failing is not fun, but it declares itself alpha code. No
comment
on ticket. (Almost a year ago.)

Would having Devel-MAT, Devel-Cover, and ZEFRAM and ETHER dists be
enough for us, or do we consider the rest absolutely required as well?

All the CPAN authors have been notified and many already have a fix in place or in development. I see no need to revert this change. We have more contentious changes in the works, so we shouldn't spend any more tuits on this one.

Thank you very much.

--
James E Keenan (jkeenan@​cpan.org)

@p5pRT
Copy link
Author

p5pRT commented Mar 20, 2017

From @iabyn

On Tue, Mar 14, 2017 at 05​:35​:32AM -0700, James E Keenan via RT wrote​:

On Tue, 14 Mar 2017 12​:08​:05 GMT, xsawyerx@​gmail.com wrote​:

On 03/13/2017 01​:29 PM, Dave Mitchell wrote​:

If we're to back out PERL_OP_PARENT by default for 5.26.0, we need to
do
it very soon. Of the list above, there have been *no* new fixed
releases.

I privately emailed to Karen fixes for List-Gather and Cond-Expr, and
Zefram said that he had fixed his in his repositories, but there has
been
no movement since.

This thread concerns whether PERL_OP_PARENT should be kept enabled by
default for the 5.26.0 release in 2 months or so. I've included
specific questions for some people I've Cced in this email.

Karen, are the patches I emailed you a while ago suitable? Are you likely
to be able to produce new releases of

  01/2015 #116694 ETHER/List-Gather-0.12.tar.gz
  01/2015 #117254 ETHER/Cond-Expr-0.04.tar.gz

in time for 5.26.0?

Zefram, you said a while ago that you had fixes which would be released as
part of bigger fixes. Are they likely to be in time for 5.26.0?

  02/2012 #117385 ZEFRAM/Array-Base-0.005.tar.gz
  01/2015 #117255 ZEFRAM/Hash-SharedMem-0.004.tar.gz
  11/2010 F #114490 ZEFRAM/Params-Classify-0.013.tar.gz
  02/2012 #117409 ZEFRAM/Scope-Cleanup-0.002.tar.gz
  02/2012 #117410 ZEFRAM/String-Base-0.001.tar.gz

Paul, I understand a fix is in for Devel-MAT. Is there likely to be a
new release in time for 5.26.0?

As for the remaining distributions broken by PERL_OP_PARENT, here is
a reverse dependency tree for each distro, based on info from
metacpan.org (including month and year of last release)​:

05/2015 github #4 TOKUHIROM/B-Tap-0.15.tar.gz
  reverse deps​:
  09/2014 Test-Kantan-0.40
  05/2016 YATT-Lite-0.101
  12/2015 SQL-Concat-0.001
  12/2015 MOP4Import-Declare-0.004
  03/2014 Test-Power-0.14

02/2012 #117387 RURBAN/optimizer-0.08.tar.gz
  reverse deps​:
  06/2011 Acme-PERL-Autocorrect-1.20110629
  09/2011 optimize-0.05
  08/2002 types-0.05

08/2012 #117338 RUZ/Sub-StopCalls-0.02.tar.gz
  reverse deps​:
  --none--

01/2011 #118008 JJORE/UNIVERSAL-ref-0.14.tar.gz
  reverse deps​:
  09/2015 Test-Mocha-0.64
  07/2016 Test-Mountebank-0.001
  09/2014 DateTimeX-Auto-0.008
  09/2014 Types-XSD-0.005
  09/2014 String-Interpolate-Delayed-0.002
  09/2013 HTML-String-1.000006
  08/2013 Test-Magpie-0.11
  01/2011 Exception-NoException-0.07
  09/2010 Data-Thunk-0.07
  09/2014 Sub-Lazy-0.002
  05/2014 Acme-Fork-Lazy-0.03
  07/2010 Devel-Spy-0.07

02/2013 #117215 CLKAO/invoker-0.34.tar.gz
  reverse deps​:
  01/2013 methods-0.12
  01/2014 App-Lingua-BO-Wylie-Transliteration-0.1.0
  09/2013 App-assh-1.1.2
  03/2013 Plack-App-PgREST-0.06
  07/2012 Net-Jenkins-0.07
  07/2012 Jenkins-NotificationListener-0.06
  05/2012 Web-JenkinsNotification-0.05

Does anyone consider any of those distros important enough that we should
revert making PERL_OP_PARENT the default for 5.26.0?

--
My Dad used to say 'always fight fire with fire', which is probably why
he got thrown out of the fire brigade.

@p5pRT
Copy link
Author

p5pRT commented Mar 20, 2017

From @leonerd

On Mon, 20 Mar 2017 16​:55​:14 +0000
Dave Mitchell <davem@​iabyn.com> wrote​:

Paul, I understand a fix is in for Devel-MAT. Is there likely to be a
new release in time for 5.26.0?

Hello.

Hmm - this is the first I'm aware of an issue here.

I do see a similarish-sounding issue at

  https://rt.cpan.org/Ticket/Display.html?id=114581

perchance this is the same?

However - I believe I'm record as saying a number of times and I'll
repeat here again​: I accept that Devel​::MAT does seven-shades-of-crazy
with regard to perl internals. If a new perl release breaks it, don't
worry about that. I'm happy to accept that, and apply a fix at some
point; please don't let Devel​::MAT hold up a core release.

--
Paul "LeoNerd" Evans

leonerd@​leonerd.org.uk | https://metacpan.org/author/PEVANS
http​://www.leonerd.org.uk/ | https://www.tindie.com/stores/leonerd/

@p5pRT
Copy link
Author

p5pRT commented Mar 21, 2017

From @demerphq

On 20 March 2017 at 18​:41, Paul "LeoNerd" Evans <leonerd@​leonerd.org.uk> wrote​:

On Mon, 20 Mar 2017 16​:55​:14 +0000
Dave Mitchell <davem@​iabyn.com> wrote​:

Paul, I understand a fix is in for Devel-MAT. Is there likely to be a
new release in time for 5.26.0?

Hello.

Hmm - this is the first I'm aware of an issue here.

I do see a similarish-sounding issue at

https://rt.cpan.org/Ticket/Display.html?id=114581

perchance this is the same?

However - I believe I'm record as saying a number of times and I'll
repeat here again​: I accept that Devel​::MAT does seven-shades-of-crazy
with regard to perl internals. If a new perl release breaks it, don't
worry about that. I'm happy to accept that, and apply a fix at some
point; please don't let Devel​::MAT hold up a core release.

I wonder if this is something we should get the various people in our
universe to agree on a common standard.

I mean, if Paul could have gone to his pause account, for example, and
been able to set a flag on the distro about being of this nature, then
we on the core team could manage these things better. If we encourage
a culture of "hack, but flag", then we would be able to get a better
grip on the modules where the author is knowingly risking breakage in
new perl, and doesnt care, or want to be a blocker for perl updates...

Yves

--
perl -Mre=debug -e "/just|another|perl|hacker/"

@p5pRT
Copy link
Author

p5pRT commented Mar 21, 2017

From @iabyn

On Mon, Mar 20, 2017 at 05​:41​:02PM +0000, Paul "LeoNerd" Evans wrote​:

Hmm - this is the first I'm aware of an issue here.

I do see a similarish-sounding issue at

https://rt.cpan.org/Ticket/Display.html?id=114581

perchance this is the same?

Yes, that's it.

However - I believe I'm record as saying a number of times and I'll
repeat here again​: I accept that Devel​::MAT does seven-shades-of-crazy
with regard to perl internals. If a new perl release breaks it, don't
worry about that. I'm happy to accept that, and apply a fix at some
point; please don't let Devel​::MAT hold up a core release.

Ok thanks.

--
A power surge on the Bridge is rapidly and correctly diagnosed as a faulty
capacitor by the highly-trained and competent engineering staff.
  -- Things That Never Happen in "Star Trek" #9

@p5pRT
Copy link
Author

p5pRT commented Mar 28, 2017

From @jkeenan

On Mon, 20 Mar 2017 16​:56​:12 GMT, davem wrote​:

On Tue, Mar 14, 2017 at 05​:35​:32AM -0700, James E Keenan via RT wrote​:

On Tue, 14 Mar 2017 12​:08​:05 GMT, xsawyerx@​gmail.com wrote​:

On 03/13/2017 01​:29 PM, Dave Mitchell wrote​:

If we're to back out PERL_OP_PARENT by default for 5.26.0, we
need to
do
it very soon. Of the list above, there have been *no* new fixed
releases.

I privately emailed to Karen fixes for List-Gather and Cond-Expr,
and
Zefram said that he had fixed his in his repositories, but there
has
been
no movement since.

This thread concerns whether PERL_OP_PARENT should be kept enabled by
default for the 5.26.0 release in 2 months or so. I've included
specific questions for some people I've Cced in this email.

Karen, are the patches I emailed you a while ago suitable? Are you
likely
to be able to produce new releases of

01/2015 #116694 ETHER/List-Gather-0.12.tar.gz
01/2015 #117254 ETHER/Cond-Expr-0.04.tar.gz

in time for 5.26.0?

Perhaps you should attach them to these bug tickets​:

List-Gather​: https://rt.cpan.org/Ticket/Display.html?id=116694

Cond-Expr​: https://rt.cpan.org/Ticket/Display.html?id=117254

[snip]

--
James E Keenan (jkeenan@​cpan.org)

@p5pRT
Copy link
Author

p5pRT commented Mar 29, 2017

From @iabyn

On Tue, Mar 28, 2017 at 07​:18​:13AM -0700, James E Keenan via RT wrote​:

On Mon, 20 Mar 2017 16​:56​:12 GMT, davem wrote​:

Karen, are the patches I emailed you a while ago suitable? Are you
likely
to be able to produce new releases of

01/2015 #116694 ETHER/List-Gather-0.12.tar.gz
01/2015 #117254 ETHER/Cond-Expr-0.04.tar.gz

in time for 5.26.0?

Perhaps you should attach them to these bug tickets​:

List-Gather​: https://rt.cpan.org/Ticket/Display.html?id=116694

Cond-Expr​: https://rt.cpan.org/Ticket/Display.html?id=117254

Now attached.

--
Counsellor Troi states something other than the blindingly obvious.
  -- Things That Never Happen in "Star Trek" #16

@p5pRT
Copy link
Author

p5pRT commented Apr 16, 2017

From @iabyn

This is the ticket that concerns distributions broken by enabling
PERL_OP_PARENT by default for 5.26.0.

I think I've done about all I can for this, and propose removing it from
the 5.26.0 blockers list.

--
No man treats a motor car as foolishly as he treats another human being.
When the car will not go, he does not attribute its annoying behaviour to
sin, he does not say, You are a wicked motorcar, and I shall not give you
any more petrol until you go. He attempts to find out what is wrong and
set it right.
  -- Bertrand Russell,
  Has Religion Made Useful Contributions to Civilization?

@p5pRT
Copy link
Author

p5pRT commented Apr 20, 2017

From @pjcj

On Mon, Mar 13, 2017 at 01​:43​:39PM +0100, Paul Johnson wrote​:

On Mon, Mar 13, 2017 at 12​:29​:24PM +0000, Dave Mitchell wrote​:

04/2016 F github \#161  PJCJ/Devel\-Cover\-1\.23\.tar\.gz

Several belong to list members, so would be good to hear whether they
intend to fix or is wontfix.

The original intention was to enable PERL_OP_PARENT by default early in the
5.25.x cycle, then decide nearer to 5.26.0 release time whether to back it
out or not.

If we're to back out PERL_OP_PARENT by default for 5.26.0, we need to do
it very soon. Of the list above, there have been *no* new fixed releases.

I *will* get a new release of Devel​::Cover out soon incorporating this
fix.

Not sure whether it can be classified as soon - in the grand scheme of
this problem it probably is - but Devel​::Cover 1.24 is now released
incorporating this fix. It works with all 5.25 versions including
5.25.12 so it looks like Devel​::Cover, at least, is ready for 5.26.0.

--
Paul Johnson - paul@​pjcj.net
http​://www.pjcj.net

@p5pRT
Copy link
Author

p5pRT commented May 20, 2017

From @ikegami

Also breaks

  IKEGAMI/Syntax-Feature-Loop-v1.6.0.tar.gz
  IKEGAMI/Syntax-Feature-QwComments-v1.12.0.tar.gz

Will fix this weekend.

1 similar comment
@p5pRT
Copy link
Author

p5pRT commented May 20, 2017

From @ikegami

Also breaks

  IKEGAMI/Syntax-Feature-Loop-v1.6.0.tar.gz
  IKEGAMI/Syntax-Feature-QwComments-v1.12.0.tar.gz

Will fix this weekend.

@p5pRT
Copy link
Author

p5pRT commented Dec 23, 2017

From @jkeenan

On Sat, 20 May 2017 00​:47​:15 GMT, ikegami@​adaelis.com wrote​:

Also breaks

IKEGAMI/Syntax\-Feature\-Loop\-v1\.6\.0\.tar\.gz
IKEGAMI/Syntax\-Feature\-QwComments\-v1\.12\.0\.tar\.gz

Will fix this weekend.

Were these fixed?

--
James E Keenan (jkeenan@​cpan.org)

@p5pRT p5pRT added the BBC Blead Breaks CPAN - changes in blead broke a cpan module(s) label Oct 19, 2019
@toddr
Copy link
Member

toddr commented Feb 17, 2020

@jkeenan this is an old BBC ticket. Are we ok to close it?

@toddr toddr added the Closable? We might be able to close this ticket, but we need to check with the reporter label Feb 17, 2020
@eserte
Copy link
Contributor

eserte commented Feb 17, 2020

Also breaks

IKEGAMI/Syntax-Feature-Loop-v1.6.0.tar.gz
IKEGAMI/Syntax-Feature-QwComments-v1.12.0.tar.gz

Will fix this weekend.

Were these fixed?

I see newer versions for these modules, but there's a dependency on Lexical-Sub which is broken for newer perls (https://rt.cpan.org/Dist/Display.html?Name=Lexical-Var).

@eserte
Copy link
Contributor

eserte commented Feb 17, 2020

From the modules mentioned in this ticket (except the revdeps) the following do not have a fixed stable release:

  • TOKUHIROM/B-Tap-0.15.tar.gz
  • ADUITSIS/IPC-Mmap-0.21.tar.gz (but the latest devel version works)
  • RURBAN/optimizer-0.08.tar.gz
  • RUZ/Sub-StopCalls-0.02.tar.gz
  • JJORE/UNIVERSAL-ref-0.14.tar.gz
  • ETHER/Sub-Call-Tail-0.06.tar.gz

@toddr
Copy link
Member

toddr commented Feb 17, 2020

@eserte IMO the point of this ticket was to track and decide if we needed to revert/fix problems prior to the release of 5.30. Now 5.30 is out, I think we should be closing this ticket.

If tickets should be opened as a result of changes in core, I'm unclear that it is the responsibility of perl/perl5 to do so. My concern is that we would have to leave A LOT of tickets open indefinitely if the policy was that all of CPAN had to be working before we could close anything.

That is my opinion though, how do you see this should be handled?

@eserte
Copy link
Contributor

eserte commented Feb 17, 2020

Formally, if the breaking change is documented in perldelta and all affected modules are notified, then there's no need to keep the issue open. I just wanted to give a small overview of the still broken modules — maybe somebody here finds one of these (or the unlisted revdeps) important enough to fix it?

@toddr
Copy link
Member

toddr commented Feb 17, 2020

It was documented.

These were reported.

This one is @karenetheridge 's module. I believe it is broken on something earlier so its kinda moot that this is broken there: https://rt.cpan.org/Public/Bug/Display.html?id=112244

  • ETHER/Sub-Call-Tail-0.06.tar.gz

@toddr toddr closed this as completed Feb 17, 2020
@ikegami
Copy link
Contributor

ikegami commented Feb 18, 2020

From @jkeenan

On Sat, 20 May 2017 00​:47​:15 GMT, ikegami@​adaelis.com wrote​:

Also breaks

IKEGAMI/Syntax\-Feature\-Loop\-v1\.6\.0\.tar\.gz
IKEGAMI/Syntax\-Feature\-QwComments\-v1\.12\.0\.tar\.gz

Will fix this weekend.

Were these fixed?

Yes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BBC Blead Breaks CPAN - changes in blead broke a cpan module(s) Closable? We might be able to close this ticket, but we need to check with the reporter
Projects
None yet
Development

No branches or pull requests

4 participants