[PATCH] vim: Patch Ruby Mail's depracated method

Daniel Kahn Gillmor dkg at fifthhorseman.net
Tue Jun 25 13:36:55 PDT 2019


On Mon 2019-03-18 15:10:51 +0100, Tarvi Verro wrote:
> Method is not supported in Ruby's Mail 2.8.0+.
> ---
>  vim/notmuch.vim | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/vim/notmuch.vim b/vim/notmuch.vim
> index ad8b7c80..541698cd 100644
> --- a/vim/notmuch.vim
> +++ b/vim/notmuch.vim
> @@ -666,7 +666,7 @@ ruby << EOF
>  				date = Time.at(e.newest_date).strftime(date_fmt)
>  				subject = e.messages.first['subject']
>  				if $mail_installed
> -					subject = Mail::Field.new("Subject: " + subject).to_s
> +					subject = Mail::Field.parse("Subject: " + subject).to_s
>  				else
>  					subject = subject.force_encoding('utf-8')
>  				end
> -- 

I see this patch is marked "notmuch::moreinfo", but i don't see any
request for more info specifically, so i thought i'd fill that in here.

I don't know vim or ruby, but i would expect a patch like this to
contain at least a pointer to Ruby's Mail package's documentation about
a backward-incompatible change introduced in a certain version, if not
documentation about the expected upgrade path.  Can you provide those
pointers?

Thanks for offering fixes to the notmuch-vim codebase!

              --dkg
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 227 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20190625/f2077819/attachment.sig>


More information about the notmuch mailing list