various distribution fixes:
authorTony Cook <tony@develop-help.com>
Tue, 2 Jan 2007 11:12:53 +0000 (11:12 +0000)
committerTony Cook <tony@develop-help.com>
Tue, 2 Jan 2007 11:12:53 +0000 (11:12 +0000)
 - 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

MANIFEST
MANIFEST.SKIP
README
t/92manifest.t [new file with mode: 0644]

index 90c4d10..718fdec 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
index e29b7f3..eded43b 100644 (file)
@@ -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 07aa8d4..8a60f9c 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
@@ -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;