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.21.6-473-g8c2e27d breaks GFUJI/macro-0.06.tar.gz #14545

Closed
p5pRT opened this issue Feb 27, 2015 · 7 comments
Labels

Comments

@p5pRT
Copy link
Collaborator

@p5pRT p5pRT commented Feb 27, 2015

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

Searchable as RT123947$

@p5pRT

This comment has been minimized.

Copy link
Collaborator Author

@p5pRT p5pRT commented Feb 27, 2015

From @andk

bisect


commit 8c2e27d
Author​: Father Chrysostomos <sprout@​cpan.org>
Date​: Fri Dec 12 22​:42​:11 2014 -0800

  Deparse $#{1} with braces

sample report


http​://www.cpantesters.org/cpan/report/5635ff42-be29-11e4-9d50-88838971dd2f

Perl -V


Summary of my perl5 (revision 5 version 21 subversion 7) configuration​:
  Commit id​: 846dac6
  Platform​:
  osname=linux, osvers=3.16.0-4-amd64, archname=x86_64-linux-ld
  uname='linux k83 3.16.0-4-amd64 #1 smp debian 3.16.7-ckt2-1 (2014-12-08) x86_64 gnulinux '
  config_args='-Dprefix=/home/sand/src/perl/repoperls/installed-perls/perl/v5.21.7/127e -Dmyhostname=k83 -Dinstallusrbinperl=n -Uversiononly -Dusedevel -des -Ui_db -Uuseithreads -Duselongdouble -DDEBUGGING=-g'
  hint=recommended, useposix=true, d_sigaction=define
  useithreads=undef, usemultiplicity=undef
  use64bitint=define, use64bitall=define, uselongdouble=define
  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 -g',
  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='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/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=-lnsl -lgdbm -ldb -ldl -lm -lcrypt -lutil -lc -lpthread -lgdbm_compat
  perllibs=-lnsl -ldl -lm -lcrypt -lutil -lc -lpthread
  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 -g -L/usr/local/lib -fstack-protector-strong'

Characteristics of this binary (from libperl)​:
  Compile-time options​: HAS_TIMES PERLIO_LAYERS PERL_DONT_CREATE_GVSV
  PERL_HASH_FUNC_ONE_AT_A_TIME_HARD PERL_MALLOC_WRAP
  PERL_NEW_COPY_ON_WRITE 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_LONG_DOUBLE USE_PERLIO USE_PERL_ATOF
  Built under linux
  Compiled at Jan 10 2015 00​:45​:48
  %ENV​:
  PERL5LIB=""
  PERL5OPT=""
  PERL5_CPANPLUS_IS_RUNNING="19515"
  PERL5_CPAN_IS_RUNNING="19515"
  PERL_MM_USE_DEFAULT="1"
  @​INC​:
  /home/sand/src/perl/repoperls/installed-perls/perl/v5.21.7/127e/lib/site_perl/5.21.7/x86_64-linux-ld
  /home/sand/src/perl/repoperls/installed-perls/perl/v5.21.7/127e/lib/site_perl/5.21.7
  /home/sand/src/perl/repoperls/installed-perls/perl/v5.21.7/127e/lib/5.21.7/x86_64-linux-ld
  /home/sand/src/perl/repoperls/installed-perls/perl/v5.21.7/127e/lib/5.21.7
  .
--
andreas

@p5pRT

This comment has been minimized.

Copy link
Collaborator Author

@p5pRT p5pRT commented Mar 2, 2015

From @tonycoz

On Thu Feb 26 23​:40​:31 2015, andreas.koenig.7os6VVqR@​franz.ak.mind.de wrote​:

bisect
------
commit 8c2e27d
Author​: Father Chrysostomos <sprout@​cpan.org>
Date​: Fri Dec 12 22​:42​:11 2014 -0800

Deparse $#{1} with braces

Reported upstream as https://rt.cpan.org/Public/Bug/Display.html?id=102439 by Slaven Rezić.

Tony

@p5pRT

This comment has been minimized.

Copy link
Collaborator Author

@p5pRT p5pRT commented Mar 2, 2015

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

@p5pRT

This comment has been minimized.

Copy link
Collaborator Author

@p5pRT p5pRT commented Mar 2, 2015

From @tonycoz

On Thu Feb 26 23​:40​:31 2015, andreas.koenig.7os6VVqR@​franz.ak.mind.de wrote​:

bisect
------
commit 8c2e27d
Author​: Father Chrysostomos <sprout@​cpan.org>
Date​: Fri Dec 12 22​:42​:11 2014 -0800

Deparse $#{1} with braces

The change also converts $#_ into $#{_} which is unnecessary.

Unfortunately macro​::__want_param() and macro​::__param_replace() can't handle that form of access to @​_, and so $#{_} isn't replaced with the actual macro parameter.

Since we don't need to deparse $#_ as $#{_} I've modified B​::Deparse so it's again deparsed as $#_.

Tony

@p5pRT

This comment has been minimized.

Copy link
Collaborator Author

@p5pRT p5pRT commented Mar 2, 2015

From @tonycoz

On Sun Mar 01 22​:12​:40 2015, tonyc wrote​:

Since we don't need to deparse $#_ as $#{_} I've modified B​::Deparse
so it's again deparsed as $#_.

Oops, in b6bba88.

Tony

@p5pRT

This comment has been minimized.

Copy link
Collaborator Author

@p5pRT p5pRT commented Mar 18, 2015

From @steve-m-hay

Fixed in b6bba88 so closing this ticket.

@p5pRT p5pRT closed this Mar 18, 2015
@p5pRT

This comment has been minimized.

Copy link
Collaborator Author

@p5pRT p5pRT commented Mar 18, 2015

@steve-m-hay - Status changed from 'open' to 'resolved'

@p5pRT p5pRT added the Severity Low label Oct 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.