The maildir specification says the following about unique names:

    A unique name can be anything that doesn't contain a colon (or
    slash) and doesn't start with a dot.

    [http://cr.yp.to/proto/maildir.html]

So disallowing a unique name to have a comma breaks maildir import on
systems where there is a comma in the names.
---

Argh! What's worse than Forgotten Attachment Syndrome (FAS),
(especially on my first post to a list)?

[For what it's worth, I *had* attached the patch to the message I had
composed in sup before it crashed. So copying the draft into mutt
defeated my two-rule technique for avoiding FAS:

  1. Always write in the past tense, ("I've attached", never "I will
     attach").

  2. Never type a lie.

That usually does the trick, but did fail in this case. ;-)

 lib/sup/maildir.rb |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/lib/sup/maildir.rb b/lib/sup/maildir.rb
index c6577c1..a2dbae4 100644
--- a/lib/sup/maildir.rb
+++ b/lib/sup/maildir.rb
@@ -212,7 +212,7 @@ private
 
   def maildir_data msg
     fn = File.basename @ids_to_fns[msg]
-    fn =~ %r{^([^:,]+):([12]),([DFPRST]*)$}
+    fn =~ %r{^([^:]+):([12]),([DFPRST]*)$}
     [($1 || fn), ($2 || "2"), ($3 || "")]
   end
 
-- 
1.6.3.3

Attachment: signature.asc
Description: Digital signature

_______________________________________________
sup-talk mailing list
[email protected]
http://rubyforge.org/mailman/listinfo/sup-talk

Reply via email to