Bleadperl v5.23.2-283-g7f399b4 breaks MSCHWERN/perl5i-v2.13.1.tar.gz

[p5pRT]

Migrated from rt.perl.org#126545 (status was 'resolved')

Searchable as RT126545$

[p5pRT]

From @andk

Thanks to Slaven Rezić for the discovery.

bisect

---

commit 7f399b4
Author​: Father Chrysostomos <sprout@​cpan.org>
Date​: Sun Sep 20 15​:06​:39 2015 -0700

  [perl #126064] Apply scalar context to stat args

cpantesters

---

http​://www.cpantesters.org/cpan/report/6ac0d19e-80c5-11e5-a5a6-d3d2e0bfc7aa

perl -V

---

Summary of my perl5 (revision 5 version 23 subversion 3) configuration​:
 
  Platform​:
  osname=linux, osvers=3.16.0-4-amd64, archname=x86_64-linux
  uname='linux cvrsnica-jessie 3.16.0-4-amd64 #1 smp debian 3.16.7-ckt9-3~deb8u1 (2015-04-24) x86_64 gnulinux '
  config_args='-ds -e -Dprefix=/opt/perl-5.23.3 -Dusedevel -Dusemallocwrap=no -Dcf_email=srezic@​cpan.org'
  hint=recommended, useposix=true, d_sigaction=define
  useithreads=undef, usemultiplicity=undef
  use64bitint=define, use64bitall=define, uselongdouble=undef
  usemymalloc=n, bincompat5005=undef
  Compiler​:
  cc='cc', ccflags ='-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',
  cppflags='-fwrapv -fno-strict-aliasing -pipe -fstack-protector-strong -I/usr/local/include'
  ccversion='', gccversion='4.9.2', 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='double', nvsize=8, Off_t='off_t', lseeksize=8
  alignbytes=8, 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/4.9/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 -ldb -ldl -lm -lcrypt -lutil -lc -lgdbm_compat
  perllibs=-lpthread -lnsl -ldl -lm -lcrypt -lutil -lc
  libc=libc-2.19.so, so=so, useshrplib=false, libperl=libperl.a
  gnulibc_version='2.19'
  Dynamic Linking​:
  dlsrc=dl_dlopen.xs, dlext=so, d_dlsymun=undef, ccdlflags='-Wl,-E'
  cccdlflags='-fPIC', lddlflags='-shared -O2 -L/usr/local/lib -fstack-protector-strong'

Characteristics of this binary (from libperl)​:
  Compile-time options​: HAS_TIMES PERLIO_LAYERS PERL_COPY_ON_WRITE
  PERL_DONT_CREATE_GVSV
  PERL_HASH_FUNC_ONE_AT_A_TIME_HARD PERL_PRESERVE_IVUV
  PERL_USE_DEVEL USE_64_BIT_ALL USE_64_BIT_INT
  USE_LARGE_FILES USE_LOCALE USE_LOCALE_COLLATE
  USE_LOCALE_CTYPE USE_LOCALE_NUMERIC USE_LOCALE_TIME
  USE_PERLIO USE_PERL_ATOF
  Built under linux
  Compiled at Oct 16 2015 08​:39​:42
  %ENV​:
  [...]

--
andreas

[p5pRT]

From @dcollinsn

This is, I believe, a duplicate of 126162 (
https://rt-archive.perl.org/perl5/Ticket/Display.html?id=126162), where we discuss how
stat() should handle being passed an array (or more specifically, how it
should not). Patches in that bug add warnings and perldelta notes, but seem
to have been forgotten as there is no update since late September.

[p5pRT]

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

[p5pRT]

From @tonycoz

On Mon Nov 02 20​:52​:54 2015, andreas.koenig.7os6VVqR@​franz.ak.mind.de wrote​:

Thanks to Slaven Rezić for the discovery.

bisect
------
commit 7f399b4
Author​: Father Chrysostomos <sprout@​cpan.org>
Date​: Sun Sep 20 15​:06​:39 2015 -0700

[perl #126064] Apply scalar context to stat args

cpantesters
-----------
http​://www.cpantesters.org/cpan/report/6ac0d19e-80c5-11e5-a5a6-
d3d2e0bfc7aa

Upstream, this has an open ticket (https​://github.com/evalEmpire/perl5i/issues/291) since November, and a PR (evalEmpire/perl5i#293) since yesterday.

I think we can close it.

Tony

[p5pRT]

From @rjbs

On Tue Nov 03 07​:25​:44 2015, dcollinsn@​gmail.com wrote​:

This is, I believe, a duplicate of 126162 (
https://rt-archive.perl.org/perl5/Ticket/Display.html?id=126162), where we discuss how
stat() should handle being passed an array (or more specifically, how it
should not). Patches in that bug add warnings and perldelta notes, but seem
to have been forgotten as there is no update since late September.

I think Dan is right. We should probably apply the patch in #126162 and perl5i should change from @​_ to $_[0].

Thoughts?

--
rjbs

[p5pRT]

From @rjbs

I've applied the warnings. There is no action needed for this ticket.

--
rjbs

[p5pRT]

@rjbs - Status changed from 'open' to 'resolved'