Permalink
Browse files

Don't permit running with autodie < 2.12

autodie obliterated the IO layers set by the open pragma prior
to version 2.12, so now we refuse to run with autodie before
that bug was fixed.

RT #54777: autodie loses open pragma
GH #7: autodie breaks the open set by utf8::all
  • Loading branch information...
1 parent 70b85f5 commit e81cc6c0b5e88800018d4d55b6a941d52b84a4b7 @doherty committed Jul 10, 2012
Showing with 23 additions and 3 deletions.
  1. +1 −0 .gitignore
  2. +1 −0 Changes
  3. +2 −0 dist.ini
  4. +8 −3 lib/utf8/all.pm
  5. +11 −0 t/autodie.t
View
@@ -1,2 +1,3 @@
utf8-all*
.build
+*.orig
View
@@ -2,6 +2,7 @@ Revision history for Perl module {{$dist->name}}
{{$NEXT}}
* Use Import::Into instead of home-grown "solution" [GH #10]
+ * Don't permit running with autodie < 2.12, due to RT #54777 [GH #7]
0.004 2012-01-04
* Fix test suite for less current versions of Perl [getty, doherty]
View
@@ -6,3 +6,5 @@ copyright_holder = Michael Schwern <mschwern@cpan.org> ; he wrote it, not me
copyright_year = 2009
[@Author::DOHERTY]
+[Conflicts]
+autodie = 2.11
View
@@ -65,9 +65,14 @@ sub _encode_argv {
return;
}
-=for Pod::Coverage
-utf8_open
-unimport
+=head1 INTERACTION WITH AUTODIE
+
+If you use L<autodie>, which is a great idea, you need to use at least version
+B<2.12>, released on L<June 26, 2012|https://metacpan.org/source/PJF/autodie-2.12/Changes#L3>.
+Otherwise, autodie obliterates the IO layers set by the L<open> pragma. See
+L<RT #54777|https://rt.cpan.org/Ticket/Display.html?id=54777> and
+L<GH #7|https://github.com/doherty/utf8-all/issues/7>.
+
=cut
1;
View
@@ -0,0 +1,11 @@
+use strict;
+use warnings;
+use Test::More tests => 1;
+
+if ( eval { require autodie; 1 } ) {
+ ok $autodie::VERSION > 2.11
+ or diag "autodie is version $autodie::VERSION, but must be greater than version 2.11";
+}
+else {
+ pass;
+}

0 comments on commit e81cc6c

Please sign in to comment.