Fix monkey-patching of Tempfile
authorMichael Hamann <michael@content-space.de>
Fri, 8 Oct 2010 14:03:36 +0000 (10:03 -0400)
committerRich Lane <rlane@club.cc.cmu.edu>
Wed, 13 Oct 2010 06:29:54 +0000 (23:29 -0700)
commit60573298a2258c101a85b3de8121f73d7aec2d51
tree4f69a5d892f071ea7c9c75ecaf9e6d3999867275
parent07f0be019f60901594b6a1c0e0d042bea52367d7
Fix monkey-patching of Tempfile

In Ruby 1.9.2 the interal function make_tmpname has changed it's
parameters and the second parameter can be nil now. This breaks the
monkey-patched Tempfile class in sup. Additionally, monkey-patching is
not really necessary as the Tempfile class supports an array as basename
instead of a string containing prefix and a suffix since Ruby version
1.8.7. The only place I've found where the suffix might matter is
directly in message-chunks.rb where I've changed the basename to an
array. Additionally the new make_tmpname function from Ruby 1.9.2 is
monkey-patched for Ruby versions lower than 1.8.7 now.

NOTE: As I don't have a working Ruby 1.8 setup for sup I haven't tested
this with older versions, but the code works here when the version check
is changed to 1.9.3 and I can't see anything that shouldn't work with
Ruby 1.8.6 and below.
lib/sup/message-chunks.rb