Skip Menu |

This queue is for tickets about the PerlX-Assert CPAN distribution.

Report information
The Basics
Id: 114938
Status: resolved
Priority: 0/
Queue: PerlX-Assert
People
Owner: Nobody in particular
Requestors: ANDK [...] cpan.org
gregoa [...] cpan.org
jkeenan [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: 0.904
Fixed in: (no value)

History Show all quoted text
  Tue May 31 23:51:04 2016 ANDK [...] cpan.org - Ticket created
Subject: Fails with bleadperl if Devel::Declare is installed
As per subject. Sample fail report: http://www.cpantesters.org/cpan/report/d9b730e8-1643-11e6-9ff0-2cca9ae6fa17 This test t/03dd.t is skipped when Devel::Declare is not installed, so there are PASSes with later bleadperls because Devel::Declare is only a "suggests". Complementary bleadperl ticket: https://rt.perl.org/Public/Bug/Display.html?id=128139 HTH&&Thanks,
  Sun Jun 05 17:09:44 2016 gregoa [...] cpan.org - Ticket #115112: Ticket created
From: gregoa [...] cpan.org
Subject: libperlx-assert-perl: Unescaped left brace in regex is deprecated
We have the following bug reported to the Debian package of PerlX-Assert (https://bugs.debian.org/826489): It doesn't seem to be a bug in the packaging, so you may want to take a look. Thanks! ------8<-----------8<-----------8<-----------8<-----------8<----- Package: libperlx-assert-perl Version: 0.904-1 Severity: normal User: debian-perl@lists.debian.org Usertags: perl-5.24-transition Building this package triggers deprecation warnings with Perl 5.24 (currently in experimental), and probably with Perl 5.22 (current sid) too. Unescaped left brace in regex is deprecated, passed through in regex; marked by <-- HERE in m/\A{ <-- HERE / at /<<PKGBUILDDIR>>/blib/lib/PerlX/Assert/DD.pm line 102. The offending file is installed in a binary package, so this probably has runtime effects as well. A full build log is available at http://perl.debian.net/rebuild-logs/perl-5.24-throwaway/libperlx-assert-perl_0.904-1/ -- Niko Tyni ntyni@debian.org ------8<-----------8<-----------8<-----------8<-----------8<----- Thanks for considering, gregor herrmann, Debian Perl Group
  Mon Apr 03 12:30:59 2017 jkeenan [...] cpan.org - Ticket #120908: Ticket created
Subject: PerlX-Assert: Unescaped left brace in regex is illegal here in regex
Today I was assessing the perl-5.26.0-readiness of this library. Using perl 5 blead I built the library and called 'make test'. Here is an excerpt from my results: ##### $ make test cp lib/PerlX/Assert/DD.pm blib/lib/PerlX/Assert/DD.pm Skip blib/lib/PerlX/Assert/Keyword.pm (unchanged) Skip blib/lib/PerlX/Assert.pm (unchanged) PERL_DL_NONLAZY=1 "/home/jkeenan/testing/blead/bin/perl" "-MExtUtils::Command::MM" "-MTest::Harness" "-e" "undef *Test::Harness::Switches; test_harness(0, 'blib/lib', 'blib/arch')" t/*.t t/01basic.t ..... ok t/02kwapi.t ..... skipped: broken by PerlX::Assert's change to decision logic t/03dd.t ........ Unescaped left brace in regex is illegal here in regex; marked by <-- HERE in m/\A{ <-- HERE / at /home/jkeenan/Downloads/PerlX-Assert-0.904/blib/lib/PerlX/Assert/DD.pm line 102. Compilation failed in require at /home/jkeenan/Downloads/PerlX-Assert-0.904/blib/lib/PerlX/Assert.pm line 50. BEGIN failed--compilation aborted at 02kwapi.t line 2. Compilation failed in require at t/03dd.t line 27. t/03dd.t ........ Dubious, test returned 255 (wstat 65280, 0xff00) No subtests run t/04syntax.t .... ok t/05renaming.t .. ok t/99moops.t ..... skipped: Test requires Moops 0.030 Test Summary Report ------------------- t/03dd.t (Wstat: 65280 Tests: 0 Failed: 0) Non-zero exit status: 255 Parse errors: No plan found in TAP output Files=6, Tests=18, 0 wallclock secs ( 0.02 usr 0.00 sys + 0.35 cusr 0.02 csys = 0.39 CPU) Result: FAIL Failed 1/6 test programs. 0/18 subtests failed. Makefile:854: recipe for target 'test_dynamic' failed make: *** [test_dynamic] Error 255 ##### I am attaching a patch which gets 'make test' to PASS. If you could evaluate and apply it and generate a new CPAN release, then this library should be installable under perl-5.26.0. Thank you very much. Jim Keenan
Subject: PerlX-Assert.20170403.diff
--- lib/PerlX/Assert/DD.pm.orig 2017-04-03 12:24:28.712564699 -0400 +++ lib/PerlX/Assert/DD.pm 2017-04-03 12:24:47.543294317 -0400 @@ -99,7 +99,7 @@ $self->inc_offset(1); $self->skipspace; - if ($self->get_remainder =~ /\A{/) + if ($self->get_remainder =~ /\A\{/) { require Carp; Carp::croak("Unexpected comma between assertion name and block");
  Mon Apr 03 12:45:13 2017 jkeenan [...] cpan.org - Ticket #120908: Correspondence added
On Mon Apr 03 12:30:59 2017, JKEENAN wrote: Show quoted text
> > I am attaching a patch which gets 'make test' to PASS. If you could > evaluate and apply it and generate a new CPAN release, then this > library should be installable under perl-5.26.0. >
Many other CPAN distributions have a dependency on PerlX-Assert (see XML attached), so a new CPAN release will enhance the 5.26.0-readiness of those distributions as well. Thank you very much. Jim Keenan
Subject: PerlX-Assert.revdeps.20170403.xml
<?xml version="1.0"?> <reversedependencies> <dist> <name>PerlX-Assert</name> <version>PerlX-Assert</version> <depended_on_by count="1"> <dist> <name>Moops</name> <depended_on_by count="26"> <dist> <name>Acme-Resume</name></dist> <dist> <name>Amazon-SES</name></dist> <dist> <name>Bencher-Scenarios-Accessors</name></dist> <dist> <name>CairoX-Sweet</name></dist> <dist> <name>DBIx-Deployer</name></dist> <dist> <name>Data-PaginatedTable</name></dist> <dist> <name>Dist-Iller</name> <depended_on_by count="2"> <dist> <name>Dist-Iller-Config-Author-CSSON</name></dist> <dist> <name>Dist-Zilla-MintingProfile-MapMetro-Map</name> <depended_on_by count="1"> <dist> <name>Task-MapMetro-Dev</name></dist> </depended_on_by></dist> </depended_on_by></dist> <dist> <name>Kavorka-TraitFor-Parameter-doc</name> <depended_on_by count="2"> <dist> <name>Map-Metro</name> <depended_on_by count="20"> <dist> <name>App-Map-Metro</name></dist> <dist> <name>Dist-Zilla-Plugin-MapMetro-MakeGraphViz</name></dist> <dist> <name>Dist-Zilla-Plugin-MapMetro-MakeLinePod</name></dist> <dist> <name>Map-Metro-Plugin-Map-Amsterdam</name> <depended_on_by count="1"> <dist> <name>Task-MapMetro-Maps</name></dist> </depended_on_by></dist> <dist> <name>Map-Metro-Plugin-Map-Athens</name></dist> <dist> <name>Map-Metro-Plugin-Map-Barcelona</name></dist> <dist> <name>Map-Metro-Plugin-Map-Berlin</name></dist> <dist> <name>Map-Metro-Plugin-Map-Bilbao</name></dist> <dist> <name>Map-Metro-Plugin-Map-Brescia</name></dist> <dist> <name>Map-Metro-Plugin-Map-Brussels</name></dist> <dist> <name>Map-Metro-Plugin-Map-Budapest</name></dist> <dist> <name>Map-Metro-Plugin-Map-Copenhagen</name></dist> <dist> <name>Map-Metro-Plugin-Map-Gothenburg</name></dist> <dist> <name>Map-Metro-Plugin-Map-Helsinki</name></dist> <dist> <name>Map-Metro-Plugin-Map-Lille</name></dist> <dist> <name>Map-Metro-Plugin-Map-Lyon</name></dist> <dist> <name>Map-Metro-Plugin-Map-Madrid</name></dist> <dist> <name>Map-Metro-Plugin-Map-Oslo</name></dist> <dist> <name>Map-Metro-Plugin-Map-Stockholm</name></dist> <dist> <name>Task-MapMetro-Maps</name></dist> </depended_on_by></dist> <dist> <name>Stenciller</name> <depended_on_by count="4"> <dist> <name>Dist-Zilla-Plugin-Stenciller-HtmlExamples</name></dist> <dist> <name>Dist-Zilla-Plugin-Stenciller-MojoliciousTests</name></dist> <dist> <name>Pod-Elemental-Transformer-Stenciller</name></dist> <dist> <name>Stenciller-Plugin-ToMojoliciousTest</name></dist> </depended_on_by></dist> </depended_on_by></dist> <dist> <name>Map-Metro</name></dist> <dist> <name>MarpaX-Languages-M4</name></dist> <dist> <name>MooX-Object-Pluggable</name></dist> <dist> <name>MoopsX-ListObjects</name></dist> <dist> <name>MoopsX-UsingMoose</name></dist> <dist> <name>OpenGbg</name></dist> <dist> <name>Opendata-GTFS-Feed</name></dist> <dist> <name>Perl-Core</name></dist> <dist> <name>Pod-Elemental-Transformer-Stenciller</name></dist> <dist> <name>Pod-Weaver-Section-Badges</name> <depended_on_by count="1"> <dist> <name>Dist-Zilla-PluginBundle-SHANTANU</name> <depended_on_by count="2"> <dist> <name>Task-BeLike-Shantanu</name></dist> <dist> <name>Task-LiveDVD-FrankfurtPM</name></dist> </depended_on_by></dist> </depended_on_by></dist> <dist> <name>Stenciller</name></dist> <dist> <name>Syntax-Collection-Basic</name> <depended_on_by count="2"> <dist> <name>Mojolicious-Plugin-BootstrapHelpers</name> <depended_on_by count="2"> <dist> <name>Mojolicious-Plugin-DbicSchemaViewer</name></dist> <dist> <name>Task-MojoliciousPlugins-PerlAcademy</name></dist> </depended_on_by></dist> <dist> <name>Task-BeLike-CSSON</name></dist> </depended_on_by></dist> <dist> <name>Syntax-Highlight-WithEmacs</name></dist> <dist> <name>Task-BeLike-CSSON</name></dist> <dist> <name>Task-LiveDVD-FrankfurtPM</name></dist> <dist> <name>WebService-Intercom</name></dist> <dist> <name>WebService-Nonoh</name></dist> <dist> <name>XML-Simple-Sugar</name></dist> </depended_on_by></dist> </depended_on_by> </dist> <debug></debug> </reversedependencies>
  Thu May 04 11:36:00 2017 perl [...] toby.ink - Ticket #120908: Taken
  Sun May 07 03:56:53 2017 perl [...] toby.ink - Ticket #115112: Merged into ticket #114938
  Sun May 07 03:56:53 2017 perl [...] toby.ink - Merged into ticket #114938
  Sun May 07 03:57:21 2017 perl [...] toby.ink - Ticket #120908: Merged into ticket #114938
  Sun May 07 03:57:22 2017 perl [...] toby.ink - Merged into ticket #114938
  Sun May 07 03:57:56 2017 perl [...] toby.ink - Correspondence added
Fixed in 0.905.
  Sun May 07 03:57:57 2017 The RT System itself - Status changed from 'new' to 'open'
  Sun May 07 03:58:07 2017 perl [...] toby.ink - Status changed from 'open' to 'resolved'