various distribution fixes:

 - README was missing from the dist, add it in
 - exclude build junk from the MANIFEST
 - added a test to make our MANIFEST is complete
 - sort the MANIFEST case-insensitively, since EU::Manifest::mkmanifest
   likes it that way

diff --git a/MANIFEST b/MANIFEST
index 90c4d101667b34afda2034c99de15d1bdfab7193..718fdec820ba51513b1c2c9bec75effd4ecca81b 100644 (file)
--- a/MANIFEST
+++ b/MANIFEST
@@ -1,11 +1,13 @@
 Changes                                History
 Changes                                History

+examples/win32_fw.pl           Example - get a window by name on win32 and grab it
+imss.h                         Declarations of the functions the XS gives access to
+Makefile.PL                    Build script

 MANIFEST                       This list of files
 MANIFEST.SKIP                  Files not to include in the MANIFEST
 MANIFEST                       This list of files
 MANIFEST.SKIP                  Files not to include in the MANIFEST

-Makefile.PL                    Build script
+META.yml                       Module meta-data (added by MakeMaker)
+README

 Screenshot.pm                  Main perl implementation
 Screenshot.xs                  Interface to C code
 Screenshot.pm                  Main perl implementation
 Screenshot.xs                  Interface to C code

-examples/win32_fw.pl           Example - get a window by name on win32 and grab it
-imss.h                         Declarations of the functions the XS gives access to

 scwin32.c                      Win32 implementation
 scx11.c                                X11 implementation
 t/00load.t                     Test - can we load the modules
 scwin32.c                      Win32 implementation
 scx11.c                                X11 implementation
 t/00load.t                     Test - can we load the modules


@@ -15,4 +17,4 @@ t/30tkx11.t                   Test - X11 implementation via Tk
 t/40tkwin32.t                  Test - Win32 implementation via Tk
 t/90pod.t                      Test - check POD is valid
 t/91podcover.t                 Test - check all functions are covered by POD
 t/40tkwin32.t                  Test - Win32 implementation via Tk
 t/90pod.t                      Test - check POD is valid
 t/91podcover.t                 Test - check all functions are covered by POD

-META.yml                                 Module meta-data (added by MakeMaker)
+t/92manifest.t                 Test - check we match the MANIFEST

diff --git a/MANIFEST.SKIP b/MANIFEST.SKIP
index e29b7f370cd6ad86c3dce3e478ced2f868e28a8a..eded43b53e2ba166d538aaad812c4e98ccafed06 100644 (file)
--- a/MANIFEST.SKIP
+++ b/MANIFEST.SKIP
@@ -11,3 +11,6 @@ vc60\.pdb
 \.o$
 \.bak$
 ^Screenshot.c$
 \.o$
 \.bak$
 ^Screenshot.c$

+^Imager-Screenshot-.*\.tar\.gz$
+^Imager-Screenshot-.*/
+

diff --git a/README b/README
index 07aa8d4d7f04095c0c49ea99b9839466262412ab..8a60f9c17a8656086ccdf7f8a5541de6ffc665dd 100755 (executable)
--- a/README
+++ b/README
@@ -7,7 +7,7 @@ Requires:
   X11 header files and libraries for X11 support.
   A C compiler compatible with that used to build perl itself.
 
   X11 header files and libraries for X11 support.
   A C compiler compatible with that used to build perl itself.
 

-Optional
+Optional:

   Perl/Tk
 
 Installation:
   Perl/Tk
 
 Installation:


@@ -24,3 +24,7 @@ Tested under:
  - Win32 (mingw)
  - Win32/X11 (cygwin as of Jan 1 2007)
  - X11 (Debian Linux)
  - Win32 (mingw)
  - Win32/X11 (cygwin as of Jan 1 2007)
  - X11 (Debian Linux)

+
+License:
+
+Imager::Screenshot is distributed under the same terms as perl itself.

diff --git a/t/92manifest.t b/t/92manifest.t
new file mode 100644 (file)
index 0000000..df49233
--- /dev/null
+++ b/t/92manifest.t
@@ -0,0 +1,21 @@
+#!perl -w
+use strict;
+use Test::More;
+use ExtUtils::Manifest 'fullcheck';
+
+# by default EU::Manifest complains to STDERR, suppress that
+$ExtUtils::Manifest::Quiet = 1;
+
+my $dev_tests = -e '.svn';
+if ($dev_tests) {
+  plan tests => 2; # for local tests we check that there's no extras
+}
+else {
+  plan tests => 1; # a user may have created junk playing with it
+}
+
+my ($missing, $extra) = fullcheck();
+
+is_deeply($missing, [], "No files missing");
+is_deeply($extra,   [], "No extra files")
+  if $dev_tests;