Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 589942 - app-dicts/myspell-en-20160201 should install en_US hyphenation file
Summary: app-dicts/myspell-en-20160201 should install en_US hyphenation file
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal enhancement (vote)
Assignee: Gino McCarty
URL:
Whiteboard:
Keywords: PullRequest
Depends on:
Blocks:
 
Reported: 2016-07-28 21:50 UTC by Ryoto Yayame
Modified: 2021-06-03 14:30 UTC (History)
3 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
myspell-en-20160201.ebuild patch to install hyph_en_US.dic (myspell-en-20160201-install-en-us-hyphenation.patch,234 bytes, patch)
2016-07-28 21:50 UTC, Ryoto Yayame
Details | Diff
Add a comment about why hyph_en_US.dic is not installed (myspell-en-20160201.ebuild-comment-about-hyph_en_us.patch,399 bytes, patch)
2016-08-06 11:27 UTC, Ryoto Yayame
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Ryoto Yayame 2016-07-28 21:50:07 UTC
Created attachment 441836 [details, diff]
myspell-en-20160201.ebuild patch to install hyph_en_US.dic

app-dicts/myspell-en-20160201 installs "hyph_en_GB.dic", but not "hyph_en_US.dic".

Patch provided.

Thanks in advance.
Comment 1 Ryoto Yayame 2016-08-06 11:27:12 UTC
Created attachment 442670 [details, diff]
Add a comment about why hyph_en_US.dic is not installed

Alright, actually I had LibreOffice uninstalled during this time, and when I tried to reinstall it, it pulled dev-libs/hyphen-2.8.8, which installs the same "/usr/share/hyphen/hyph_en_US.dic" file.

I suppose this is why this file wasn't included in myspell-en-20160201.ebuild?

There should be a comment in the ebuild about this, so it is not forgotten. Patch added for this.

"hyph_en_US.dic" from hyphen-2.8.8 is 104KB, while the one from myspell-en-20160201 is 106KB. There aren't too many different lines, and they seem to come from the same source (Hunspell project).

dev-libs/hyphen includes a library though, which may not be desirable for people who don't have LibreOffice or another program using it. Plus it may be hard to find for people thinking myspell-en is all there is.

I have no idea how things are handled if both were to be installed with a different version suffix (e.g., the thesaurus files in myspell-en have a '_v2' suffix), to avoid the collision. If it's just a version number, it could be annoying to maintain on the long-term, though, if dev-libs/hyphen sometimes include a version number too, which might collide again with myspell-en.

Maybe things should be reversed, with dev-libs/hyphen not installing "hyph_en_US.dic", and depending instead on myspell-en for it, according to future L10N USE flags (neither support L10N for currently).
Comment 2 Gino McCarty 2018-11-03 17:52:24 UTC
Hey guys, I'd like some input from dev-libs/hyphen maintainers

considering adding hyph_en_US.dic back to myspell-en and this will require changes to be made to dev-libs/hyphen

PR: https://github.com/gentoo/gentoo/pull/10220
Comment 3 Gino McCarty 2019-03-19 22:21:28 UTC
have had a PR locked up for this for a long time now, hopefully I can get it merged soon
Comment 4 Larry the Git Cow gentoo-dev 2019-07-14 12:42:08 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=cd8da3ea5bd0c4668e7b9f6d85f9a19dc89d21f2

commit cd8da3ea5bd0c4668e7b9f6d85f9a19dc89d21f2
Author:     Gino <onigino@protonmail.com>
AuthorDate: 2018-11-12 07:46:12 +0000
Commit:     Michał Górny <mgorny@gentoo.org>
CommitDate: 2019-07-14 12:41:57 +0000

    dev-libs/hyphen-2.8.8: Do not install hyph_en_US.dic file
    
    Bug: https://bugs.gentoo.org/589942
    
    Signed-off-by: Gino McCarty <onigino@protonmail.com>
    Closes: https://github.com/gentoo/gentoo/pull/11820
    [mgorny: corrected not to remove docs, fixed EAPI 7]
    Signed-off-by: Michał Górny <mgorny@gentoo.org>

 dev-libs/hyphen/hyphen-2.8.8-r1.ebuild | 34 ++++++++++++++++++++++++++++++++++
 1 file changed, 34 insertions(+)