Skip Menu |

This queue is for tickets about the Lexical-SealRequireHints CPAN distribution.

Report information
The Basics
Id: 107927
Status: resolved
Priority: 0/
Queue: Lexical-SealRequireHints
People
Owner: Nobody in particular
Requestors: rt [...] illuminated.co.uk
Cc:
AdminCc:
Bug Information
Severity: Critical
Broken in: 0.009
Fixed in: (no value)

History Show all quoted text
  Thu Oct 22 13:29:53 2015 rt [...] illuminated.co.uk - Ticket created
Subject: t/defsv.t assumes lexical $_ will trigger a warning
In perl 5.23.4, lexical $_ is a compile error, so is $@, ""; fails. This test should be skipped in perl 5.23.4 and above.
  Sat Oct 31 16:33:21 2015 SREZIC [...] cpan.org - Correspondence added
On 2015-10-22 13:29:53, SKINGTON wrote: Show quoted text
> In perl 5.23.4, lexical $_ is a compile error, so > is $@, ""; > fails. > > This test should be skipped in perl 5.23.4 and above.
Patch attached.
Subject: Lexical-SealRequireHints-0.009-RT107927.patch
From a803baa3a29996c1466f7e215eac1f56ee5c1e50 Mon Sep 17 00:00:00 2001 From: Slaven Rezic <slaven@rezic.de> Date: Sat, 31 Oct 2015 21:24:30 +0100 Subject: [PATCH] no lexical $_ with perl 5.23.4 and later (RT #107927) --- t/defsv.t | 1 + 1 file changed, 1 insertion(+) diff --git a/t/defsv.t b/t/defsv.t index 4f3cab2..749431a 100644 --- a/t/defsv.t +++ b/t/defsv.t @@ -14,6 +14,7 @@ SKIP: { is $retval, "t::context_0 return"; SKIP: { skip "no lexical \$_ on this perl", 2 if "$]" < 5.009001; + skip "no lexical \$_ on this perl", 2 if "$]" >= 5.023004; eval q{ no warnings "$]" >= 5.017009 ? "experimental" : "deprecated"; -- 2.1.2
  Sat Oct 31 16:33:22 2015 The RT System itself - Status changed from 'new' to 'open'
  Sat Jul 15 16:40:27 2017 ZEFRAM [...] cpan.org - Correspondence added
Was fixed in L:SRH-0.010.
  Sat Jul 15 16:40:28 2017 ZEFRAM [...] cpan.org - Status changed from 'open' to 'resolved'