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-387-gb06e47f breaks ETHER/WWW-Mechanize-1.73.tar.gz #13934

Closed
p5pRT opened this issue Jun 17, 2014 · 11 comments
Closed

Bleadperl v5.21.0-387-gb06e47f breaks ETHER/WWW-Mechanize-1.73.tar.gz #13934

p5pRT opened this issue Jun 17, 2014 · 11 comments

Comments

@p5pRT
Copy link

p5pRT commented Jun 17, 2014

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

Searchable as RT122121$

@p5pRT
Copy link
Author

p5pRT commented Jun 17, 2014

From @andk

git bisect


commit b06e47f
Author​: Karl Williamson <khw@​cpan.org>
Date​: Fri Jun 13 11​:18​:01 2014 -0600

  Experimentally cause 'use encoding' to fail.

also affected


MICHEL/Business-KontoCheck-5.4.tar.gz
MIYAGAWA/XML-Atom-0.41.tar.gz
GFUJI/HTML-FillInForm-Lite-1.13.tar.gz
DYACOB/Regexp-Ethiopic-0.15.tar.gz
COSIMO/Device-Gsm-1.60.tar.gz
YKAR/WWW-Google-PageRank-0.17.tar.gz
DDB/Convert-YText-v0.1.2.tar.gz
AUDREYT/Convert-EastAsianWidth-1.02.tar.gz

There will certainly be more.

perl -V


Summary of my perl5 (revision 5 version 21 subversion 1) configuration​:
  Commit id​: b06e47f
  Platform​:
  osname=linux, osvers=3.14-1-amd64, archname=x86_64-linux-thread-multi-ld
  uname='linux k83 3.14-1-amd64 #1 smp debian 3.14.5-1 (2014-06-05) x86_64 gnulinux '
  config_args='-Dprefix=/home/sand/src/perl/repoperls/installed-perls/perl/v5.21.0-387-gb06e47f/a2da -Dmyhostname=k83 -Dinstallusrbinperl=n -Uversiononly -Dusedevel -des -Ui_db -Duseithreads -Duselongdouble -DDEBUGGING=-g'
  hint=recommended, useposix=true, d_sigaction=define
  useithreads=define, usemultiplicity=define
  use64bitint=define, use64bitall=define, uselongdouble=define
  usemymalloc=n, bincompat5005=undef
  Compiler​:
  cc='cc', ccflags ='-D_REENTRANT -D_GNU_SOURCE -fwrapv -fno-strict-aliasing -pipe -fstack-protector -I/usr/local/include -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64',
  optimize='-O2 -g',
  cppflags='-D_REENTRANT -D_GNU_SOURCE -fwrapv -fno-strict-aliasing -pipe -fstack-protector -I/usr/local/include'
  ccversion='', gccversion='4.8.3', 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='long double', nvsize=16, Off_t='off_t', lseeksize=8
  alignbytes=16, 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.8/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 -lpthread -lc -lgdbm_compat
  perllibs=-lnsl -ldl -lm -lcrypt -lutil -lpthread -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 -g -L/usr/local/lib -fstack-protector'

Characteristics of this binary (from libperl)​:
  Compile-time options​: HAS_TIMES MULTIPLICITY PERLIO_LAYERS
  PERL_DONT_CREATE_GVSV
  PERL_HASH_FUNC_ONE_AT_A_TIME_HARD
  PERL_IMPLICIT_CONTEXT PERL_MALLOC_WRAP
  PERL_NEW_COPY_ON_WRITE PERL_PRESERVE_IVUV
  PERL_USE_DEVEL USE_64_BIT_ALL USE_64_BIT_INT
  USE_ITHREADS USE_LARGE_FILES USE_LOCALE
  USE_LOCALE_COLLATE USE_LOCALE_CTYPE
  USE_LOCALE_NUMERIC USE_LONG_DOUBLE USE_PERLIO
  USE_PERL_ATOF USE_REENTRANT_API
  Built under linux
  Compiled at Jun 13 2014 19​:53​:16
  @​INC​:
  /home/sand/src/perl/repoperls/installed-perls/perl/v5.21.0-387-gb06e47f/a2da/lib/site_perl/5.21.1/x86_64-linux-thread-multi-ld
  /home/sand/src/perl/repoperls/installed-perls/perl/v5.21.0-387-gb06e47f/a2da/lib/site_perl/5.21.1
  /home/sand/src/perl/repoperls/installed-perls/perl/v5.21.0-387-gb06e47f/a2da/lib/5.21.1/x86_64-linux-thread-multi-ld
  /home/sand/src/perl/repoperls/installed-perls/perl/v5.21.0-387-gb06e47f/a2da/lib/5.21.1
  .

--
andreas

@p5pRT
Copy link
Author

p5pRT commented Jun 17, 2014

From @rjbs

* "Andreas J. Koenig via RT" <perlbug-followup@​perl.org> [2014-06-17T15​:54​:15]

also affected
-------------
MICHEL/Business-KontoCheck-5.4.tar.gz
MIYAGAWA/XML-Atom-0.41.tar.gz
GFUJI/HTML-FillInForm-Lite-1.13.tar.gz
DYACOB/Regexp-Ethiopic-0.15.tar.gz
COSIMO/Device-Gsm-1.60.tar.gz
YKAR/WWW-Google-PageRank-0.17.tar.gz
DDB/Convert-YText-v0.1.2.tar.gz
AUDREYT/Convert-EastAsianWidth-1.02.tar.gz

There will certainly be more.

I will be interested to compare this list to the one in
http​://www.nntp.perl.org/group/perl.perl5.porters/2014/06/msg216911.html

(We're already planning to revert this, but I'm hoping we'll catch a few
surprising cases while testing, first.)

--
rjbs

@p5pRT
Copy link
Author

p5pRT commented Jun 17, 2014

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

@p5pRT
Copy link
Author

p5pRT commented Jun 17, 2014

From @khwilliamson

On 06/17/2014 02​:17 PM, Ricardo Signes wrote​:

* "Andreas J. Koenig via RT" <perlbug-followup@​perl.org> [2014-06-17T15​:54​:15]

also affected
-------------
MICHEL/Business-KontoCheck-5.4.tar.gz
MIYAGAWA/XML-Atom-0.41.tar.gz
GFUJI/HTML-FillInForm-Lite-1.13.tar.gz
DYACOB/Regexp-Ethiopic-0.15.tar.gz
COSIMO/Device-Gsm-1.60.tar.gz
YKAR/WWW-Google-PageRank-0.17.tar.gz
DDB/Convert-YText-v0.1.2.tar.gz
AUDREYT/Convert-EastAsianWidth-1.02.tar.gz

There will certainly be more.

I will be interested to compare this list to the one in
http​://www.nntp.perl.org/group/perl.perl5.porters/2014/06/msg216911.html

(We're already planning to revert this, but I'm hoping we'll catch a few
surprising cases while testing, first.)

On irc we decided we'd do a CPAN smoke of a branch instead of alarming
module authors possibly unnecessarily. So I will revert the removal in
a few hours

@p5pRT
Copy link
Author

p5pRT commented Jun 18, 2014

From @andk

On Tue, 17 Jun 2014 15​:49​:27 -0700, "karl williamson via RT" <perlbug-followup@​perl.org> said​:

  > On 06/17/2014 02​:17 PM, Ricardo Signes wrote​:

* "Andreas J. Koenig via RT" <perlbug-followup@​perl.org> [2014-06-17T15​:54​:15]

also affected
-------------
MICHEL/Business-KontoCheck-5.4.tar.gz
MIYAGAWA/XML-Atom-0.41.tar.gz
GFUJI/HTML-FillInForm-Lite-1.13.tar.gz
DYACOB/Regexp-Ethiopic-0.15.tar.gz
COSIMO/Device-Gsm-1.60.tar.gz
YKAR/WWW-Google-PageRank-0.17.tar.gz
DDB/Convert-YText-v0.1.2.tar.gz
AUDREYT/Convert-EastAsianWidth-1.02.tar.gz

There will certainly be more.

I will be interested to compare this list to the one in
http​://www.nntp.perl.org/group/perl.perl5.porters/2014/06/msg216911.html

(We're already planning to revert this, but I'm hoping we'll catch a few
surprising cases while testing, first.)

  > On irc we decided we'd do a CPAN smoke of a branch instead of alarming
  > module authors possibly unnecessarily. So I will revert the removal in
  > a few hours

Can somebody start writing a policy for what is acceptabel to try out on
blead and what not? I will adjust the amount of work I'm willing to
invest into BBC smoking.

So far I was under the impression that the blead branch is for things we
have carefukly considered and really want for the next perl version.
Things like "what breaks when I turn this know here" are clearly not
expected. The downside of trying things out on blead is not only that
authors are alarmed but also that statistics get sour.

--
andreas

@p5pRT
Copy link
Author

p5pRT commented Jun 18, 2014

From @khwilliamson

On 06/17/2014 07​:21 PM, Andreas Koenig wrote​:

On Tue, 17 Jun 2014 15​:49​:27 -0700, "karl williamson via RT" <perlbug-followup@​perl.org> said​:

On 06/17/2014 02​:17 PM, Ricardo Signes wrote​:

* "Andreas J. Koenig via RT" <perlbug-followup@​perl.org> [2014-06-17T15​:54​:15]

also affected
-------------
MICHEL/Business-KontoCheck-5.4.tar.gz
MIYAGAWA/XML-Atom-0.41.tar.gz
GFUJI/HTML-FillInForm-Lite-1.13.tar.gz
DYACOB/Regexp-Ethiopic-0.15.tar.gz
COSIMO/Device-Gsm-1.60.tar.gz
YKAR/WWW-Google-PageRank-0.17.tar.gz
DDB/Convert-YText-v0.1.2.tar.gz
AUDREYT/Convert-EastAsianWidth-1.02.tar.gz

