Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the SVN-Notify CPAN distribution.

Report information
The Basics
Id: 20267
Status: resolved
Worked: 10 min
Priority: 0/
Queue: SVN-Notify

People
Owner: dwheeler [...] cpan.org
Requestors: rjbs [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 2.62
Fixed in: 2.63



Subject: html::entities required in test
In trying to "cpan SVN::Notify" I had tests in options.t fail because HTML::Entities was not available. Some test files skip if it isn't there, but options.t does not.
Subject: Re: [rt.cpan.org #20267] html::entities required in test
Date: Mon, 3 Jul 2006 20:23:39 -0700
To: bug-SVN-Notify [...] rt.cpan.org
From: David Wheeler <david [...] kineticode.com>
Thanks. This patch will be in the next release: Index: t/options.t =================================================================== --- t/options.t (revision 2916) +++ t/options.t (working copy) @@ -80,3 +80,8 @@ @ARGV = (%testopts, '--bugzilla-url' => 'url', '--handler' => 'HTML'); ok $opts = SVN::Notify->get_options, "Get SVN::Notify + HTML options"; is_deeply($opts, \%params, "Check new results"); + +BEGIN { + package HTML::Entities; + $INC{'HTML/Entities.pm'} = __FILE__; +} Index: Changes =================================================================== --- Changes (revision 2928) +++ Changes (working copy) @@ -1,6 +1,8 @@ Revision history for Perl extension SVN::Notify 2.63 + - The tests in t/options.t no longer fail when HTML::Entities is not + installed. Thanks to Ricardo Signes for the spot! 2.62 2006-06-30T18:03:21 - Changed to() accessor to return the first value in the array in scalar
Fixed in 2.63. Thanks for the report.