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.23.2-283-g7f399b4 breaks MSCHWERN/perl5i-v2.13.1.tar.gz #15025

Closed
p5pRT opened this issue Nov 3, 2015 · 7 comments
Closed

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

p5pRT opened this issue Nov 3, 2015 · 7 comments

Comments

@p5pRT
Copy link

p5pRT commented Nov 3, 2015

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

Searchable as RT126545$

@p5pRT
Copy link
Author

p5pRT commented Nov 3, 2015

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
Copy link
Author

p5pRT commented Nov 3, 2015

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
Copy link
Author

p5pRT commented Nov 3, 2015

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

@p5pRT
Copy link
Author

p5pRT commented Mar 23, 2016

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
Copy link
Author

p5pRT commented Mar 23, 2016

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
Copy link
Author

p5pRT commented Apr 7, 2016

From @rjbs

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

--
rjbs

@p5pRT p5pRT closed this as completed Apr 7, 2016
@p5pRT
Copy link
Author

p5pRT commented Apr 7, 2016

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant