Closed
Bug 1061086
Opened 10 years ago
Closed 8 years ago
A unit test for importing Eudora mail messages
Categories
(MailNews Core :: Import, defect)
MailNews Core
Import
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: hiro, Unassigned)
References
Details
Attachments
(1 file)
5.79 KB,
patch
|
standard8
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Reporter | ||
Comment 1•10 years ago
|
||
try server results: https://tbpl.mozilla.org/?tree=Thunderbird-Try&rev=5983325be682
Assignee: nobody → hiikezoe
Attachment #8482109 -
Flags: review?(standard8)
Updated•10 years ago
|
Attachment #8482109 -
Flags: review?(standard8) → review+
This looks like it is done. I'll check for bitrot and then let it be checked in.
Status: NEW → ASSIGNED
Comment 3•9 years ago
|
||
given bug 917961, perhaps we need even more tests?
Comment 4•9 years ago
|
||
(In reply to :aceman from comment #2) > This looks like it is done. I'll check for bitrot and then let it be checked > in.
Assignee: hiikezoe → nobody
Status: ASSIGNED → NEW
Flags: needinfo?(acelists)
Whiteboard: [patchlove]
I checked and this attachment still applies. I can't test if it works, we could do a try run. Would it make sense to apply it to TB aurora/beta/release where the Eudora importer exists? If the importer is removed in bug 1243498, the patch is not needed on trunk.
Flags: needinfo?(acelists)
Whiteboard: [patchlove]
Comment 6•8 years ago
|
||
We won't fix this, since we will remove the Eudora importer in bug 1243498. Besides, this patch is highly insufficient. Eudora stores embedded images and attachments in special folders and without testing some of that, the test is really incomplete.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•