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.9-10-g3a0fe090c1 breaks SPROUT/JE-0.066.tar.gz #15832

Closed
p5pRT opened this issue Jan 25, 2017 · 8 comments
Closed

Bleadperl v5.25.9-10-g3a0fe090c1 breaks SPROUT/JE-0.066.tar.gz #15832

p5pRT opened this issue Jan 25, 2017 · 8 comments

Comments

@p5pRT
Copy link

p5pRT commented Jan 25, 2017

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

Searchable as RT130639$

@p5pRT
Copy link
Author

p5pRT commented Jan 25, 2017

From @andk

bisect


commit 3a0fe09
Author​: Zefram <zefram@​fysh.org>
Date​: Sun Jan 22 03​:20​:08 2017 +0000

  handle errors in gen_constant_list

diagnostics


Fail report shortly after commit​: http​://www.cpantesters.org/cpan/report/2f3bb378-e177-11e6-af27-bba69f28caea
Pass report immediately before commit​: http​://www.cpantesters.org/cpan/report/8bca308e-e077-11e6-a267-6a5f9f28caea

perl -V


Summary of my perl5 (revision 5 version 25 subversion 10) configuration​:
  Commit id​: 3a0fe09
  Platform​:
  osname=linux
  osvers=4.8.0-2-amd64
  archname=x86_64-linux
  uname='linux k93msid 4.8.0-2-amd64 #1 smp debian 4.8.11-1 (2016-12-02) x86_64 gnulinux '
  config_args='-Dprefix=/home/sand/src/perl/repoperls/installed-perls/host/k93msid/v5.25.9-10-g3a0fe090c1/89ad -Dmyhostname=k93msid -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 -Uuseithreads -Uuselongdouble -DDEBUGGING=-g'
  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'
  optimize='-O2 -g'
  cppflags='-fwrapv -fno-strict-aliasing -pipe -fstack-protector-strong -I/usr/local/include'
  ccversion=''
  gccversion='6.2.1 20161124'
  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/6/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 -ldl -lm -lcrypt -lutil -lc
  perllibs=-lpthread -lnsl -ldl -lm -lcrypt -lutil -lc
  libc=libc-2.24.so
  so=so
  useshrplib=false
  libperl=libperl.a
  gnulibc_version='2.24'
  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_COPY_ON_WRITE
  PERL_DONT_CREATE_GVSV
  PERL_MALLOC_WRAP
  PERL_OP_PARENT
  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 Jan 22 2017 03​:43​:33
  @​INC​:
  /home/sand/src/perl/repoperls/installed-perls/host/k93msid/v5.25.9-10-g3a0fe090c1/89ad/lib/site_perl/5.25.10/x86_64-linux
  /home/sand/src/perl/repoperls/installed-perls/host/k93msid/v5.25.9-10-g3a0fe090c1/89ad/lib/site_perl/5.25.10
  /home/sand/src/perl/repoperls/installed-perls/host/k93msid/v5.25.9-10-g3a0fe090c1/89ad/lib/5.25.10/x86_64-linux
  /home/sand/src/perl/repoperls/installed-perls/host/k93msid/v5.25.9-10-g3a0fe090c1/89ad/lib/5.25.10
  .

--
andreas

@p5pRT
Copy link
Author

p5pRT commented Jan 25, 2017

From zefram@fysh.org

Subject​: [perl #130639] Bleadperl v5.25.9-10-g3a0fe090c1 breaks
SPROUT/JE-0.066.tar.gz

In my own tests, JE-0.066 fails both before and after the commit, in
identical fashion. There's a lot of

  perl​: utf8.c​:1418​: Perl_utf8n_to_uvchr_error​: Assertion `0' failed.

So I'm dubious about the identification of the breaking commit.

-zefram

@p5pRT
Copy link
Author

p5pRT commented Jan 25, 2017

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

@p5pRT
Copy link
Author

p5pRT commented Jan 26, 2017

From @khwilliamson

On Wed, 25 Jan 2017 13​:31​:26 -0800, zefram@​fysh.org wrote​:

Subject​: [perl #130639] Bleadperl v5.25.9-10-g3a0fe090c1 breaks
SPROUT/JE-0.066.tar.gz

In my own tests, JE-0.066 fails both before and after the commit, in
identical fashion. There's a lot of

perl&#8203;: utf8\.c&#8203;:1418&#8203;: Perl\_utf8n\_to\_uvchr\_error&#8203;: Assertion \`0' failed\.

So I'm dubious about the identification of the breaking commit.

-zefram

Those asserts are to catch in DEBUGGING builds violations of the changed API. It is now illegal to call Perl_utf8n_to_uvchr with input length 0. If you have nothing to decode, don't try to decode.
--
Karl Williamson

@p5pRT
Copy link
Author

p5pRT commented Jan 26, 2017

From zefram@fysh.org

Suppressing the utf8 assertions, I think I've got to what this ticket
is about. The commit identification is fine. The problem reduces to:

$ good_perl -lwe 'print ":".."~"; print 1'
:
1
$ bad_perl -lwe 'print ":".."~"; print 1'
Argument ":" isn't numeric in range (or flop) at -e line 1.
$

-zefram

@p5pRT
Copy link
Author

p5pRT commented Jan 27, 2017

From zefram@fysh.org

Fixed by b369834.

-zefram

@p5pRT
Copy link
Author

p5pRT commented Mar 20, 2017

From @iabyn

On Fri, Jan 27, 2017 at 04​:44​:02AM +0000, Zefram wrote​:

Fixed by b369834.

... so I'm closing the ticket.

--
"Strange women lying in ponds distributing swords is no basis for a system
of government. Supreme executive power derives from a mandate from the
masses, not from some farcical aquatic ceremony."
  -- Dennis, "Monty Python and the Holy Grail"

@p5pRT
Copy link
Author

p5pRT commented Mar 20, 2017

@iabyn - 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