Add ruby 1.8 fallbacks for IO.binread

1 files changed, 14 insertions, 2 deletions

diff --git a/lib/chef/mixin/template.rb b/lib/chef/mixin/template.rb
index f92bbb4325..f515f5c00d 100644
--- a/lib/chef/mixin/template.rb
+++ b/lib/chef/mixin/template.rb
@@ -23,6 +23,18 @@ class Chef
   module Mixin
     module Template
 
+      # A compatibility wrapper around IO.binread so it works on Ruby 1.8.7.
+      # --
+      # Used in the TemplateContext class, but that method namespace is shared
+      # with user code, so we want to avoid adding methods there when possible.
+      def self.binread(file)
+        if IO.respond_to?(:binread)
+          IO.binread(file)
+        else
+          IO.read(file)
+        end
+      end
+
       # == ChefContext
       # ChefContext was previously used to mix behavior into Erubis::Context so
       # that it would be available to templates. This behavior has now moved to
@@ -93,11 +105,11 @@ class Chef
           partial_context._extend_modules(@_extension_modules)
 
           template_location = @template_finder.find(partial_name, options)
-          _render_template(IO.binread(template_location), partial_context)
+          _render_template(Mixin::Template.binread(template_location), partial_context)
         end
 
         def render_template(template_location)
-          _render_template(IO.binread(template_location), self)
+          _render_template(Mixin::Template.binread(template_location), self)
         end
 
         def render_template_from_string(template)