Open
Bug 765926
Opened 12 years ago
Updated 2 years ago
make xpcshell and mochitests tests pass with maildir as default pluggable store
Categories
(MailNews Core :: Backend, defect)
MailNews Core
Backend
Tracking
(Not tracked)
NEW
People
(Reporter: Bienvenu, Assigned: benc)
References
(Depends on 3 open bugs, Blocks 1 open bug)
Details
Attachments
(1 file)
|
3.14 KB,
patch
|
neil
:
review+
|
Details | Diff | Splinter Review |
There are several unit test failures that show up when I run with a build that defaults pluggable stores to maildir. I'm going to put the patches that fix those issues in this bug..
|
Reporter | |
Comment 1•12 years ago
|
||
The pluggable stores set the store token when you call GetNewMsgOutputStream, so the imap mail folder object shouldn't do so. The store also sets the message offset. The imap mail folder code was basically assuming berkeley mailbox, which doesn't work with maildir. The removal of the srcKeyArray and srcMsgs is an unrelated change, but those variables aren't used.
Attachment #634212 -
Flags: review?(neil)
|
||
Updated•12 years ago
|
Attachment #634212 -
Flags: review?(neil) → review+
|
|
Reporter | |
Updated•12 years ago
|
Attachment #634212 -
Attachment description: fix maildir store token for fake headers → fix maildir store token for fake headers - checked in.
|
|
Reporter | |
Comment 2•12 years ago
|
||
maildir store token patch checked in - http://hg.mozilla.org/comm-central/rev/8c820d9d2612
|
||
Updated•12 years ago
|
OS: Windows 7 → All
Hardware: x86_64 → All
Patch has been backed out in Bug 786595 due to incorrect message body being displayed in certain circumstances.
|
||
Updated•11 years ago
|
Blocks: maildirblockers
|
|
||
Updated•11 years ago
|
|
|
||
Updated•4 years ago
|
Summary: make xpcshell and mozmill tests pass with maildir as default pluggable store → make xpcshell and mochitests tests pass with maildir as default pluggable store
|
Assignee | |
Updated•3 years ago
|
Assignee: nobody → benc
|
||
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•