React to the removal of ISO-2022-KR and ISO-2022-CN

RESOLVED FIXED in Thunderbird 36.0

Status

MailNews Core
Internationalization
RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: hsivonen, Assigned: Magnus Melin)

Tracking

(Depends on: 1 bug)

unspecified
Thunderbird 36.0
Dependency tree / graph

Thunderbird Tracking Flags

(thunderbird36 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
Bug 997115 removed ISO-2022-CN and ISO-2022-KR from mozilla-central.

comm-central should either import the code to c-c or adjust charsetalias.properties, etc., to complete the removal on the c-c side.

Bug 7965 and bug 159865 motivate the inclusion of ISO-2022-CN and ISO-2022-CN-EXT on the basis of the old Sun CDE mail client using those encodings for sending email.

For ISO-2022-KR, bug 7964 refers to unnamed legacy clients, so the case for inclusion is even less strong.
(Assignee)

Updated

3 years ago
Blocks: 1054354

Comment 2

3 years ago
(In reply to Henri Sivonen (:hsivonen) from comment #0)

> For ISO-2022-KR, bug 7964 refers to unnamed legacy clients, so the case for
> inclusion is even less strong.

FWIW, bug 7964 comment 3 refers to a client named Netscape Communicator ...
(In reply to j.j. (mostly inactive) from comment #2)
> (In reply to Henri Sivonen (:hsivonen) from comment #0)
> 
> > For ISO-2022-KR, bug 7964 refers to unnamed legacy clients, so the case for
> > inclusion is even less strong.
> 
> FWIW, bug 7964 comment 3 refers to a client named Netscape Communicator ...

Added presumably in NS 4.0 (June 11, 1997) and removed by Netscape 4.5 (October 19, 1998).

I'm not sure how important 16 year old legacy archives are for support, particularly for dangerous (non-ASCII-safe) charsets like ISO-2022-*.
(Assignee)

Comment 4

3 years ago
Created attachment 8526006 [details] [diff] [review]
bug1025886_react_to_rem_ISO-2022-KR_and_ISO-2022-CN.patch

Sounds like we don't want to support those anymore.
Assignee: nobody → mkmelin+mozilla
Status: NEW → ASSIGNED
Attachment #8526006 - Flags: review?(Pidgeot18)
Comment on attachment 8526006 [details] [diff] [review]
bug1025886_react_to_rem_ISO-2022-KR_and_ISO-2022-CN.patch

Review of attachment 8526006 [details] [diff] [review]:
-----------------------------------------------------------------

There's another mention of iso-2022-kr in the MapiMessage.cpp file, but that list already contains charsets that we don't support, so we don't need to (and probably shouldn't) change that list.
Attachment #8526006 - Flags: review?(Pidgeot18) → review+
(Assignee)

Comment 6

3 years ago
https://hg.mozilla.org/comm-central/rev/06c5670de0c9 -> FIXED
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 36.0

Updated

3 years ago
No longer blocks: 1054354
status-thunderbird36: --- → fixed
(Assignee)

Updated

3 years ago
Depends on: 1176848
You need to log in before you can comment on or make changes to this bug.