Re: imap-hash.el and tramp-imap.el

[Michael Albinus]

Ted Zlatanov <address@hidden> writes:

Hi Ted,

> Attached are imap-hash.el and tramp-imap.el, I worked on them this
> weekend.

Thanks for this. I've played a little bit with them (no real test yet).

> imap-hash.el lets you treat an IMAP mailbox as a hash.  It lets you get
> individual keys' headers and body data (keys are message UIDs) or map a
> function across *all the messages in the mailbox*.  The function is only
> called for those that match a given subject, but really I should be
> using SEARCH.  So the library is not optimized for speed.  Also, it does
> not handle invalid mailbox names gracefully yet.  Consider it an alpha
> version.

Could you, please, add "(require 'assoc)"? It's not loaded by default.

> tramp-imap.el uses imap-hash.el to provide a Tramp interface to read and
> write files as messages in an IMAP mailbox.  It's also an alpha version,
> but it will at least let you (given a valid IMAP mailbox) read and write
> directory contents.  It uses EPG (thanks to Daiki Ueno's help) to
> encrypt the file contents when they are stored as messages.

As usual, I have some few patches :-)

* imaps didn't work for me any longer. I've patched
  `tramp-imap-make-iht' until it worked ... maybe my sledge-hammer patch
  could be done more elegant. There is still an error message in the
  mini-buffer when opening the IMAP folder, but it doesn't seem to hurt.

* I have renamed `tramp-imap-handle-file-inode' to
  `tramp-imap-get-file-inode'. tramp-*-handle-* function names shall be
  reserved to basic file name operations implementation.

* I have used `tramp-current-host' to solve the problem of unresolved
  vector in `tramp-imap-passphrase-callback-function'.  That variable
  was introduced in tramp.el for exactly this kind of problems.

* When you byte-compile tramp-imap.el, there is a warning about unknown
  function `tramp-imap-get-message-headers'. No idea, what to do here.

My patch is appended.

I really would like to add tramp-imap.el to the CVS repository. I test
on different machines, and the current situation is inconvenient for
me. Do you agree for a checkin?

> Ted

Best regards, Michael.

txtP3oXT11nIk.txt
Description: Text Data