There will certainly be more.

I will be interested to compare this list to the one in
http​://www.nntp.perl.org/group/perl.perl5.porters/2014/06/msg216911.html

(We're already planning to revert this, but I'm hoping we'll catch a few
surprising cases while testing, first.)

On irc we decided we'd do a CPAN smoke of a branch instead of alarming
module authors possibly unnecessarily. So I will revert the removal in
a few hours

Can somebody start writing a policy for what is acceptabel to try out on
blead and what not? I will adjust the amount of work I'm willing to
invest into BBC smoking.

So far I was under the impression that the blead branch is for things we
have carefukly considered and really want for the next perl version.
Things like "what breaks when I turn this know here" are clearly not
expected. The downside of trying things out on blead is not only that
authors are alarmed but also that statistics get sour.

The encoding pragma is now reinstated.

I'm sorry to have put you to this trouble. I think we have been taking
your excellent work too much for granted, and should more carefully
consider things before increasing that work. It just seems that the
whole process is automated, when in fact it isn't. Also having CPAN
smokes is a fairly recent innovation and only a few people know how to
do them, so we forget that that option exists, so using blead this early
in the cycle seemed like a reasonable thing to do.

In this case, the decision was made some years ago that this pragma was
coming out in 5.22. That is what I thought, based on comments made at
various times. So it seemed like the right thing to do, to follow up on
that decision.

@p5pRT
Copy link
Author

p5pRT commented Jun 18, 2014

From @andk

On Tue, 17 Jun 2014 19​:42​:15 -0600, Karl Williamson <public@​khwilliamson.com> said​:

  > I'm sorry to have put you to this trouble. I think we have been
  > taking your excellent work too much for granted, and should more
  > carefully consider things before increasing that work. It just seems
  > that the whole process is automated, when in fact it isn't.

Working on it. It's getting better every year but it's still a challenge
to get a sufficient number of reports at the right time into shape.
Every bug that appears shadows others and diminishes the chance to find
those. That's why the success of the statistics programs depends on
discipline on the commits in trunk.

I'm sorry that my tone was a bit shrill, I had thin nerves this morning.
Anyway, Thanks for your understanding && Cheers,
--
andreas

@p5pRT
Copy link
Author

p5pRT commented Sep 14, 2014

From @jkeenan

On Tue Jun 17 18​:42​:50 2014, public@​khwilliamson.com wrote​:

On 06/17/2014 07​:21 PM, Andreas Koenig wrote​:

On Tue, 17 Jun 2014 15​:49​:27 -0700, "karl williamson via RT"
<perlbug-followup@​perl.org> said​:

On 06/17/2014 02​:17 PM, Ricardo Signes wrote​:

* "Andreas J. Koenig via RT" <perlbug-followup@​perl.org> [2014-
06-17T15​:54​:15]

also affected
-------------
MICHEL/Business-KontoCheck-5.4.tar.gz
MIYAGAWA/XML-Atom-0.41.tar.gz
GFUJI/HTML-FillInForm-Lite-1.13.tar.gz
DYACOB/Regexp-Ethiopic-0.15.tar.gz
COSIMO/Device-Gsm-1.60.tar.gz
YKAR/WWW-Google-PageRank-0.17.tar.gz
DDB/Convert-YText-v0.1.2.tar.gz
AUDREYT/Convert-EastAsianWidth-1.02.tar.gz

There will certainly be more.

I will be interested to compare this list to the one in
http​://www.nntp.perl.org/group/perl.perl5.porters/2014/06/msg216911.html

[snip]

(We're already planning to revert this, but I'm hoping >
The encoding pragma is now reinstated.

Karl, Andreas​: Does this mean that this ticket is now closable?

Thank you very much.

--
James E Keenan (jkeenan@​cpan.org)

@p5pRT
Copy link
Author

p5pRT commented Sep 15, 2014

From @andk

On Sun, 14 Sep 2014 05​:36​:56 -0700, "James E Keenan via RT" <perlbug-followup@​perl.org> said​:

  > Karl, Andreas​: Does this mean that this ticket is now closable?

As far as I understand it, yes.

--
andreas

@p5pRT
Copy link
Author

p5pRT commented Sep 16, 2014

From @jkeenan

On Sun Sep 14 20​:59​:25 2014, andreas.koenig.7os6VVqR@​franz.ak.mind.de wrote​:

On Sun, 14 Sep 2014 05​:36​:56 -0700, "James E Keenan via RT"
<perlbug-followup@​perl.org> said​:

Karl, Andreas​: Does this mean that this ticket is now closable?

As far as I understand it, yes.

Closing on recommendation from Andreas.

Thank you very much.
--
James E Keenan (jkeenan@​cpan.org)

@p5pRT p5pRT closed this as completed Sep 16, 2014
@p5pRT
Copy link
Author

p5pRT commented Sep 16, 2014

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