Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rmail to Mail: initial work #61

Closed

Conversation

gauteh
Copy link
Member

@gauteh gauteh commented May 20, 2013

This is a work in progress to get the train moving for RMail to Mail migration, there are lots of bits around that are RMail specific which needs to be updated. This PR also includes several other fixes which should be split out and merged first:

  • syck to psych
  • remove iconv
  • fix_encoding in util.rb (without iconv it works kind of like Decoder in heliotrope)
  • using heliotrope style safe_id (MD5 of message id and refs) in stead of current message-id
  • fixes to_enum errors
  • use with_coder for loading/storing some sources from yaml

I have based the branch on the maildir-syncback branch with updated develop stuff merged in.

I got roughly 25k mails indexed and setup on this one, indexing is about half the speed of RMail at the time. I also started collecting problematic emails so that they can be kept for regression testing (I still need to anonymize them though)

Supports ruby 2.0, probably breaks 1.8.

implementation is based on suggestions by @rakoo in #22.

Related issues: #22, #23, #17.

gauteh and others added 30 commits April 13, 2013 14:28
Add tool to convert YAML objects in configs from syck to psych.
When sending a message, the account selector was ignoring
the account set by the before-edit hook.  Fix this
by running the hook before setting up the account selector.
gauteh and others added 27 commits July 23, 2013 11:55
commit ecdda4e
Author: Gaute Hope <eg@gaute.vetsj.com>
Date:   Mon Jul 29 09:27:24 2013 +0200

    refactor

commit 93b362a
Author: Gaute Hope <eg@gaute.vetsj.com>
Date:   Mon Jul 29 09:22:52 2013 +0200

    Update source.rb documentation comments
Rubygem source does no longer accepts connections via HTTP and we must use
HTTPS instead.
@gauteh gauteh closed this Aug 12, 2013
@gauteh
Copy link
Member Author

gauteh commented Aug 12, 2013

Skipped safe_id/safe_refs for now and pushed the state-of-art stuff to the use-mail branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants