From: Decklin Foster <decklin@red-bean.com>
Date: Sun, 26 Oct 2008 18:31:12 +0000 (-0400)
Subject: Don't redirect run-mailcap to /dev/null
X-Git-Url: https://git.notmuchmail.org/git?a=commitdiff_plain;h=1d10748b5a58c3e7cf125fe45ba44f82e47c1970;p=sup

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 7eabcdf..cc895f3 100644
--- 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