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 fb85c04 (?^...) breaks CPAN #10658

Closed
p5pRT opened this issue Sep 23, 2010 · 27 comments
Closed

Bleadperl fb85c04 (?^...) breaks CPAN #10658

p5pRT opened this issue Sep 23, 2010 · 27 comments

Comments

@p5pRT
Copy link

@p5pRT p5pRT commented Sep 23, 2010

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

Searchable as RT78008$

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Sep 23, 2010

From @andk

This ticket is considered to hold pointers to all CPAN distros that are
broken by the new C< (?^​: > regexp syntax. According to my smokers as of
yesterday this includes the following​:

ADAMK/Aspect-0.92.tar.gz
GAAS/Data-Dump-1.17.tar.gz # https://rt.cpan.org//Ticket/Display.html?id=61552
DLAND/Regexp-Assemble-0.34.tar.gz
CEBJYRE/Test-ConsistentVersion-v0.2.2.tar.gz
FDALY/Test-Deep-0.106.tar.gz
EXODIST/Test-Exception-LessClever-0.005.tar.gz
RGARCIA/Test-LongString-0.13.tar.gz
BDFOY/Test-Output-0.16.tar.gz
CHORNY/Test-Warn-0.22.tar.gz
ADAMK/YAML-0.72.tar.gz
TODDR/YAML-Syck-1.14.tar.gz
INGY/YAML-LibYAML-0.33.tar.gz

--
andreas
perlin' for berlin

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Sep 23, 2010

From @avar

On Thu, Sep 23, 2010 at 06​:39, Andreas J. Koenig via RT
<perlbug-followup@​perl.org> wrote​:

This ticket is considered to hold pointers to all CPAN distros that are
broken by the new C< (?^​: > regexp syntax. According to my smokers as of
yesterday this includes the following​:

ADAMK/Aspect-0.92.tar.gz
GAAS/Data-Dump-1.17.tar.gz  # https://rt.cpan.org//Ticket/Display.html?id=61552
DLAND/Regexp-Assemble-0.34.tar.gz
CEBJYRE/Test-ConsistentVersion-v0.2.2.tar.gz
FDALY/Test-Deep-0.106.tar.gz
EXODIST/Test-Exception-LessClever-0.005.tar.gz
RGARCIA/Test-LongString-0.13.tar.gz
BDFOY/Test-Output-0.16.tar.gz
CHORNY/Test-Warn-0.22.tar.gz
ADAMK/YAML-0.72.tar.gz
TODDR/YAML-Syck-1.14.tar.gz

I've fixed this in YAML​::Syck and uploaded it as 1.15 to CPAN.

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Sep 23, 2010

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

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Sep 24, 2010

From @khwilliamson

(Andreas J. Koenig) (via RT) wrote​:

# New Ticket Created by (Andreas J. Koenig)
# Please include the string​: [perl #78008]
# in the subject line of all future correspondence about this issue.
# <URL​: http​://rt.perl.org/rt3/Ticket/Display.html?id=78008 >

This ticket is considered to hold pointers to all CPAN distros that are
broken by the new C< (?^​: > regexp syntax. According to my smokers as of
yesterday this includes the following​:

ADAMK/Aspect-0.92.tar.gz
GAAS/Data-Dump-1.17.tar.gz # https://rt.cpan.org//Ticket/Display.html?id=61552
DLAND/Regexp-Assemble-0.34.tar.gz
CEBJYRE/Test-ConsistentVersion-v0.2.2.tar.gz
FDALY/Test-Deep-0.106.tar.gz
EXODIST/Test-Exception-LessClever-0.005.tar.gz
RGARCIA/Test-LongString-0.13.tar.gz
BDFOY/Test-Output-0.16.tar.gz
CHORNY/Test-Warn-0.22.tar.gz
ADAMK/YAML-0.72.tar.gz
TODDR/YAML-Syck-1.14.tar.gz
INGY/YAML-LibYAML-0.33.tar.gz

I spent some time looking at a problem that came up with one of these.
When you compile a stringified regex, an extra layer of parens surrounds
it, like (?^​:(?^​:foo)). This happens in 5.12 as well, but with -xism
instead of the caret. The problem today was that some code was trying
to parse regexes apparently avoid this double layer, and it was still
looking for the old -xism, so the double layer happened, and the tests
failed.

But Avar reminded me on IRC that there is an API to do that parsing for
you, and it has been updated to know about the new caret syntax​:

use re;
my ($pat, $mods) = regexp_pattern($ref);

$mods will contain all the non-default modifiers, and $pat the guts of
the regex.

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Sep 24, 2010

From @khwilliamson

karl williamson wrote​:

(Andreas J. Koenig) (via RT) wrote​:

# New Ticket Created by (Andreas J. Koenig) # Please include the
string​: [perl #78008]
# in the subject line of all future correspondence about this issue. #
<URL​: http​://rt.perl.org/rt3/Ticket/Display.html?id=78008 >

This ticket is considered to hold pointers to all CPAN distros that are
broken by the new C< (?^​: > regexp syntax. According to my smokers as of
yesterday this includes the following​:

ADAMK/Aspect-0.92.tar.gz
GAAS/Data-Dump-1.17.tar.gz #
https://rt.cpan.org//Ticket/Display.html?id=61552
DLAND/Regexp-Assemble-0.34.tar.gz
CEBJYRE/Test-ConsistentVersion-v0.2.2.tar.gz
FDALY/Test-Deep-0.106.tar.gz
EXODIST/Test-Exception-LessClever-0.005.tar.gz
RGARCIA/Test-LongString-0.13.tar.gz
BDFOY/Test-Output-0.16.tar.gz
CHORNY/Test-Warn-0.22.tar.gz
ADAMK/YAML-0.72.tar.gz
TODDR/YAML-Syck-1.14.tar.gz
INGY/YAML-LibYAML-0.33.tar.gz

I spent some time looking at a problem that came up with one of these.
When you compile a stringified regex, an extra layer of parens surrounds
it, like (?^​:(?^​:foo)). This happens in 5.12 as well, but with -xism
instead of the caret. The problem today was that some code was trying
to parse regexes apparently avoid this double layer, and it was still
looking for the old -xism, so the double layer happened, and the tests
failed.

But Avar reminded me on IRC that there is an API to do that parsing for
you, and it has been updated to know about the new caret syntax​:

use re;
my ($pat, $mods) = regexp_pattern($ref);

$mods will contain all the non-default modifiers, and $pat the guts of
the regex.

I forgot to say that you need to say re​::regexp_pattern($ref);
to get the namespace.

I've also patched perldelta, which is attached, as perlbug has stopped
working for me.

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Sep 24, 2010

From @khwilliamson

0001-perldelta-Give-better-instructions-about.patch
From 8df611470f97a7fd0f2d7e5cb10ea45f0612e703 Mon Sep 17 00:00:00 2001
From: Karl Williamson <public@khwilliamson.com>
Date: Thu, 23 Sep 2010 19:50:34 -0600
Subject: [PATCH] perldelta: Give better instructions about (?^...)

Avar pointed out that there is an existing API wwhich helps in coping
with this
---
 pod/perldelta.pod |   17 ++++++++++++++---
 1 files changed, 14 insertions(+), 3 deletions(-)

diff --git a/pod/perldelta.pod b/pod/perldelta.pod
index 5bb3706..7b76c9d 100644
--- a/pod/perldelta.pod
+++ b/pod/perldelta.pod
@@ -68,11 +68,22 @@ L</Selected Bug Fixes> section.
 Default regular expression modifiers are now notated by using
 C<(?^...)>.  Code relying on the old stringification will fail.  The
 purpose of this is so that when new modifiers are added, such code will
-not have to change, as the stringification will automatically
-incorporate the new modifiers.
+not have to change (after this one time), as the stringification will
+automatically incorporate the new modifiers.
 
 Code that needs to work properly with both old- and new-style regexes
-can use something like the following:
+can avoid the whole issue by using:
+
+ use re qw(regexp_pattern);
+ my ($pat, $mods) = regexp_pattern($re_ref);
+
+where C<$re_ref> is a reference to a compiled regular expression.  Upon
+return, C<$mods> will be a string containing all the non-default
+modifiers used when the regular expression was compiled, and C<$pattern>
+the actual pattern.
+
+If the actual stringification is important, you can use something like
+the following:
 
     # Accept both old and new-style stringification
     my $modifiers = (qr/foobar/ =~ /\Q(?^/) ? '^' : '-xism';
-- 
1.5.6.3

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Sep 24, 2010

From @khwilliamson

karl williamson wrote​:

karl williamson wrote​:

(Andreas J. Koenig) (via RT) wrote​:

# New Ticket Created by (Andreas J. Koenig) # Please include the
string​: [perl #78008]
# in the subject line of all future correspondence about this issue.
# <URL​: http​://rt.perl.org/rt3/Ticket/Display.html?id=78008 >

This ticket is considered to hold pointers to all CPAN distros that are
broken by the new C< (?^​: > regexp syntax. According to my smokers as of
yesterday this includes the following​:

ADAMK/Aspect-0.92.tar.gz
GAAS/Data-Dump-1.17.tar.gz #
https://rt.cpan.org//Ticket/Display.html?id=61552
DLAND/Regexp-Assemble-0.34.tar.gz
CEBJYRE/Test-ConsistentVersion-v0.2.2.tar.gz
FDALY/Test-Deep-0.106.tar.gz
EXODIST/Test-Exception-LessClever-0.005.tar.gz
RGARCIA/Test-LongString-0.13.tar.gz
BDFOY/Test-Output-0.16.tar.gz
CHORNY/Test-Warn-0.22.tar.gz
ADAMK/YAML-0.72.tar.gz
TODDR/YAML-Syck-1.14.tar.gz
INGY/YAML-LibYAML-0.33.tar.gz

I spent some time looking at a problem that came up with one of these.
When you compile a stringified regex, an extra layer of parens
surrounds it, like (?^​:(?^​:foo)). This happens in 5.12 as well, but
with -xism instead of the caret. The problem today was that some code
was trying to parse regexes apparently avoid this double layer, and it
was still looking for the old -xism, so the double layer happened, and
the tests failed.

But Avar reminded me on IRC that there is an API to do that parsing
for you, and it has been updated to know about the new caret syntax​:

use re;
my ($pat, $mods) = regexp_pattern($ref);

$mods will contain all the non-default modifiers, and $pat the guts of
the regex.

I forgot to say that you need to say re​::regexp_pattern($ref);
to get the namespace.

I've also patched perldelta, which is attached, as perlbug has stopped
working for me.

And another thing about this, I ended up having to grep for -xism​:,
-ism​:, -sm​:, -m​: in order to find all places in blead that needed work.
  Hopefully I got them all :)

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Sep 24, 2010

From @rgarcia

On 23 September 2010 08​:39, Andreas J. Koenig via RT
<perlbug-followup@​perl.org> wrote​:

This ticket is considered to hold pointers to all CPAN distros that are
broken by the new C< (?^​: > regexp syntax. According to my smokers as of
yesterday this includes the following​:
...
RGARCIA/Test-LongString-0.13.tar.gz

I pushed a fix on github ; will release later.

http​://github.com/rgs/Test-LongString/commit/344e3692e1f21e45c67f15c09f893c77a86ce367

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Sep 29, 2010

From @andk

On Wed, 22 Sep 2010 23​:39​:59 -0700, (Andreas J. Koenig) (via RT) <perlbug-followup@​perl.org> said​:

  > This ticket is considered to hold pointers to all CPAN distros that are
  > broken by the new C< (?^​: >

I just want to give you an updated list of affected modules, the list is
probably still not complete but enough things have moved to warrant an
update. Test​::LongString, YAML​::Syck have been fixed and new versions
uploaded (Thank you!), new modules have been discovered and added.

ADAMK/Aspect-0.92.tar.gz
GAAS/Data-Dump-1.17.tar.gz # fixed in repo according to Gisle Aas
DEXTER/Exception-Base-0.24.tar.gz
DROLSKY/Moose-1.14.tar.gz # fixed in repo according to Dave Rolsky
DLAND/Regexp-Assemble-0.34.tar.gz
CEBJYRE/Test-ConsistentVersion-v0.2.2.tar.gz
FDALY/Test-Deep-0.106.tar.gz # patch in #61741
EXODIST/Test-Exception-LessClever-0.005.tar.gz # patch in #61686
BDFOY/Test-Output-0.16.tar.gz
CHORNY/Test-Warn-0.22.tar.gz # patch in #61649
ADAMK/YAML-0.72.tar.gz
INGY/YAML-LibYAML-0.34.tar.gz

--
andreas

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Oct 19, 2010

From @andk

On Wed, 29 Sep 2010 06​:41​:31 +0200, andreas.koenig.7os6VVqR@​franz.ak.mind.de (Andreas J. Koenig) said​:

This ticket is considered to hold pointers to all CPAN distros that are
broken by the new C< (?^​: >

Since 5.13.6 is close I want to update the list again. Note that it is
still far from being complete, I only add things that jump into my eye,
there was no systematic investigation yet.

Data​::Dump, Exception​::Base, Moose, Test​::Deep have been fixed and new
versions uploaded. Thank you!

ADAMK/ Aspect-0.92.tar.gz
ADAMK/ YAML-0.72.tar.gz
BDFOY/ Test-Output-0.16.tar.gz
BOOK/ Regexp-Log-0.05.tar.gz
CEBJYRE/ Test-ConsistentVersion-v0.2.2.tar.gz
CHORNY/ Test-Warn-0.22.tar.gz patch in #61649
CLKAO/ Test-Log-Log4perl-0.29.tar.gz
DLAND/ Regexp-Assemble-0.34.tar.gz
ESM/ CGI-Alert-2.03.tar.gz
EXODIST/ Fennec-0.032.tar.gz
EXODIST/ Test-Exception-LessClever-0.005.tar.gz patch in #61686
INGY/ Test-Base-0.59.tar.gz
INGY/ YAML-LibYAML-0.34.tar.gz
MAKAMAKA/ Text-CSV-1.19.tar.gz
NODINE/ Slay-Makefile-Gress-0.07.tar.gz

--
andreas

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Oct 20, 2010

From @briandfoy

I think I've fixed Test​::Output for blead. CPAN Testers doesn't seem to have any reports for
blead though. :)

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Oct 21, 2010

From @andk

On Wed, 20 Oct 2010 16​:48​:44 -0700, "brian d foy via RT" <perlbug-followup@​perl.org> said​:

  > I think I've fixed Test​::Output for blead. CPAN Testers doesn't seem
  > to have any reports for blead though. :)

I have sent 4 testers reports with recent bleadperls and 7 with not so
recent bleadperls.

test PASS Test-Output-0.16_02 (perl-5.13.5) x86_64-linux-ld 2.6.32-2-amd64
test PASS Test-Output-0.16_02 (perl-5.13.5) x86_64-linux-thread-multi 2.6.32-2-amd64
test PASS Test-Output-0.16_02 (perl-5.13.5) x86_64-linux-thread-multi-ld 2.6.32-2-amd64
test PASS Test-Output-0.16_02 (perl-5.13.5) x86_64-linux 2.6.32-2-amd64
test PASS Test-Output-0.16_02 (perl-5.13.4) x86_64-linux 2.6.32-2-amd64
test PASS Test-Output-0.16_02 (perl-5.13.4) x86_64-linux-thread-multi-ld 2.6.32-2-amd64
test PASS Test-Output-0.16_02 (perl-5.13.4) x86_64-linux-ld 2.6.32-2-amd64
test PASS Test-Output-0.16_02 (perl-5.13.4) x86_64-linux-thread-multi 2.6.32-2-amd64
test PASS Test-Output-0.16_02 (perl-5.13.3) x86_64-linux-thread-multi-ld 2.6.32-2-amd64
test PASS Test-Output-0.16_02 (perl-5.13.2) x86_64-linux 2.6.32-2-amd64
test PASS Test-Output-0.16_02 (perl-5.13.1) x86_64-linux-ld 2.6.32-2-amd64
test PASS Test-Output-0.16_02 (perl-5.12.2) x86_64-linux-thread-multi 2.6.32-2-amd64
test PASS Test-Output-0.16_02 (perl-5.12.1) x86_64-linux 2.6.32-2-amd64
test PASS Test-Output-0.16_02 (perl-5.10.1) x86_64-linux-ld 2.6.30-1-amd64
test PASS Test-Output-0.16_02 (perl-5.8.9) x86_64-linux 2.6.26-1-amd64
test PASS Test-Output-0.16_02 (perl-5.8.7) x86_64-linux-thread-multi-ld 2.6.26-1-amd64
test PASS Test-Output-0.16_02 (perl-5.8.3) x86_64-linux-ld 2.6.32-2-amd64
test PASS Test-Output-0.16_02 (perl-5.6.2) x86_64-linux 2.6.32-2-amd64

There seems something broken on cpantesters throughput, my score in
http​://stats.cpantesters.org/testers.html hasn't changed for several days.

--
andreas

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Dec 2, 2010

From @andk

On Tue, 19 Oct 2010 06​:53​:52 +0200, andreas.koenig.7os6VVqR@​franz.ak.mind.de (Andreas J. Koenig) said​:

This ticket is considered to hold pointers to all CPAN distros that are
broken by the new C< (?^​: >

This is where we're standing today. Ten unpatched, 6 patched, two fixed
in dev releases.

ADAMK/ Aspect-0.92.tar.gz #62257
ADAMK/ YAML-0.72.tar.gz patch in #62258
ASKSH/ Getopt-LL-1.0.0.tar.gz #62713
BDFOY/ Test-Output-0.16.tar.gz fixed in Test-Output-0.16_03.tar.gz
BOOK/ Regexp-Log-0.05.tar.gz #62260
CHORNY/ Test-Warn-0.22.tar.gz patch in #61649
CLKAO/ Test-Log-Log4perl-0.29.tar.gz #62256
DLAND/ Regexp-Assemble-0.34.tar.gz patch in #62262
ESM/ CGI-Alert-2.03.tar.gz #62263
EXODIST/ Fennec-0.032.tar.gz #62264
EXODIST/ Test-Exception-LessClever-0.005.tar.gz patch in #61686
INGY/ Test-Base-0.59.tar.gz patch in #62265
INGY/ YAML-LibYAML-0.34.tar.gz patch in #62266
MCAST/ Test-Unit-0.25.tar.gz #62771
MIROD/ XML-Twig-3.37.tar.gz #62781
NEVESENIN/Regexp-RegGrp-0.0203.tar.gz fixed in Regexp-RegGrp-0.03_01.tar.gz
NODINE/ Slay-Makefile-Gress-0.07.tar.gz #61858
RHANDOM/ Template-Alloy-1.013.tar.gz #62770

--
andreas

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Apr 2, 2011

From @obra

On Wed Dec 01 20​:18​:19 2010, andreas.koenig.7os6VVqR@​franz.ak.mind.de wrote​:

On Tue, 19 Oct 2010 06​:53​:52 +0200,
andreas.koenig.7os6VVqR@​franz.ak.mind.de (Andreas J. Koenig) said​:

This ticket is considered to hold pointers to all CPAN distros
that are
broken by the new C< (?^​: >

The following dists are still broken on CPAN

ADAMK/ YAML-0.72.tar.gz patch in #62258
ASKSH/ Getopt-LL-1.0.0.tar.gz #62713
BOOK/ Regexp-Log-0.05.tar.gz #62260
DLAND/ Regexp-Assemble-0.34.tar.gz patch in #62262
ESM/ CGI-Alert-2.03.tar.gz #62263
EXODIST/ Test-Exception-LessClever-0.005.tar.gz patch in #61686
INGY/ Test-Base-0.59.tar.gz patch in #62265
INGY/ YAML-LibYAML-0.34.tar.gz patch in #62266
MCAST/ Test-Unit-0.25.tar.gz #62771

Of these, I'd love it if someone could run reverse-deps and tell me how much of
CPAN depend on them. YAML​::LibYAML and Test​::Base are both probably
showstopper-grade broken.

The following, I believe, are no longer an issue​:

EXODIST/ Fennec-0.032.tar.gz #62264

Major new release, no obvious changelog

ADAMK/ Aspect-0.92.tar.gz #62257

Fix released

CHORNY/ Test-Warn-0.22.tar.gz patch in #61649

Fixed in 0.23

BDFOY/ Test-Output-0.16.tar.gz fixed in Test-
Output-0.16_03.tar.gz

CLKAO/ Test-Log-Log4perl-0.29.tar.gz #62256

Fixed in 0.30

MIROD/ XML-Twig-3.37.tar.gz #62781

Fixed in 3.38

NEVESENIN/Regexp-RegGrp-0.0203.tar.gz fixed in Regexp-
RegGrp-0.03_01.tar.gz
NODINE/ Slay-Makefile-Gress-0.07.tar.gz #61858

Fixed in 0.08

RHANDOM/ Template-Alloy-1.013.tar.gz #62770

Fixed

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Apr 3, 2011

From @cpansprout

On Fri Apr 01 23​:07​:13 2011, jesse wrote​:

On Wed Dec 01 20​:18​:19 2010, andreas.koenig.7os6VVqR@​franz.ak.mind.de
wrote​:

On Tue, 19 Oct 2010 06​:53​:52 +0200,
andreas.koenig.7os6VVqR@​franz.ak.mind.de (Andreas J. Koenig) said​:

This ticket is considered to hold pointers to all CPAN distros
that are
broken by the new C< (?^​: >

The following dists are still broken on CPAN

ADAMK/ YAML-0.72.tar.gz patch in #62258
ASKSH/ Getopt-LL-1.0.0.tar.gz #62713
BOOK/ Regexp-Log-0.05.tar.gz #62260
DLAND/ Regexp-Assemble-0.34.tar.gz patch in #62262
ESM/ CGI-Alert-2.03.tar.gz #62263
EXODIST/ Test-Exception-LessClever-0.005.tar.gz patch in #61686
INGY/ Test-Base-0.59.tar.gz patch in #62265
INGY/ YAML-LibYAML-0.34.tar.gz patch in #62266
MCAST/ Test-Unit-0.25.tar.gz #62771

Of these, I'd love it if someone could run reverse-deps and tell me
how much of
CPAN depend on them. YAML​::LibYAML and Test​::Base are both probably
showstopper-grade broken.

YAML​: deps.cpantesters.org has a bug in it (reported) that is causing
hundreds of modules to be listed (not all of which use YAML.pm), so I
cannot tell.

Getopt-LL​: Class-Dot-Model and File-BSED

Regexp-Log​:
Apache-Logmonster
App-Hachero
Regexp-Log-BlueCoat
Regexp-Log-Common
Regexp-Log-Helix
Regexp-Log-RealServer
Regexp-Log-WMS

Regexp-Assemble​: 30 dependents

CGI-Alert​: none

Test-Exception-LessClever​: 25

Test-Base​: 152

YAML-LibYAML​: About 40 immediate dependents, but the same
deps.cpantesters.org bug is bloating the recursive results

Test-Unit​: none

There are more modules to be added to the list of those broken by (?^).
These are from the list at <https://gist.github.com/884673> (just the
ones that were obvious; there may be more)​:

Apache-LogRegex-1.5​: none

Data-FormValidator-Constraints-Words-0.04​:
Labyrinth
CPAN-Testers-WWW-Wiki
Labyrinth-Plugin-Core

Parse-RandGen-0.202​: none

Regexp-English-1.00​: none

Regexp-HTMLify-0.001​: none

Regexp-Parser-0.20​: none

Switch-Perlish-1.0.5​: none

Template-Parser-CET-0.05​:
Task-BeLike-MELO
Task-Template-Benchmark

Test-DatabaseRow-1.04​:
Acme-PETEK-Testkit
Bot-BasicBot-Pluggable-Module-Notes
CGI-FormBuilder-Config-Simple

Regex-Iterator-0.3​: none

Text-Quote-0.3​: AI-Prolog

YAML-Old-0.81​: none

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Apr 4, 2011

From @obra

On Sun 3.Apr'11 at 14​:29​:40 -0700, Father Chrysostomos via RT wrote​:

Thanks for the triage.

YAML​: deps.cpantesters.org has a bug in it (reported) that is causing
hundreds of modules to be listed (not all of which use YAML.pm), so I
cannot tell.

I've confirmed with ingy that he's working to fix
YAML​::LibYAML and Test​::Base today.

To make an arbitrary call​: If a module doesn't have at least 5
authored-by-someone-else downstream modules using it or some
other exceptional reason for being a blocker, I won't consider
a failure due to this change a blocker.

To that end, I'd consider the following blcoking​:

Regexp-Assemble​: 30 dependents
Test-Exception-LessClever​: 25
Test-Base​: 152
YAML-LibYAML​: About 40 immediate dependents, but the same deps.cpantesters.org bug is bloating the recursive results
YAML​::XS

I'd not consider the following blocking

Getopt-LL​: Class-Dot-Model and File-BSED
Regexp-Log​:
Apache-Logmonster
App-Hachero
Regexp-Log-BlueCoat
Regexp-Log-Common
Regexp-Log-Helix
Regexp-Log-RealServer
Regexp-Log-WMS
CGI-Alert​: none
Test-Unit​: none
Apache-LogRegex-1.5​: none
Data-FormValidator-Constraints-Words-0.04​:
Labyrinth
CPAN-Testers-WWW-Wiki
Labyrinth-Plugin-Core

Parse-RandGen-0.202​: none

Regexp-English-1.00​: none

Regexp-HTMLify-0.001​: none

Regexp-Parser-0.20​: none

Switch-Perlish-1.0.5​: none

Template-Parser-CET-0.05​:
Task-BeLike-MELO
Task-Template-Benchmark

Test-DatabaseRow-1.04​:
Acme-PETEK-Testkit
Bot-BasicBot-Pluggable-Module-Notes
CGI-FormBuilder-Config-Simple

Regex-Iterator-0.3​: none

Text-Quote-0.3​: AI-Prolog

YAML-Old-0.81​: none

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Apr 5, 2011

From chromatic@wgz.org

On Sunday 03 April 2011 at 14​:29, Father Chrysostomos wrote​:

Regexp-English-1.00​: none

Fixed in 1.01, not that it should block anything.

-- c

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Apr 6, 2011

From @obra

On Sun Apr 03 21​:11​:41 2011, jesse@​bestpractical.com wrote​:

To make an arbitrary call​: If a module doesn't have at least 5
authored-by-someone-else downstream modules using it or some
other exceptional reason for being a blocker, I won't consider
a failure due to this change a blocker.

To that end, I'd consider the following blcoking​:

Regexp-Assemble​: 30 dependents
Test-Exception-LessClever​: 25
Test-Base​: 152
YAML-LibYAML​: About 40 immediate dependents, but the same
deps.cpantesters.org bug is bloating the recursive results
YAML​::XS

Ingy has uploaded new versions of his modules. David and Chad have
confirmed to me that they're actively working to release new versions of
their modules that work correctly on 5.14.

With that, I'm thrilled to be able to resolve this ticket as a 5.14
blocker.

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Apr 6, 2011

@obra - Status changed from 'open' to 'resolved'

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Apr 7, 2011

From david@landgren.net

On 06/04/2011 16​:15, Jesse via RT wrote​:

On Sun Apr 03 21​:11​:41 2011, jesse@​bestpractical.com wrote​:

To make an arbitrary call​: If a module doesn't have at least 5
authored-by-someone-else downstream modules using it or some
other exceptional reason for being a blocker, I won't consider
a failure due to this change a blocker.

To that end, I'd consider the following blcoking​:

Regexp-Assemble​: 30 dependents

Fixed by 0.35 (slightly amazed to discover it has 30 dependants)

Test-Exception-LessClever​: 25
Test-Base​: 152
YAML-LibYAML​: About 40 immediate dependents, but the same
deps.cpantesters.org bug is bloating the recursive results
YAML​::XS

Ingy has uploaded new versions of his modules. David and Chad have
confirmed to me that they're actively working to release new versions of
their modules that work correctly on 5.14.

With that, I'm thrilled to be able to resolve this ticket as a 5.14
blocker.

David

--
There's bum trash in my hall and my place is ripped
I've totalled another amp, I'm calling in sick

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Apr 17, 2011

From @cpansprout

On Wed Apr 06 07​:15​:33 2011, jesse wrote​:

On Sun Apr 03 21​:11​:41 2011, jesse@​bestpractical.com wrote​:

To make an arbitrary call​: If a module doesn't have at least 5
authored-by-someone-else downstream modules using it or some
other exceptional reason for being a blocker, I won't consider
a failure due to this change a blocker.

To that end, I'd consider the following blcoking​:

Regexp-Assemble​: 30 dependents
Test-Exception-LessClever​: 25
Test-Base​: 152
YAML-LibYAML​: About 40 immediate dependents, but the same
deps.cpantesters.org bug is bloating the recursive results
YAML​::XS

Ingy has uploaded new versions of his modules. David and Chad have
confirmed to me that they're actively working to release new versions of
their modules that work correctly on 5.14.

With that, I'm thrilled to be able to resolve this ticket as a 5.14
blocker.

Did you perhaps confuse YAML.pm with YAML​::XS? YAML.pm has not been
fixed yet. The deps.cpantesters.org bug has been fixed, so I have been
able to find out that there are 1203 (!) distributions that depend on
it, either directly or indirectly.

I’m re-opening this ticket. (If you disagree, please just close it.)

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Apr 17, 2011

@cpansprout - Status changed from 'resolved' to 'open'

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Apr 17, 2011

From @cpansprout

On Wed Apr 06 07​:15​:33 2011, jesse wrote​:

On Sun Apr 03 21​:11​:41 2011, jesse@​bestpractical.com wrote​:

To make an arbitrary call​: If a module doesn't have at least 5
authored-by-someone-else downstream modules using it or some
other exceptional reason for being a blocker, I won't consider
a failure due to this change a blocker.

To that end, I'd consider the following blcoking​:

Regexp-Assemble​: 30 dependents
Test-Exception-LessClever​: 25
Test-Base​: 152
YAML-LibYAML​: About 40 immediate dependents, but the same
deps.cpantesters.org bug is bloating the recursive results
YAML​::XS

Ingy has uploaded new versions of his modules. David and Chad have
confirmed to me that they're actively working to release new versions of
their modules that work correctly on 5.14.

With that, I'm thrilled to be able to resolve this ticket as a 5.14
blocker.

Did you perhaps confuse YAML.pm with YAML​::XS? YAML.pm has not been
fixed yet. The deps.cpantesters.org bug has been fixed, so I have been
able to find out that there are 1203 (!) distributions that depend on
it, either directly or indirectly.

I’m re-opening this ticket. (If you disagree, please just close it.)

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Apr 19, 2011

From @obra

Adam,

You'd said you were going to get a new release of YAML that works on
5.14 out at the QA hackathon. It looks like that didn't happen.

Is there anything that we can do to help that happen in the next
24 hours, so that YAML doesn't explode on 5.14.0RC1?

-Jesse

On Sun 17.Apr'11 at 14​:03​:04 -0700, Father Chrysostomos via RT wrote​:

On Wed Apr 06 07​:15​:33 2011, jesse wrote​:

On Sun Apr 03 21​:11​:41 2011, jesse@​bestpractical.com wrote​:

To make an arbitrary call​: If a module doesn't have at least 5
authored-by-someone-else downstream modules using it or some
other exceptional reason for being a blocker, I won't consider
a failure due to this change a blocker.

To that end, I'd consider the following blcoking​:

Regexp-Assemble​: 30 dependents
Test-Exception-LessClever​: 25
Test-Base​: 152
YAML-LibYAML​: About 40 immediate dependents, but the same
deps.cpantesters.org bug is bloating the recursive results
YAML​::XS

Ingy has uploaded new versions of his modules. David and Chad have
confirmed to me that they're actively working to release new versions of
their modules that work correctly on 5.14.

With that, I'm thrilled to be able to resolve this ticket as a 5.14
blocker.

Did you perhaps confuse YAML.pm with YAML​::XS? YAML.pm has not been
fixed yet. The deps.cpantesters.org bug has been fixed, so I have been
able to find out that there are 1203 (!) distributions that depend on
it, either directly or indirectly.

I’m re-opening this ticket. (If you disagree, please just close it.)

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Apr 19, 2011

From @obra

Ingy has just uploaded YAML.pm 0.73, which I've verified passes on blead.

I'm resolving this ticket until we find another catastrophic blead-
killer.

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Apr 19, 2011

From [Unknown Contact. See original ticket]

Ingy has just uploaded YAML.pm 0.73, which I've verified passes on blead.

I'm resolving this ticket until we find another catastrophic blead-
killer.

@p5pRT
Copy link
Author

@p5pRT p5pRT commented Apr 19, 2011

@obra - Status changed from 'open' to 'resolved'

@p5pRT p5pRT closed this Apr 19, 2011
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