Merge pull request #195 from MichaelTimbert/119

Check MANIFEST when PR is created

MANIFEST.SKIP

@@ -131,4 +131,8 @@
 
 # Avoid MYMETA files
 ^MYMETA\.
+
+# Avoid MANIFEST test
+t/manifest.t
+
 #!end included /usr/share/perl/5.20/ExtUtils/MANIFEST.SKIP

Makefile.PL

@@ -127,6 +127,7 @@ test_requires      'MIME::Base32'            => 0;
 test_requires      'Test::Fatal'             => 0;
 test_requires      'Test::Differences'       => 0;
 test_requires      'Test::More'              => 1.302015;
+test_requires      'Test::NoWarnings'        => 0;
 
 use_ppport 3.19;
 cc_include_paths 'include';

t/manifest.t

@@ -0,0 +1,37 @@
+#!perl
+use v5.14.2;
+use strict;
+use warnings;
+use utf8;
+use Test::More tests => 2;
+use Test::NoWarnings;
+
+use File::Basename qw( dirname );
+
+chdir dirname( dirname( __FILE__ ) ) or BAIL_OUT( "chdir: $!" );
+
+my $makebin = 'make';
+
+sub make {
+    my @make_args = @_;
+
+    undef $ENV{MAKEFLAGS};
+
+    my $command = join( ' ', $makebin, '-s', @make_args );
+    my $output = `$command 2>&1`;
+
+    if ( $? == -1 ) {
+        BAIL_OUT( "failed to execute: $!" );
+    }
+    elsif ( $? & 127 ) {
+        BAIL_OUT( "child died with signal %d, %s coredump\n", ( $? & 127 ), ( $? & 128 ) ? 'with' : 'without' );
+    }
+
+    return $output, $? >> 8;
+}
+
+subtest "distcheck" => sub {
+    my ( $output, $status ) = make "distcheck";
+    is $status, 0,  $makebin . ' distcheck exits with value 0';
+    is $output, "", $makebin . ' distcheck gives empty output';
+};