Re: [PATCH] VIM: Add better attachment support
authorFranz Fellner <alpine.art.de@gmail.com>
Fri, 10 Oct 2014 09:57:17 +0000 (11:57 +0200)
committerW. Trevor King <wking@tremily.us>
Fri, 7 Nov 2014 18:05:11 +0000 (10:05 -0800)
5c/f5032687a5437be3952da019df6e35fb8c3938 [new file with mode: 0644]

diff --git a/5c/f5032687a5437be3952da019df6e35fb8c3938 b/5c/f5032687a5437be3952da019df6e35fb8c3938
new file mode 100644 (file)
index 0000000..7bc0b97
--- /dev/null
@@ -0,0 +1,221 @@
+Return-Path: <alpine.art.de@gmail.com>\r
+X-Original-To: notmuch@notmuchmail.org\r
+Delivered-To: notmuch@notmuchmail.org\r
+Received: from localhost (localhost [127.0.0.1])\r
+       by olra.theworths.org (Postfix) with ESMTP id 992F4431FC0\r
+       for <notmuch@notmuchmail.org>; Fri, 10 Oct 2014 02:57:32 -0700 (PDT)\r
+X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
+X-Spam-Flag: NO\r
+X-Spam-Score: -0.799\r
+X-Spam-Level: \r
+X-Spam-Status: No, score=-0.799 tagged_above=-999 required=5\r
+       tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
+       FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
+Received: from olra.theworths.org ([127.0.0.1])\r
+       by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
+       with ESMTP id V6bSIO5lbZ8n for <notmuch@notmuchmail.org>;\r
+       Fri, 10 Oct 2014 02:57:27 -0700 (PDT)\r
+Received: from mail-lb0-f172.google.com (mail-lb0-f172.google.com\r
+       [209.85.217.172]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
+       (No client certificate requested)\r
+       by olra.theworths.org (Postfix) with ESMTPS id 54ABA431FBC\r
+       for <notmuch@notmuchmail.org>; Fri, 10 Oct 2014 02:57:27 -0700 (PDT)\r
+Received: by mail-lb0-f172.google.com with SMTP id b6so2770935lbj.31\r
+       for <notmuch@notmuchmail.org>; Fri, 10 Oct 2014 02:57:24 -0700 (PDT)\r
+DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113;\r
+       h=date:message-id:from:to:subject:in-reply-to:references:mime-version\r
+       :content-type:content-disposition:content-transfer-encoding;\r
+       bh=xsEj9F7mlTZYd099w6I26QdRxXI6GHupvt4ER3j4Hpg=;\r
+       b=Xy/Qbbo+KilA0XtzBzpMUcpoQoBq1Se6N15J39MKbZtPNMW7KKak/IGo7asaPvEoK9\r
+       JegiDQxqFltsupWA74drkc0dL+NAFQSLUiNa0fBz6+EbxPwuoZKOszLQO7YbshKSDR2e\r
+       kHvipIbQIU9Pm+ncD4HTW6iAH0GTv+5Rc1d2xvrJlC3Z2KoFZ9GINAJamn/0PPpN2sTT\r
+       TYjIFUOA733cAIhp2kuv8b4FBD1C94ScZpQEep+P0IHtvgcGPIaQNnqx1fYzeBaF+2MH\r
+       mz7GEZBfPVHAYsHO18kZOUmWlGj16AosyLOPLYW9QSIwHOQo7TxbOEmHxO28ybUjeOR8\r
+       Olng==\r
+X-Received: by 10.152.115.229 with SMTP id jr5mr3612680lab.7.1412935041371;\r
+       Fri, 10 Oct 2014 02:57:21 -0700 (PDT)\r
+Received: from localhost ([91.0.201.230]) by mx.google.com with ESMTPSA id\r
+       ju20sm1687807lab.14.2014.10.10.02.57.19 for <notmuch@notmuchmail.org>\r
+       (version=TLSv1.2 cipher=RC4-SHA bits=128/128);\r
+       Fri, 10 Oct 2014 02:57:20 -0700 (PDT)\r
+Date: Fri, 10 Oct 2014 11:57:17 +0200\r
+Message-ID: <20141010115717.GI28601@TP_L520.localdomain>\r
+From: Franz Fellner <alpine.art.de@gmail.com>\r
+To: notmuch@notmuchmail.org\r
+Subject: Re: [PATCH] VIM: Add better attachment support\r
+In-Reply-To: <1412236132-9624-1-git-send-email-imain@stemwinder.org>\r
+References: <1412236132-9624-1-git-send-email-imain@stemwinder.org>\r
+MIME-Version: 1.0\r
+Content-Type: text/plain; charset=utf-8\r
+Content-Disposition: inline\r
+Content-Transfer-Encoding: 8bit\r
+X-BeenThere: notmuch@notmuchmail.org\r
+X-Mailman-Version: 2.1.13\r
+Precedence: list\r
+List-Id: "Use and development of the notmuch mail system."\r
+       <notmuch.notmuchmail.org>\r
+List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
+List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
+List-Post: <mailto:notmuch@notmuchmail.org>\r
+List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
+List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
+       <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
+X-List-Received-Date: Fri, 10 Oct 2014 09:57:32 -0000\r
+\r
+It works as described. But IMHO it would be better to prompt the user to\r
+enter the location where he wants to save the attachment(s). This path\r
+could be taken if s:notmuch_attachment_tmpdir_default is set to empty.\r
+Or implement a different function for "save to".\r
+\r
+On Thu,  2 Oct 2014 00:48:52 -0700, Ian Main <imain@stemwinder.org> wrote:\r
+> This patch changes how the notmuch vim client supports attachments:\r
+> \r
+> - For each attachment an 'Attachment <number>: <filename>' is added\r
+>   to the header\r
+> - You can then use 'e' to extract the attachment under the cursor\r
+>   or use it elsewhere to extract all attachments (the prior behavior)\r
+> - You can use 'v' to 'view' the attachment using xdg-open by default.\r
+> \r
+>     Ian\r
+> ---\r
+>  vim/notmuch.txt |  8 +++++++-\r
+>  vim/notmuch.vim | 60 +++++++++++++++++++++++++++++++++++++++++++++++++++++++--\r
+>  2 files changed, 65 insertions(+), 3 deletions(-)\r
+> \r
+> diff --git a/vim/notmuch.txt b/vim/notmuch.txt\r
+> index 4374102..838a904 100644\r
+> --- a/vim/notmuch.txt\r
+> +++ b/vim/notmuch.txt\r
+> @@ -72,6 +72,9 @@ q  Quit view\r
+>  A   Archive (-inbox -unread)\r
+>  I   Mark as read (-unread)\r
+>  t   Tag (prompted)\r
+> +e       Extract attachment on the current 'Attachment' line or all\r
+> +    attachments if the cursor is elsewhere.\r
+> +v       View attachment on the current 'Attachment' line.\r
+>  s   Search\r
+>  p   Save patches\r
+>  r   Reply\r
+> @@ -148,6 +151,9 @@ You can also configure your externail mail reader and sendemail program:\r
+>  >\r
+>      let g:notmuch_reader = 'mutt -f %s'\r
+>      let g:notmuch_sendmail = 'sendmail'\r
+> -<\r
+> +\r
+> +You can also configure what probram is used to view attachments:\r
+> +\r
+> +    let g:notmuch_view_attachment = 'xdg-open'\r
+>  \r
+>  vim:tw=78:ts=8:noet:ft=help:\r
+> diff --git a/vim/notmuch.vim b/vim/notmuch.vim\r
+> index 331e930..a5830b5 100644\r
+> --- a/vim/notmuch.vim\r
+> +++ b/vim/notmuch.vim\r
+> @@ -35,6 +35,7 @@ let g:notmuch_show_maps = {\r
+>      \ 't':          'show_tag("")',\r
+>      \ 'o':          'show_open_msg()',\r
+>      \ 'e':          'show_extract_msg()',\r
+> +    \ 'v':          'show_view_attachment()',\r
+>      \ 's':          'show_save_msg()',\r
+>      \ 'p':          'show_save_patches()',\r
+>      \ 'r':          'show_reply()',\r
+> @@ -58,6 +59,8 @@ let s:notmuch_date_format_default = '%d.%m.%y'\r
+>  let s:notmuch_datetime_format_default = '%d.%m.%y %H:%M:%S'\r
+>  let s:notmuch_reader_default = 'mutt -f %s'\r
+>  let s:notmuch_sendmail_default = 'sendmail'\r
+> +let s:notmuch_view_attachment_default = 'xdg-open'\r
+> +let s:notmuch_attachment_tmpdir_default = '~/.notmuch/tmp'\r
+>  let s:notmuch_folders_count_threads_default = 0\r
+>  \r
+>  function! s:new_file_buffer(type, fname)\r
+> @@ -147,13 +150,53 @@ function! s:show_info()\r
+>      ruby vim_puts get_message.inspect\r
+>  endfunction\r
+>  \r
+> +function! s:show_view_attachment()\r
+> +    let line = getline(".")\r
+> +ruby << EOF\r
+> +    m = get_message\r
+> +    line = VIM::evaluate('line')\r
+> +\r
+> +    match = line.match(/^Attachment (\d*):/)\r
+> +    if match and match.length == 2\r
+> +            a = m.mail.attachments[match[1].to_i - 1]\r
+> +            tmpdir = VIM::evaluate('g:notmuch_attachment_tmpdir')\r
+> +            tmpdir = File.expand_path(tmpdir)\r
+> +            Dir.mkdir(tmpdir) unless Dir.exists?(tmpdir)\r
+> +            filename = File.expand_path("#{tmpdir}/#{a.filename}")\r
+> +            vim_puts "Viewing attachment #{filename}"\r
+> +            File.open(filename, 'w') do |f|\r
+> +                    f.write a.body.decoded\r
+> +                    cmd = VIM::evaluate('g:notmuch_view_attachment')\r
+> +                    system(cmd, filename)\r
+> +            end\r
+> +    else\r
+> +            vim_puts "No attachment on this line."\r
+> +    end\r
+> +EOF\r
+> +endfunction\r
+> +\r
+>  function! s:show_extract_msg()\r
+> +    let line = getline(".")\r
+>  ruby << EOF\r
+>      m = get_message\r
+> -    m.mail.attachments.each do |a|\r
+> +    line = VIM::evaluate('line')\r
+> +\r
+> +    # If the user is on a line that has an 'Attachment'\r
+> +    # line, we just extract the one attachment.\r
+> +    match = line.match(/^Attachment (\d*):/)\r
+> +    if match and match.length == 2\r
+> +            a = m.mail.attachments[match[1].to_i - 1]\r
+>              File.open(a.filename, 'w') do |f|\r
+>                      f.write a.body.decoded\r
+> -                    print "Extracted '#{a.filename}'"\r
+> +                    vim_puts "Extracted #{a.filename}"\r
+> +            end\r
+> +    else\r
+> +            # Extract them all..\r
+> +            m.mail.attachments.each do |a|\r
+> +                    File.open(a.filename, 'w') do |f|\r
+> +                            f.write a.body.decoded\r
+> +                            vim_puts "Extracted #{a.filename}"\r
+> +                    end\r
+>              end\r
+>      end\r
+>  EOF\r
+> @@ -326,6 +369,11 @@ ruby << EOF\r
+>                      b << "To: %s" % msg['to']\r
+>                      b << "Cc: %s" % msg['cc']\r
+>                      b << "Date: %s" % msg['date']\r
+> +                    cnt = 0\r
+> +                    nm_m.mail.attachments.each do |a|\r
+> +                            cnt += 1\r
+> +                            b << "Attachment %d: %s" % [cnt, a.filename]\r
+> +                    end\r
+>                      nm_m.body_start = b.count\r
+>                      b << "--- %s ---" % part.mime_type\r
+>                      part.convert.each_line do |l|\r
+> @@ -420,6 +468,14 @@ function! s:set_defaults()\r
+>              endif\r
+>      endif\r
+>  \r
+> +    if !exists('g:notmuch_attachment_tmpdir')\r
+> +            let g:notmuch_attachment_tmpdir = s:notmuch_attachment_tmpdir_default\r
+> +    endif\r
+> +\r
+> +    if !exists('g:notmuch_view_attachment')\r
+> +            let g:notmuch_view_attachment = s:notmuch_view_attachment_default\r
+> +    endif\r
+> +\r
+>      if !exists('g:notmuch_folders_count_threads')\r
+>              if exists('g:notmuch_rb_count_threads')\r
+>                      let g:notmuch_count_threads = g:notmuch_rb_count_threads\r
+> -- \r
+> 1.9.3\r
+> \r
+> _______________________________________________\r
+> notmuch mailing list\r
+> notmuch@notmuchmail.org\r
+> http://notmuchmail.org/mailman/listinfo/notmuch\r