Skip Menu |

This queue is for tickets about the Catalyst-Runtime CPAN distribution.

Report information
The Basics
Id: 41920
Status: resolved
Priority: 0/
Queue: Catalyst-Runtime
People
Owner: Nobody in particular
Requestors: RADEK [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Unimportant
Broken in: 5.7015
Fixed in: (no value)
Attachments
signature.asc

History Show all quoted text
  Tue Dec 23 19:45:39 2008 RADEK [...] cpan.org - Ticket created
Subject: YAML test requirement missing
Tests (eg. live_fork.t optional_memleak.t optional_stress.t) use YAML, but it's not marked as required in the Makefile.PL (so tests fail if it's missing). Fix: --- Makefile.PL~ 2008-10-15 17:04:04.000000000 -0400 +++ Makefile.PL 2008-12-23 19:43:03.423327894 -0500 @@ -30,6 +30,8 @@ requires 'URI' => '1.35'; requires 'Text::Balanced'; # core in 5.8.x but mentioned for completeness +test_requires 'YAML'; + if (-e 'inc/.author') { build_requires 'Test::Perl::Critic';
  Thu Dec 25 09:50:39 2008 rafl [...] debian.org - Correspondence added
Subject: Re: [rt.cpan.org #41920] YAML test requirement missing
Date: Thu, 25 Dec 2008 15:47:42 +0100
To: Radoslaw Zielinski via RT <bug-Catalyst-Runtime [...] rt.cpan.org>
From: Florian Ragwitz <rafl [...] debian.org>
On Tue, Dec 23, 2008 at 07:45:39PM -0500, Radoslaw Zielinski via RT wrote: Show quoted text
> Tests (eg. live_fork.t optional_memleak.t optional_stress.t) use YAML, > but it's not marked as required in the Makefile.PL (so tests fail if > it's missing). Fix:
Fixed in 5.80/trunk by skipping those tests if YAML isn't present. -- BOFH excuse #215: High nuclear activity in your area.
Download signature.asc
application/pgp-signature 189b

Message body not shown because it is not plain text.

  Thu Dec 25 09:50:40 2008 The RT System itself - Status changed from 'new' to 'open'
  Wed Feb 04 09:51:58 2009 rafl [...] debian.org - Status changed from 'open' to 'resolved'