Bug 967640 - ru_RU.dic contains both CRLF *and* LF line terminators
Summary: ru_RU.dic contains both CRLF *and* LF line terminators
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: hunspell-ru
Version: 19
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Caolan McNamara
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-05-27 18:07 UTC by Mike FABIAN
Modified: 2013-05-29 14:48 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-05-29 12:14:52 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Mike FABIAN 2013-05-27 18:07:10 UTC
[mfabian@Fedora-19-Beta-RC4-x86_64-netins myspell]$ file ru_RU.dic 
ru_RU.dic: ISO-8859 text, with CRLF, LF line terminators
[mfabian@Fedora-19-Beta-RC4-x86_64-netins myspell]$

Comment 1 Caolan McNamara 2013-05-28 09:48:49 UTC
Well, hunspell doesn't care whether a line ends in CRLF or LF it handles both equivalently and we presumably have a bunch of mixed line endings in a pile of dictionaries so is there any pressing reason to change them ?

Comment 2 Mike FABIAN 2013-05-29 05:13:58 UTC
I stumbled over this when working on ibus-typing-booster, but I
added a workaround for ibus-typing-booster already to handle both
line endings. So it is not urgent to change this, but it is not nice
to have this mixed line endings.

Comment 3 Caolan McNamara 2013-05-29 12:14:52 UTC
I'm inclined to mark this as wontfix/notabug and force "just sucking it down" that the dicts are allowed to have variable line terminators because seeing as hunspell accept that then the dicts are "just like that" in the wild and cleaning them up locally in fedora doesn't help the global story.

Comment 4 Mike FABIAN 2013-05-29 14:48:42 UTC
Yes, of course, it would be good to get in contact with upstream to
resolve that upstream though.


Note You need to log in before you can comment on or make changes to this bug.