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.7-165-g613abc6 breaks PERLANCAR/Org-Parser-0.43.tar.gz #14381

Closed
p5pRT opened this issue Dec 31, 2014 · 7 comments
Closed

Bleadperl v5.21.7-165-g613abc6 breaks PERLANCAR/Org-Parser-0.43.tar.gz #14381

p5pRT opened this issue Dec 31, 2014 · 7 comments

Comments

@p5pRT
Copy link

p5pRT commented Dec 31, 2014

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

Searchable as RT123527$

@p5pRT
Copy link
Author

p5pRT commented Dec 31, 2014

From @andk

bisect


commit 613abc6
Author​: Karl Williamson <khw@​cpan.org>
Date​: Mon Dec 29 13​:15​:57 2014 -0700

  Raise warning on multi-byte char in single-byte locale

diagnostics


http​://www.cpantesters.org/cpan/report/972d6b7e-905d-11e4-9c21-482a5ef060ad

perl -V


Summary of my perl5 (revision 5 version 21 subversion 8) configuration​:
  Commit id​: 215c513
  Platform​:
  osname=linux, osvers=3.16.0-4-amd64, archname=x86_64-linux
  uname='linux k83 3.16.0-4-amd64 #1 smp debian 3.16.7-2 (2014-11-06) x86_64 gnulinux '
  config_args='-Dprefix=/home/sand/src/perl/repoperls/installed-perls/perl/v5.21.7-167-g215c513/165a -Dmyhostname=k83 -Dinstallusrbinperl=n -Uversiononly -Dusedevel -des -Ui_db -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 -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='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=-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_PERLIO USE_PERL_ATOF
  Built under linux
  Compiled at Dec 30 2014 03​:14​:54
  %ENV​:
  PERL5LIB="/tmp/loop_over_bdir-20539-7xysKW/FusionInventory-Agent-2.3.14-iVWWzL/blib/arch​:/tmp/loop_over_bdir-20539-7xysKW/FusionInventory-Agent-2.3.14-iVWWzL/blib/lib"
  PERL5OPT=""
  PERL5_CPANPLUS_IS_RUNNING="25666"
  PERL5_CPAN_IS_RUNNING="25666"
  PERL_MM_USE_DEFAULT="1"
  @​INC​:
  /tmp/loop_over_bdir-20539-7xysKW/FusionInventory-Agent-2.3.14-iVWWzL/blib/arch
  /tmp/loop_over_bdir-20539-7xysKW/FusionInventory-Agent-2.3.14-iVWWzL/blib/lib
  /home/sand/src/perl/repoperls/installed-perls/perl/v5.21.7-167-g215c513/165a/lib/site_perl/5.21.8/x86_64-linux
  /home/sand/src/perl/repoperls/installed-perls/perl/v5.21.7-167-g215c513/165a/lib/site_perl/5.21.8
  /home/sand/src/perl/repoperls/installed-perls/perl/v5.21.7-167-g215c513/165a/lib/5.21.8/x86_64-linux
  /home/sand/src/perl/repoperls/installed-perls/perl/v5.21.7-167-g215c513/165a/lib/5.21.8
  .
--
andreas

@p5pRT
Copy link
Author

p5pRT commented Mar 28, 2015

From @khwilliamson

On Wed Dec 31 04​:41​:16 2014, andreas.koenig.7os6VVqR@​franz.ak.mind.de wrote​:

bisect
------
commit 613abc6
Author​: Karl Williamson <khw@​cpan.org>
Date​: Mon Dec 29 13​:15​:57 2014 -0700

Raise warning on multi-byte char in single-byte locale

diagnostics
-----------
http​://www.cpantesters.org/cpan/report/972d6b7e-905d-11e4-9c21-
482a5ef060ad

There was an error in this patch that has now been fixed in blead with 008e8e8.

However, the warnings will still be raised when the locale is not UTF-8. The cpantester report was using a UTF-8 locale, so if it were not for the blead bug, it might have missed the problem with non-UTF-8 locales. I'm not sure what the best way forward is. If the cpan testing doesn't try with a non-UTF-8 locale, this bug will appear "fixed" to it.

The module's tests are assuming a UTF-8 locale without checking (AFAIK), so there is an issue in the module or its tests. I have submitted a pull request to the module containing the simplest solution,

perlancar/perl-Org-Parser#5

but there are perhaps better alternatives, which I've spelled out as best as I know in the commit message, so that the author can make a determination of how to proceed.
--
Karl Williamson

@p5pRT
Copy link
Author

p5pRT commented Mar 28, 2015

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

@p5pRT
Copy link
Author

p5pRT commented Apr 20, 2015

From @khwilliamson

Since a pull request has been issued for the distribution, I've removed this from the list of blockers for v5.22

--
Karl Williamson

@p5pRT
Copy link
Author

p5pRT commented Apr 20, 2015

From [Unknown Contact. See original ticket]

Since a pull request has been issued for the distribution, I've removed this from the list of blockers for v5.22

--
Karl Williamson

@p5pRT
Copy link
Author

p5pRT commented Aug 24, 2015

From @khwilliamson

Since there is no longer anything for the perl core to do regarding this, I'm closing the ticket, even though the pull request for the module has not been acted upon
--
Karl Williamson

@p5pRT p5pRT closed this as completed Aug 24, 2015
@p5pRT
Copy link
Author

p5pRT commented Aug 24, 2015

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