Hello community,
here is the log from the commit of package perl-Test-FailWarnings for
openSUSE:Factory checked in at 2013-09-27 17:57:02
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/perl-Test-FailWarnings (Old)
and /work/SRC/openSUSE:Factory/.perl-Test-FailWarnings.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "perl-Test-FailWarnings"
Changes:
--------
---
/work/SRC/openSUSE:Factory/perl-Test-FailWarnings/perl-Test-FailWarnings.changes
2013-08-27 21:04:26.000000000 +0200
+++
/work/SRC/openSUSE:Factory/.perl-Test-FailWarnings.new/perl-Test-FailWarnings.changes
2013-09-27 17:57:04.000000000 +0200
@@ -1,0 +2,10 @@
+Mon Sep 2 11:12:09 UTC 2013 - [email protected]
+
+- Update to version 0.007
+ * In some situations, the source of a local warning can have an
+ absolute filename and would not be reported if ALLOW_DEPS were true.
+ Now absolute filenames are made relative to the original cwd before
+ checking if they are local or not.
+ * Fixed synopsis to match new failing test label
+
+-------------------------------------------------------------------
Old:
----
Test-FailWarnings-0.006.tar.gz
New:
----
Test-FailWarnings-0.007.tar.gz
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Other differences:
------------------
++++++ perl-Test-FailWarnings.spec ++++++
--- /var/tmp/diff_new_pack.eBr5c3/_old 2013-09-27 17:57:04.000000000 +0200
+++ /var/tmp/diff_new_pack.eBr5c3/_new 2013-09-27 17:57:04.000000000 +0200
@@ -17,7 +17,7 @@
Name: perl-Test-FailWarnings
-Version: 0.006
+Version: 0.007
Release: 0
%define cpan_name Test-FailWarnings
Summary: Add test failures if warnings are caught
++++++ Test-FailWarnings-0.006.tar.gz -> Test-FailWarnings-0.007.tar.gz ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/Test-FailWarnings-0.006/Changes
new/Test-FailWarnings-0.007/Changes
--- old/Test-FailWarnings-0.006/Changes 2013-08-22 16:41:14.000000000 +0200
+++ new/Test-FailWarnings-0.007/Changes 2013-08-30 15:53:50.000000000 +0200
@@ -1,5 +1,18 @@
Revision history for Test-FailWarnings
+0.007 2013-08-30 09:53:47 America/New_York
+
+ [FIXED]
+
+ - In some situations, the source of a local warning can have an
+ absolute filename and would not be reported if ALLOW_DEPS were true.
+ Now absolute filenames are made relative to the original cwd before
+ checking if they are local or not.
+
+ [DOCUMENTED]
+
+ - Fixed synopsis to match new failing test label
+
0.006 2013-08-22 10:41:11 America/New_York
[CHANGED]
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/Test-FailWarnings-0.006/META.json
new/Test-FailWarnings-0.007/META.json
--- old/Test-FailWarnings-0.006/META.json 2013-08-22 16:41:14.000000000
+0200
+++ new/Test-FailWarnings-0.007/META.json 2013-08-30 15:53:50.000000000
+0200
@@ -4,7 +4,7 @@
"David Golden <[email protected]>"
],
"dynamic_config" : 0,
- "generated_by" : "Dist::Zilla version 4.300035, CPAN::Meta::Converter
version 2.132140",
+ "generated_by" : "Dist::Zilla version 4.300037, CPAN::Meta::Converter
version 2.132140",
"license" : [
"apache_2_0"
],
@@ -41,6 +41,8 @@
"runtime" : {
"requires" : {
"Carp" : "0",
+ "Cwd" : "0",
+ "File::Spec" : "0",
"Test::More" : "0.86",
"perl" : "5.008001",
"strict" : "0",
@@ -51,11 +53,13 @@
"requires" : {
"Capture::Tiny" : "0.12",
"ExtUtils::MakeMaker" : "0",
+ "File::Spec" : "0",
"File::Spec::Functions" : "0",
"File::Temp" : "0",
+ "IO::Handle" : "0",
+ "IPC::Open3" : "0",
"List::Util" : "0",
"Test::More" : "0.96",
- "blib" : "0",
"constant" : "0",
"lib" : "0"
}
@@ -64,7 +68,7 @@
"provides" : {
"Test::FailWarnings" : {
"file" : "lib/Test/FailWarnings.pm",
- "version" : "0.006"
+ "version" : "0.007"
}
},
"release_status" : "stable",
@@ -79,7 +83,7 @@
"web" : "https://github.com/dagolden/test-failwarnings";
}
},
- "version" : "0.006",
+ "version" : "0.007",
"x_authority" : "cpan:DAGOLDEN"
}
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/Test-FailWarnings-0.006/META.yml
new/Test-FailWarnings-0.007/META.yml
--- old/Test-FailWarnings-0.006/META.yml 2013-08-22 16:41:14.000000000
+0200
+++ new/Test-FailWarnings-0.007/META.yml 2013-08-30 15:53:50.000000000
+0200
@@ -5,17 +5,19 @@
build_requires:
Capture::Tiny: 0.12
ExtUtils::MakeMaker: 0
+ File::Spec: 0
File::Spec::Functions: 0
File::Temp: 0
+ IO::Handle: 0
+ IPC::Open3: 0
List::Util: 0
Test::More: 0.96
- blib: 0
constant: 0
lib: 0
configure_requires:
ExtUtils::MakeMaker: 6.30
dynamic_config: 0
-generated_by: 'Dist::Zilla version 4.300035, CPAN::Meta::Converter version
2.132140'
+generated_by: 'Dist::Zilla version 4.300037, CPAN::Meta::Converter version
2.132140'
license: apache
meta-spec:
url: http://module-build.sourceforge.net/META-spec-v1.4.html
@@ -32,9 +34,11 @@
provides:
Test::FailWarnings:
file: lib/Test/FailWarnings.pm
- version: 0.006
+ version: 0.007
requires:
Carp: 0
+ Cwd: 0
+ File::Spec: 0
Test::More: 0.86
perl: 5.008001
strict: 0
@@ -43,5 +47,5 @@
bugtracker: https://github.com/dagolden/test-failwarnings/issues
homepage: https://metacpan.org/release/Test-FailWarnings
repository: git://github.com/dagolden/test-failwarnings.git
-version: 0.006
+version: 0.007
x_authority: cpan:DAGOLDEN
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/Test-FailWarnings-0.006/Makefile.PL
new/Test-FailWarnings-0.007/Makefile.PL
--- old/Test-FailWarnings-0.006/Makefile.PL 2013-08-22 16:41:14.000000000
+0200
+++ new/Test-FailWarnings-0.007/Makefile.PL 2013-08-30 15:53:50.000000000
+0200
@@ -21,6 +21,8 @@
"NAME" => "Test::FailWarnings",
"PREREQ_PM" => {
"Carp" => 0,
+ "Cwd" => 0,
+ "File::Spec" => 0,
"Test::More" => "0.86",
"strict" => 0,
"warnings" => 0
@@ -28,15 +30,17 @@
"TEST_REQUIRES" => {
"Capture::Tiny" => "0.12",
"ExtUtils::MakeMaker" => 0,
+ "File::Spec" => 0,
"File::Spec::Functions" => 0,
"File::Temp" => 0,
+ "IO::Handle" => 0,
+ "IPC::Open3" => 0,
"List::Util" => 0,
"Test::More" => "0.96",
- "blib" => 0,
"constant" => 0,
"lib" => 0
},
- "VERSION" => "0.006",
+ "VERSION" => "0.007",
"test" => {
"TESTS" => "t/*.t"
}
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/Test-FailWarnings-0.006/README
new/Test-FailWarnings-0.007/README
--- old/Test-FailWarnings-0.006/README 2013-08-22 16:41:14.000000000 +0200
+++ new/Test-FailWarnings-0.007/README 2013-08-30 15:53:50.000000000 +0200
@@ -2,7 +2,7 @@
Test::FailWarnings - Add test failures if warnings are caught
VERSION
- version 0.006
+ version 0.007
SYNOPSIS
Test file:
@@ -20,8 +20,8 @@
Output:
ok 1 - first test
- not ok 2 - Caught warning
- # Failed test 'Caught warning'
+ not ok 2 - Test::FailWarnings should catch no warnings
+ # Failed test 'Test::FailWarnings should catch no warnings'
# at t/bin/main-warn.pl line 7.
# Warning was 'Argument "lkadjaks" isn't numeric in addition (+) at
t/bin/main-warn.pl line 7.'
ok 3 - add non-numeric
@@ -29,7 +29,7 @@
# Looks like you failed 1 test of 3.
DESCRIPTION
- This module hooks $SIG{__WARN__} and converts warnings to Test::More's
+ This module hooks $SIG{__WARN__} and converts warnings to Test::More
"fail()" calls. It is designed to be used with "done_testing", when you
don't need to know the test count in advance.
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/Test-FailWarnings-0.006/lib/Test/FailWarnings.pm
new/Test-FailWarnings-0.007/lib/Test/FailWarnings.pm
--- old/Test-FailWarnings-0.006/lib/Test/FailWarnings.pm 2013-08-22
16:41:14.000000000 +0200
+++ new/Test-FailWarnings-0.007/lib/Test/FailWarnings.pm 2013-08-30
15:53:50.000000000 +0200
@@ -4,14 +4,18 @@
package Test::FailWarnings;
# ABSTRACT: Add test failures if warnings are caught
-our $VERSION = '0.006'; # VERSION
+our $VERSION = '0.007'; # VERSION
use Test::More 0.86;
+use Cwd qw/getcwd/;
+use File::Spec;
use Carp;
our $ALLOW_DEPS = 0;
our @ALLOW_FROM = ();
+my $ORIG_DIR = getcwd(); # cache in case handler runs after a chdir
+
sub import {
my ( $class, @args ) = @_;
croak("import arguments must be key/value pairs")
@@ -33,7 +37,11 @@
# shortcut if ignoring dependencies and warning did not
# come from something local
- return if $ALLOW_DEPS && $filename !~ /^(?:t|xt|lib|blib)/;
+ if ( $ALLOW_DEPS ) {
+ $filename = File::Spec->abs2rel( $filename, $ORIG_DIR )
+ if File::Spec->file_name_is_absolute( $filename );
+ return if $filename !~ /^(?:t|xt|lib|blib)/;
+ }
return if grep { $package eq $_ } @ALLOW_FROM;
@@ -76,7 +84,7 @@
=head1 VERSION
-version 0.006
+version 0.007
=head1 SYNOPSIS
@@ -95,8 +103,8 @@
Output:
ok 1 - first test
- not ok 2 - Caught warning
- # Failed test 'Caught warning'
+ not ok 2 - Test::FailWarnings should catch no warnings
+ # Failed test 'Test::FailWarnings should catch no warnings'
# at t/bin/main-warn.pl line 7.
# Warning was 'Argument "lkadjaks" isn't numeric in addition (+) at
t/bin/main-warn.pl line 7.'
ok 3 - add non-numeric
@@ -105,7 +113,7 @@
=head1 DESCRIPTION
-This module hooks C<$SIG{__WARN__}> and converts warnings to L<Test::More>'s
+This module hooks C<$SIG{__WARN__}> and converts warnings to L<Test::More>
C<fail()> calls. It is designed to be used with C<done_testing>, when you
don't need to know the test count in advance.
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/Test-FailWarnings-0.006/t/00-compile.t
new/Test-FailWarnings-0.007/t/00-compile.t
--- old/Test-FailWarnings-0.006/t/00-compile.t 2013-08-22 16:41:14.000000000
+0200
+++ new/Test-FailWarnings-0.007/t/00-compile.t 2013-08-30 15:53:50.000000000
+0200
@@ -1,19 +1,17 @@
use strict;
use warnings;
-# This test was generated via Dist::Zilla::Plugin::Test::Compile 2.018
+# this test was generated with Dist::Zilla::Plugin::Test::Compile 2.022
use Test::More 0.88;
-use Capture::Tiny qw{ capture };
-
-my @module_files = qw(
-Test/FailWarnings.pm
+my @module_files = (
+ 'Test/FailWarnings.pm'
);
-my @scripts = qw(
+my @scripts = (
);
@@ -22,15 +20,26 @@
local $ENV{HOME} = File::Temp::tempdir( CLEANUP => 1 );
+use IPC::Open3;
+use IO::Handle;
+use File::Spec;
+
my @warnings;
for my $lib (@module_files)
{
- my ($stdout, $stderr, $exit) = capture {
- system($^X, '-Mblib', '-e', qq{require q[$lib]});
- };
- is($?, 0, "$lib loaded ok");
- warn $stderr if $stderr;
- push @warnings, $stderr if $stderr;
+ open my $stdout, '>', File::Spec->devnull or die $!;
+ open my $stdin, '<', File::Spec->devnull or die $!;
+ my $stderr = IO::Handle->new;
+
+ my $pid = open3($stdin, $stdout, $stderr, qq{$^X -Mblib -e"require
q[$lib]"});
+ waitpid($pid, 0);
+ is($? >> 8, 0, "$lib loaded ok");
+
+ if (my @_warnings = <$stderr>)
+ {
+ warn @_warnings;
+ push @warnings, @_warnings;
+ }
}
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn'
'--exclude=.svnignore' old/Test-FailWarnings-0.006/t/00-report-prereqs.t
new/Test-FailWarnings-0.007/t/00-report-prereqs.t
--- old/Test-FailWarnings-0.006/t/00-report-prereqs.t 2013-08-22
16:41:14.000000000 +0200
+++ new/Test-FailWarnings-0.007/t/00-report-prereqs.t 2013-08-30
15:53:50.000000000 +0200
@@ -12,12 +12,15 @@
my @modules = qw(
Capture::Tiny
Carp
+ Cwd
ExtUtils::MakeMaker
+ File::Spec
File::Spec::Functions
File::Temp
+ IO::Handle
+ IPC::Open3
List::Util
Test::More
- blib
constant
lib
perl
--
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
