From: William Morgan <wmorgan-sup@masanjin.net>
To: sup-devel@rubyforge.org
Subject: Re: [sup-devel] [Heliotrope/Turnsole] How to use IMAP?
Date: Wed, 22 Feb 2012 22:25:11 -0800 [thread overview]
Message-ID: <1329977859-turnsole-79917@terminus-est> (raw)
In-Reply-To: <1329946381-sup-2255@stapelberg.de>
Excerpts from Michael Stapelberg's message of 2012-02-22 13:39:42 -0800:
> 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).
My understanding of the Maildir specification is that there is no way to
determine the order of messages besides reading the date headers.
Ordering messages correctly at import time is important because
Heliotrope only serves messages in the reverse order to which they were
imported. If there is a better solution, please feel free to educate me.
Is there any problem with the current code besides the fact that you
don't like the big scan?
> Note that adding mails via stdin doesn’t work until you change the code like this:
> https://github.com/wmorgan/heliotrope/pull/34
Merged, thank you.
--
William <wmorgan@masanjin.net>
_______________________________________________
Sup-devel mailing list
Sup-devel@rubyforge.org
http://rubyforge.org/mailman/listinfo/sup-devel
next prev parent reply other threads:[~2012-02-23 6:40 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-08 22:48 Michael Stapelberg
2012-01-10 0:40 ` William Morgan
2012-02-06 20:45 ` Michael Stapelberg
2012-02-06 23:08 ` Matthieu Rakotojaona
2012-02-06 23:46 ` Michael Stapelberg
2012-02-07 12:27 ` Matthieu Rakotojaona
2012-02-07 19:11 ` Michael Stapelberg
2012-02-07 21:54 ` Matthieu Rakotojaona
2012-02-13 6:16 ` William Morgan
2012-02-22 21:39 ` Michael Stapelberg
2012-02-23 6:25 ` William Morgan [this message]
2012-02-23 11:24 ` Michael Stapelberg
2012-02-25 6:35 ` William Morgan
2012-02-27 9:16 ` Michael Stapelberg
2012-02-29 1:11 ` William Morgan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1329977859-turnsole-79917@terminus-est \
--to=wmorgan-sup@masanjin.net \
--cc=sup-devel@rubyforge.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox