[Jifty-commit] r5013 - in Jifty-DBI/branches/tisql: lib/Jifty/DBI/Filter

[Jifty commits]

Author: ruz
Date: Tue Feb  5 21:15:41 2008
New Revision: 5013

Modified:
   Jifty-DBI/branches/tisql/   (props changed)
   Jifty-DBI/branches/tisql/lib/Jifty/DBI/Filter/YAML.pm

Log:
 r4534 at cubic-pc (orig r4533):  ishigaki | 2007-11-25 00:25:51 +0300
 JDBI: use YAML () -> require YAML to suppress error when YAML is not installed (but ::Syck is installed)


Modified: Jifty-DBI/branches/tisql/lib/Jifty/DBI/Filter/YAML.pm
==============================================================================
--- Jifty-DBI/branches/tisql/lib/Jifty/DBI/Filter/YAML.pm	(original)
+++ Jifty-DBI/branches/tisql/lib/Jifty/DBI/Filter/YAML.pm	Tue Feb  5 21:15:41 2008
@@ -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;
 }