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.0-277-ga835cd4 breaks WRW/Number-Format-1.73.tar.gz #13995

Closed
p5pRT opened this issue Jul 22, 2014 · 5 comments
Closed

Bleadperl v5.21.0-277-ga835cd4 breaks WRW/Number-Format-1.73.tar.gz #13995

p5pRT opened this issue Jul 22, 2014 · 5 comments
Labels

Comments

@p5pRT
Copy link
Collaborator

@p5pRT p5pRT commented Jul 22, 2014

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

Searchable as RT122367$

@p5pRT
Copy link
Collaborator Author

@p5pRT p5pRT commented Jul 22, 2014

From @andk

git bisect


a835cd4 is the first bad commit
commit a835cd4
Author​: Karl Williamson <khw@​cpan.org>
Date​: Thu May 15 15​:40​:51 2014 -0600

  Localeconv() should be independent of 'use locale'

sample fail report


Kudos to Slaven Rezic for pointing me to his tests under German LC_ALL

http​://www.cpantesters.org/cpan/report/f1c97e30-0a69-11e4-acd2-446de0bfc7aa

The German locale is required to reproduce.

perl -V


Summary of my perl5 (revision 5 version 21 subversion 1) configuration​:
 
  Platform​:
  osname=linux, osvers=3.2.0-4-amd64, archname=x86_64-linux
  uname='linux eserte 3.2.0-4-amd64 #1 smp debian 3.2.57-3+deb7u2
  x86_64 gnulinux '
  config_args='-ds -e -Dprefix=/opt/perl-5.21.1 -Dusedevel
  -Dusemallocwrap=no'
  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 -I/usr/local/include -D_LARGEFILE_SOURCE
  -D_FILE_OFFSET_BITS=64',
  optimize='-O2',
  cppflags='-fwrapv -fno-strict-aliasing -pipe -fstack-protector
  -I/usr/local/include'
  ccversion='', gccversion='4.7.2', gccosandvers=''
  intsize=4, longsize=8, ptrsize=8, doublesize=8, byteorder=12345678
  d_longlong=define, longlongsize=8, d_longdbl=define, longdblsize=16
  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 -L/usr/local/lib'
  libpth=/usr/local/lib
  /usr/lib/gcc/x86_64-linux-gnu/4.7/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 -ldb -ldl -lm -lcrypt -lutil -lc
  perllibs=-lnsl -ldl -lm -lcrypt -lutil -lc
  libc=libc-2.13.so, so=so, useshrplib=false, libperl=libperl.a
  gnulibc_version='2.13'
  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'

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_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_PERLIO
  USE_PERL_ATOF
  Built under linux
  Compiled at Jul 13 2014 06​:38​:59
  %ENV​:
  PERL5LIB=""
  PERL5OPT=""
  PERL5_CPANPLUS_IS_RUNNING="14389"
  PERL5_CPAN_IS_RUNNING="14389"
  PERL5_CPAN_IS_RUNNING_IN_RECURSION="14310,14389"
  PERLDOC="-MPod​::Perldoc​::ToTextOverstrike"
  PERL_BATCH="yes"
  PERL_CPAN_REPORTER_CONFIG="/var/tmp/cpansmoker-1001/JojfIyfc7z_config.ini"
  PERL_EXTUTILS_AUTOINSTALL="--defaultdeps"
  PERL_HTML_DISPLAY_CLASS="HTML​::Display​::Mozilla"
  PERL_PARALLEL_SMOKER="yes"
  @​INC​:
  /opt/perl-5.21.1/lib/site_perl/5.21.1/x86_64-linux
  /opt/perl-5.21.1/lib/site_perl/5.21.1
  /opt/perl-5.21.1/lib/5.21.1/x86_64-linux
  /opt/perl-5.21.1/lib/5.21.1
  .

--
andreas

@p5pRT
Copy link
Collaborator Author

@p5pRT p5pRT commented Jul 29, 2014

From @khwilliamson

On Tue Jul 22 04​:49​:26 2014, andreas.koenig.7os6VVqR@​franz.ak.mind.de wrote​:

git bisect
----------
a835cd4 is the first bad commit
commit a835cd4
Author​: Karl Williamson <khw@​cpan.org>
Date​: Thu May 15 15​:40​:51 2014 -0600

Localeconv() should be independent of 'use locale'

sample fail report
------------------
Kudos to Slaven Rezic for pointing me to his tests under German LC_ALL

http​://www.cpantesters.org/cpan/report/f1c97e30-0a69-11e4-acd2-
446de0bfc7aa

The German locale is required to reproduce.

This is a bug in the module's test files. The blamed commit is actually a bug fix for the module. This same problem surfaced earlier, and details of the problem are here​:
https://rt-archive.perl.org/perl5/Ticket/Display.html?id=121317#txn-1284765

The author did not fix anything, so I created a patch to the module this time
https://rt.cpan.org/Public/Bug/Display.html?id=97607
It's been 5 years since the module was updated; some issues marked critical are in its RT queue. This patch may fix some of them.
--
Karl Williamson

@p5pRT
Copy link
Collaborator Author

@p5pRT p5pRT commented Jul 29, 2014

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

@p5pRT
Copy link
Collaborator Author

@p5pRT p5pRT commented Sep 18, 2014

From @khwilliamson

This is a bug not in Perl but the module. A patch was submitted to it
--
Karl Williamson

@p5pRT p5pRT closed this Sep 18, 2014
@p5pRT
Copy link
Collaborator Author

@p5pRT p5pRT commented Sep 18, 2014

@khwilliamson - 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
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.