From 54f11a66f18061c7326eb0a505e332d0d78755db Mon Sep 17 00:00:00 2001
From: Tony Cook <tony@develop-help.com>
Date: Wed, 9 Apr 2008 07:46:50 +0000
Subject: [PATCH] - document that overlapping windows will show in a captured
 image.   http://rt.cpan.org/Ticket/Display.html?id=32842

- add appropriate configure_requires and build_requires to META.yml
  so that automated testers will install those pre-requisites early
  enough.
  http://rt.cpan.org/Ticket/Display.html?id=29560
---
 Changes       |  8 ++++++++
 Makefile.PL   | 17 ++++++++---------
 Screenshot.pm | 10 +++++++++-
 3 files changed, 25 insertions(+), 10 deletions(-)

diff --git a/Changes b/Changes
index 28e9d91..055de06 100755
--- a/Changes
+++ b/Changes
@@ -16,6 +16,14 @@
 - document the widget parameter
   https://rt.cpan.org/Ticket/Display.html?id=32855
 
+- document that overlapping windows will show in a captured image.
+  http://rt.cpan.org/Ticket/Display.html?id=32842
+
+- add appropriate configure_requires and build_requires to META.yml
+  so that automated testers will install those pre-requisites early
+  enough.
+  http://rt.cpan.org/Ticket/Display.html?id=29560
+
 0.005 12 Mar 2007
 	- 0.005 release
 	- added a TODO section to the POD
diff --git a/Makefile.PL b/Makefile.PL
index e314bd3..b008891 100644
--- a/Makefile.PL
+++ b/Makefile.PL
@@ -47,14 +47,6 @@ For X11: X11 headers and libraries, eg. the libX11-dev package on Debian
 DEAD
 }
 
-my $extra_meta = <<META;
-configure_requires:
-  Imager: 0.54
-build_requires:
-  Imager: 0.54
-  Test::More: 0.47
-META
-
 my %opts = 
   (
    NAME => 'Imager::Screenshot',
@@ -65,7 +57,6 @@ my %opts =
 		},
    INC => Imager::ExtUtils->includes,
    TYPEMAPS => [ Imager::ExtUtils->typemap ],
-   EXTRA_META => $extra_meta,
   );
 
 $opts{LIBS} = "@lflags" if @lflags;
@@ -81,8 +72,16 @@ if ($eu_mm_version > 6.06) {
 # LICENSE was introduced in 6.30_01, but Debian etch includes
 # (as of 2007/01/12) an ExtUtils::MakeMaker versioned 6.30_01 without
 # LICENSE support
+# EXTRA_META was also introduced in 6.30_01
 if ($eu_mm_version > 6.3001) {
   $opts{LICENSE} = 'perl';
+  $opts{EXTRA_META} = <<META;
+configure_requires:
+  Imager: 0.54
+build_requires:
+  Imager: 0.54
+  Test::More: 0.47
+META
 }
 
 WriteMakefile(%opts);
diff --git a/Screenshot.pm b/Screenshot.pm
index 215eb70..6fd5834 100644
--- a/Screenshot.pm
+++ b/Screenshot.pm
@@ -330,6 +330,12 @@ Cygwin/X I got the xterm window contents even when the Windows
 screensaver was running.  The root window captured appeared to be that
 generated by my window manager.
 
+Grabbing a window with other windows overlaying it will capture the
+content of those windows where they hide the window you want to
+capture.  You may want to raise the window to top.  This may be a
+security concern if the overlapping windows contain any sensitive
+information - true for any screen capture.
+
 =head1 LICENSE
 
 Imager::Screenshot is licensed under the same terms as Perl itself.
@@ -342,7 +348,9 @@ Future plans include:
 
 =item *
 
-OS X support - I need to find out which APIs to use to do this
+OS X support - I need to find out which APIs to use to do this.  I
+found some information on the APIs used for this, but don't have a Mac
+I can test on.
 
 =item *
 
-- 
2.20.1