Don't redirect run-mailcap to /dev/null

It gets confused about stdout not being a terminal, and HTML attachments will
be converted to text/plain which uses the needsterminal entry for less(1).

diff --git a/lib/sup/message-chunks.rb b/lib/sup/message-chunks.rb
index 7eabcdf042ad5f29bc5b9423ff7c66d6c16e4361..cc895f3e480e591d077a18a83a84b85ca7d0cf5a 100644 (file)
--- a/lib/sup/message-chunks.rb
+++ b/lib/sup/message-chunks.rb
@@ -116,7 +116,7 @@ EOS
     def initial_state; :open end
     def viewable?; @lines.nil? end
     def view_default! path
-      cmd = "/usr/bin/run-mailcap --action=view '#{@content_type}:#{path}' > /dev/null 2> /dev/null"
+      cmd = "/usr/bin/run-mailcap --action=view '#{@content_type}:#{path}' 2>/dev/null"
       Redwood::log "running: #{cmd.inspect}"
       system cmd
       $? == 0