Removed the query asking if GIF support should be disabled, since
authorTony Cook <tony@develop=help.com>
Mon, 7 Aug 2006 07:02:43 +0000 (07:02 +0000)
committerTony Cook <tony@develop=help.com>
Mon, 7 Aug 2006 07:02:43 +0000 (07:02 +0000)
recent lib(un)gif are sane.

https://rt.cpan.org/Ticket/Display.html?id=20687

Some README cleanup.

Makefile.PL
README

index 2279fe5..577b0c5 100644 (file)
@@ -258,32 +258,6 @@ sub gifcheck {
     delete $formats{'ungif'};
   }
 
- RETR:
-  if (($formats{'gif'} or $formats{'ungif'}) && !$ENV{IM_SUPPRESS_PROMPT}) {
-    my $msg = <<EOFF;
-
-You have libgif or libungif installed.  They are both known to have
-bugs.  Imager can crash or display other strange behaviour after
-reading or writing gif images.  Some of the gif tests can fail since
-they stress some parts of the buggy code.
-
-libungif or libgif 4.1.4 and later is safe.
-
-Of course it's possible your operating system distributor has patched
-all of these problems and you have nothing to worry about.
-
-Do you want to remove gif support? [Y/n]
-EOFF
-    chomp $msg;
-    my $resp = prompt($msg);
-    chomp($resp);
-    if ($resp ne "n") {
-      delete $formats{'gif'};
-      delete $formats{'ungif'};
-      return;
-    }
-  }
-
   for my $frm (qw(gif ungif)) {
     checkformat($frm) if ($MANUAL and $formats{$frm});
   }
diff --git a/README b/README
index 895835f..17fde2f 100644 (file)
--- a/README
+++ b/README
@@ -89,7 +89,7 @@ included since libgif has been linked with it).
 Otherwise you could just build giflib without any X11 dependencies:
 
   # must be a clean tree
-  cd giflib-4.1.3
+  cd giflib-4.1.4
   ./configure --without-x ...
 
 Also note that libgif has a few bugs: You can run something like
@@ -106,7 +106,7 @@ not libungif on RedHat then you will probably get lots of errors
 like undefined symbol: FreeSavedImages when running make test.
 Install libungif package to fix it.
 
-Stock libungif 4.1.2 or later seems to fix all of the bugs, if you
+Stock libungif 4.1.4 or later seems to fix all of the bugs, if you
 have a problem that version of linungif (or later), let us know and
 we'll look into it.
 
@@ -190,17 +190,14 @@ potential security problem.
 
   gif: http://sourceforge.net/projects/libungif
 
-giflib/libungif has come a long way since the buggy versions around
+giflib/libungif has come a long way since the buggy versions available
 when Imager's gif support code was written.  Preferably you should get
-at least version 4.1.3.  If you have a recent Linux distribution you
+at least version 4.1.4.  If you have a recent Linux distribution you
 should be safe with whatever giflib it provides, but if you're
 building from source, please try to use the latest version.
 
 libgif 4.1.4 has no problems known to me at this point.
 
-At the time of writing you will need to manually select to install the
-4.1.2-1 of cygwin's libungif package.
-
  tiff: http://www.libtiff.org/
 
    t1: http://www.ibiblio.org/pub/Linux/libs/graphics/
@@ -247,11 +244,6 @@ you will need to install the cygwin rebase package and run:
 
   $ rebaseall -v
 
-Under cygwin you will need at least libungif 4.1.2 installed to
-prevent lockups in the gif test scripts.  At the time of writing you
-need to select libungif-4.1.2-1 manually in the setup.exe installation
-tool.
-
 If you get errors from your make tool, make sure you're using the same
 make that was used to build your perl - generally GNU make for cygwin,
 nmake for Visual C/C++ and dmake for MinGW.
@@ -297,12 +289,6 @@ The Imager module homepage is currently at:
 
  http://imager.perl.org/
 
-The current docs are rather bad as I've been busy adding features
-but hopefully they will be updated soon.  Until then you'll just
-have to use the source. The test scripts might also be a good idea.
-By activating the the #init_log lines in the test script you can get
-rather verbose debugging output from the C code.
-
 You can report bugs by pointing your browser at:
 
   https://rt.cpan.org/NoAuth/ReportBug.html?Queue=Imager