Closed
Bug 8181
Opened 25 years ago
Closed 25 years ago
[PP] Windows line breaks cause problem for Mac 5.0 mail viewing
Categories
(MailNews Core :: Backend, defect, P3)
Tracking
(Not tracked)
VERIFIED
WORKSFORME
M13
People
(Reporter: momoi, Assigned: rhp)
References
()
Details
Attachments
(2 files)
** Observed with 6/14/99 Mac build **
The above URL contains I18n smoketest file containing 5 msgs,
Latin 1, Japanese and UTF8 messages.
This was created by a Windows client.
Mac messenger 5.0 has a problem with Windows line breaks
apparently and I am not able to display Message pane header and
body correctly -- for the header, I see nothing for most of the
messages and for the body I see RFC 822 text type display.
The view pane display does work.
Reporter | ||
Comment 1•25 years ago
|
||
When I converted the smoketest file into Mac type line breaks,
all the Message headers and body text showed correctly.
Reporter | ||
Updated•25 years ago
|
OS: Windows NT → Mac System 8.5
Hardware: PC → Macintosh
Reporter | ||
Comment 2•25 years ago
|
||
Corrected the paltform & OS to Mac.
Reporter | ||
Comment 3•25 years ago
|
||
I want to add that one of the 5 smoketest msgs is mail with
an attachment. I was never able to load this message when I selected
it -- it kept on trying to load over and over again and it actually
froze the apprunner. There seems to be some sort of infinite
attempt at loading this particular message. When I changed the
line breaks type to the Mac one, I did not have this problem.
Updated•25 years ago
|
Assignee: phil → bienvenu
Comment 4•25 years ago
|
||
The message parser is supposed to be smart enough to accept any linebreak style.
If we've broken that, we should fix it. Reassigning to bienvenu.
Updated•25 years ago
|
Target Milestone: M8
Comment 5•25 years ago
|
||
No, this didn't work in 4.x either, I'm afraid. But there weren't any infinite
loops, AFAIK.
Comment 6•25 years ago
|
||
This was working before (I used the smoke test data to test my fix cross
platform on 6/9).
I don't see the infinite loop but it takes long time to process a message in the
smoke test which has an attachment.
Comment 7•25 years ago
|
||
I haven't changed the message parser in months. Is this a libmime problem?
Summary: Windows line breaks cause problem for Mac 5.0 mail viewing → [PP] Windows line breaks cause problem for Mac 5.0 mail viewing
Comment 8•25 years ago
|
||
cc'ing Rich - if this is really a problem with message display, as opposed to
database header parsing, he might have an idea.
Comment 9•25 years ago
|
||
I need some input from someone who has seen this problem - is it strictly a
message display problem, or is it a db header parsing problem? Does anyone need
more clarification about the difference?
Assignee | ||
Comment 10•25 years ago
|
||
I'm not sure on this one. Sounds like a possible problem with the smoketest
file on the Mac? These files need to be platform specific don't they? I don't
see where libmime would loop forever. It will hit the end of the buffer and
quit at that point.
Reporter | ||
Comment 11•25 years ago
|
||
About the mesage which causes attempts to load over and over again, it turns out to be the same message
which prompted me to file Bug 8899 for Windows client.
Assignee | ||
Comment 12•25 years ago
|
||
Oh, loading over and over was the problem we had with auto detection for
charset stuff. Could this be it?
- rhp
Comment 13•25 years ago
|
||
Comment 14•25 years ago
|
||
Could you try looking at the file tempMessage.eml in your temp directory
after loading a message and see if that looks OK, please? If that looks OK, then
the problem has nothing to do with the db header parsing.
Comment 15•25 years ago
|
||
Comment 16•25 years ago
|
||
This looks like the right message to me, which is all the DB can help with. I'm
not sure who's supposed to do the line conversions (i.e., the code that writes
out the message to the temp file, or the code that parses the message from the
temp file) but I'm not involved in either of those pieces of code. Anyone know
what the right thing to do is? Can someone run the version of 5.0 that actually
worked and see what kind of temp file was generated?
Reporter | ||
Comment 17•25 years ago
|
||
There is one differece I noticed when I compared the "tempMessage.eml"
generated for the Mac-type linebreak message as opposed to
Win-type linebreak for the same message. The .eml file for the latter
contained Unix-type "LF" rather than the Mac-type LF.
Reporter | ||
Comment 18•25 years ago
|
||
I didn't mean to write "Mac-type LF". It should have been "Mac-type CR"
rather.
Comment 19•25 years ago
|
||
thanks for the clarification, but I would expect the windows lines to end with
CRLF, not just LF.
Reporter | ||
Comment 20•25 years ago
|
||
The editor I'm using on my Mac can detect all 3 types of line breaks.
This is what my editor tells me:
Smoketest file for Mac-type: CR
.eml file generated from this: CR
Smoketest file for Win-tyep: CR+LF
.eml file generated from this: LF
The Linebreak detection on my editor is pretty accurate and
so from the Win-type Smoketest file, we are indeed generating
LF-type .eml file
Updated•25 years ago
|
Target Milestone: M8 → M9
Comment 21•25 years ago
|
||
I don't think this is my bug, but in any case, I'm not going to get to it by m8
Updated•25 years ago
|
Assignee: bienvenu → rhp
Comment 22•25 years ago
|
||
Sorry, Rich. I don't know who's bug this is, or if it really is a bug, but I'm
not doing the bug any favors by leaving it on my list. It could be that libmime
needs to deal with non-native line termination.
Assignee | ||
Updated•25 years ago
|
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → DUPLICATE
Assignee | ||
Comment 23•25 years ago
|
||
Comment 24•25 years ago
|
||
Kat - since you filed this, I'll let you verify/agree that it's a duplicate of
the other bug (which was actually reported later than this one :-)
Reporter | ||
Comment 25•25 years ago
|
||
From the description of 9231 alone, I'm not sure if this bug is a
duplicate of that one. Since manifestations of a backend fix can be
extensive, I'm inclined to monitor 9231's fix status and check on
this one when that one gets fixed. I'll hold off verifying this bug until
9231 is fixed.
Reporter | ||
Updated•25 years ago
|
Status: RESOLVED → REOPENED
Reporter | ||
Updated•25 years ago
|
Resolution: DUPLICATE → ---
Reporter | ||
Comment 26•25 years ago
|
||
Reporter | ||
Comment 27•25 years ago
|
||
*** Bug 10328 has been marked as a duplicate of this bug. ***
Assignee | ||
Updated•25 years ago
|
Target Milestone: M9 → M10
Assignee | ||
Comment 28•25 years ago
|
||
Hello all,
I have no clue on this one...nothing with respect to parsing line breaks has
changed with libmime so I can't see how this "worked" and then stopped working
at some point.
mscott:
Could something have changed with how we write out the tempMessage.eml file
along the way?
davidb:
This didn't work with 4.5, correct? I have a feeling that we were making it
work in the tempMessage.eml creation and that may have changed and now it
doesn't work.
Comments welcome!
In either case, this is getting bumped.
- rhp
Comment 29•25 years ago
|
||
I don't believe it worked in 4.x. Is anybody claiming that it did?
Assignee | ||
Comment 30•25 years ago
|
||
They are claiming it worked in 5.0, but since libmime parsing is the 4.x code
base, it is relavent.
- rhp
Comment 31•25 years ago
|
||
I don't think it's very relevant that it worked in 5.0 - that may just have been
a bug, right? What's really relevant is whether libmime handled non-native line
breaks in 4.x. If it hasn't, I can't see a lot of reason to try to change it
when we have so many other things to do to get to parity with 4.5
Assignee | ||
Updated•25 years ago
|
Status: REOPENED → ASSIGNED
Target Milestone: M10 → M12
Assignee | ||
Comment 32•25 years ago
|
||
I go back to my original comment posted 06/28/99. Isn't this a restriction
that we placed on these files? If this file would break 4.x, then I'm not going
out of my way to fix this one. Sorry, but I (like everyone else in the world)
has a billion things to do to get anywhere close to the 4.x feature set.
If I try to do something here, it will be post B1.
- rhp
Reporter | ||
Comment 33•25 years ago
|
||
I concede that the same problem existed on Mac under 4.x --
though Windows client has no problem reading Mac mailboxes.
I think it would be nice if direct interchange of mailbox files
between different platforms can be done under 5.0. That's why
I filed this bug.
More important to me would be that our migration/import mailbox
utility be able to handle linebreaks when migration occurs from
one platform type to another.
If this can be accomplished during 5.0, that would be wonderful.
Comment 34•25 years ago
|
||
(target milestone is M11 or M12 - add to mail beta tracking bug)
Assignee | ||
Updated•25 years ago
|
Target Milestone: M12 → M13
Assignee | ||
Comment 35•25 years ago
|
||
Just tweaking the milestone.
- rhp
Assignee | ||
Updated•25 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 25 years ago → 25 years ago
Resolution: --- → WORKSFORME
Assignee | ||
Comment 36•25 years ago
|
||
I'm sure a LOT has changed since this was originally posted (and it was
probably fixed along the way), but I copied over my PC formatted smoketest file
to the mac and everything seemed to work fine.
- rhp
Reporter | ||
Updated•25 years ago
|
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 37•25 years ago
|
||
** Checked with 12/17/99 Mac PPC M12 build **
With the above build, I don't have any problem displaying msgs
in the PC version of Smoketest file which contains both CR+LF.
Mac had this problem throughout 4.x cycles and so this is indeed
a very good change.
Marking it verified/fixed.
Updated•20 years ago
|
Product: MailNews → Core
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•