Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Test-Dependencies CPAN distribution.

Report information
The Basics
Id: 51023
Status: resolved
Priority: 0/
Queue: Test-Dependencies
People
Owner: Nobody in particular
Requestors: NCLEATON [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 0.11
Fixed in: (no value)

History Show all quoted text
  Sun Nov 01 10:47:30 2009 NCLEATON [...] cpan.org - Ticket created
Subject: [PATCH] fix bogus 'perl is not a run-time dependency' failure
It's common to list the minimum required Perl version for a module as a requirement for 'perl' in META.yml, e.g: build_requires: Test::More: 0 requires: perl: 5.006001 Test::Dependencies wrongly reports this as an error, because no module called "perl" is used by any of .pm files. The attached patch corrects this.
  Sun Nov 01 10:48:50 2009 NCLEATON [...] cpan.org - Correspondence added
forgot to attach the patch, here it is
diff -Nurd Test-Dependencies-0.11.orig/lib/Test/Dependencies.pm Test-Dependencies-0.11/lib/Test/Dependencies.pm --- Test-Dependencies-0.11.orig/lib/Test/Dependencies.pm 2007-12-27 22:52:39.000000000 +0200 +++ Test-Dependencies-0.11/lib/Test/Dependencies.pm 2009-11-01 17:40:37.000000000 +0200 @@ -201,6 +201,7 @@ } my $meta = LoadFile('META.yml') or die 'Could not load META.YML'; my %required = exists $meta->{requires} && defined $meta->{requires} ? %{$meta->{requires}} : (); + delete $required{'perl'}; my %build_required = exists $meta->{build_requires} ? %{$meta->{build_requires}} : (); foreach my $mod (sort keys %used) {
  Sun Jun 26 16:01:58 2016 EHUELS [...] cpan.org - Correspondence added
If not dealt with yet, will be dealt with in the next release.
  Sun Jun 26 16:01:59 2016 The RT System itself - Status changed from 'new' to 'open'
  Sun Jun 26 16:02:04 2016 EHUELS [...] cpan.org - Status changed from 'open' to 'resolved'