[Catalyst-commits] r10933 - trunk/Config-Any/lib/Config/Any

[caelum at dev.catalyst.perl.org]

Author: caelum
Date: 2009-07-21 11:37:40 +0000 (Tue, 21 Jul 2009)
New Revision: 10933

Modified:
   trunk/Config-Any/lib/Config/Any/YAML.pm
Log:
add warning about using YAML loaders other than ::XS

Modified: trunk/Config-Any/lib/Config/Any/YAML.pm
===================================================================
--- trunk/Config-Any/lib/Config/Any/YAML.pm	2009-07-21 11:32:28 UTC (rev 10932)
+++ trunk/Config-Any/lib/Config/Any/YAML.pm	2009-07-21 11:37:40 UTC (rev 10933)
@@ -5,6 +5,8 @@
 
 use base 'Config::Any::Base';
 
+use Carp ();
+
 =head1 NAME
 
 Config::Any::YAML - Load YAML config files
@@ -48,6 +50,10 @@
         return YAML::XS::LoadFile( $file );
     }
 
+    Carp::carp
+"Using YAML::Syck or YAML to parse configs is DEPRECATED. Please install ".
+"YAML::XS for proper YAML support";
+
     eval { require YAML::Syck; YAML::Syck->VERSION( '0.70' ) };
     unless ( $@ ) {
         open( my $fh, $file ) or die $!;
@@ -62,8 +68,8 @@
 
 =head2 requires_any_of( )
 
-Specifies that this modules requires one of L<YAML::Syck> (0.70) or L<YAML> in 
-order to work.
+Specifies that this modules requires one of L<YAML::XS>, L<YAML::Syck> (0.70) or
+L<YAML> in order to work.
 
 =cut
 
@@ -88,6 +94,8 @@
 
 =item * L<Config::Any>
 
+=item * L<YAML::XS>
+
 =item * L<YAML>
 
 =item * L<YAML::Syck>