Re: [sup-devel] [Heliotrope/Turnsole] How to use IMAP?

From: Michael Stapelberg <michael+sup@stapelberg.de>
To: sup-devel <sup-devel@rubyforge.org>
Subject: Re: [sup-devel] [Heliotrope/Turnsole] How to use IMAP?
Date: Wed, 22 Feb 2012 22:39:42 +0100	[thread overview]
Message-ID: <1329946381-sup-2255@stapelberg.de> (raw)
In-Reply-To: <1329113568-sup-8177@typhon>

Hi William,

Thanks for adding code for this. I got the chance to test this today, and I’m
afraid this doesn’t seem to be working as I would expect.

As far as I can tell from glancing at the source and the state file, this code
examines the date stored in every mail, sorts that, then checks where it
stopped before, and continues from there. I think this is a horrible way of
doing things for two reasons:

1) It involves opening every single email on disk.
2) Date specifications in emails are not to be trusted (think spam).

For the time being, I let procmail save my messages in my IMAP server and in
heliotrope with a little script, which I include here for others to see:

#!/bin/bash
# Delivers mail via dovecot and heliotrope

# load RVM
. /home/michael/.rvm/scripts/rvm

TMPFILE=$(mktemp)

cat > $TMPFILE

cat $TMPFILE | sudo -u mail -- /usr/lib/dovecot/deliver -d michael
cat $TMPFILE | ruby -I/home/michael/heliotrope/lib /home/michael/heliotrope/bin/heliotrope-add >>/tmp/last-mail.stdout 2>>/tmp/last-mail.stderr
rm $TMPFILE

After ensuring that it works, you should get rid of the stdout/stderr redirects
or redirect them properly to a place with logfile rotation, of course.

Note that adding mails via stdin doesn’t work until you change the code like this:
https://github.com/wmorgan/heliotrope/pull/34

Best regards,
Michael
_______________________________________________
Sup-devel mailing list
Sup-devel@rubyforge.org
http://rubyforge.org/mailman/listinfo/sup-devel