[Jifty-commit] r4533 - Jifty-DBI/trunk/lib/Jifty/DBI/Filter

[jifty-commit at lists.jifty.org]

Author: ishigaki
Date: Sat Nov 24 16:25:51 2007
New Revision: 4533

Modified:
   Jifty-DBI/trunk/lib/Jifty/DBI/Filter/YAML.pm

Log:
JDBI: use YAML () -> require YAML to suppress error when YAML is not installed (but ::Syck is installed)

Modified: Jifty-DBI/trunk/lib/Jifty/DBI/Filter/YAML.pm
==============================================================================
--- Jifty-DBI/trunk/lib/Jifty/DBI/Filter/YAML.pm	(original)
+++ Jifty-DBI/trunk/lib/Jifty/DBI/Filter/YAML.pm	Sat Nov 24 16:25:51 2007
@@ -8,7 +8,10 @@
 
 eval "use YAML::Syck ()";
 if ($@) { 
-    use YAML (); 
+    # We don't actually need to "use", which is checked at compile 
+    # time and would cause error when YAML is not installed.
+    # Or, eval this, too?
+    require YAML;
     $Dump = \&YAML::Dump;
     $Load = \&YAML::Load;
 }