Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 607080 - app-dicts/myspell-nl: remove NL region, don't add it
Summary: app-dicts/myspell-nl: remove NL region, don't add it
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-01-24 13:25 UTC by Erik Quaeghebeur
Modified: 2018-10-14 13:02 UTC (History)
0 users

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Erik Quaeghebeur 2017-01-24 13:25:01 UTC
In the myspell-nl ebuill, we have

	src_prepare() {
		# thesarus has ugly name
		mv th_nl_v2.dat th_nl_NL_v2.dat || die
		mv th_nl_v2.idx th_nl_NL_v2.idx || die

		# remove dutch translated license so it aint installed
		rm -rf licentie* || die
	}

Where the NL region is added to the nl language, probably in analogy with the names of the nl_NL.aff and nl_NL.dic files. However, these latter files are where the problem lies. The Dutch language (nl) is standardized across all regions where it is spoken, which includes Belgium (BE), so no region should be added. Rather, the NL region should be removed from the aff and dic file names.

I've reported this upstream <http://lists.sf.own-it.nl/pipermail/opentaal/2017-January/012204.html> and hope this will be fixed in a (hopefully) upcoming upstream version as well. N.B.: the download link is for non-thesaurus files is off-line due to website migration <http://lists.sf.own-it.nl/pipermail/opentaal/2016-December/012201.html>.
Comment 1 Michael Palimaka (kensington) gentoo-dev 2017-01-25 11:43:20 UTC
Unfortunately this package has no current maintainer - are you interested? https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers
Comment 2 Erik Quaeghebeur 2017-01-26 22:20:39 UTC
(In reply to Michael Palimaka (kensington) from comment #1)
> Unfortunately this package has no current maintainer - are you interested?
> https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers

Perhaps. I'm in contact with upstream, who are planning to prepare a new release. At that point it makes sense to update the ebuild. (I think that the current source files are still mirrored on gentoo mirrors, so there is no urgency.)
Comment 3 Larry the Git Cow gentoo-dev 2018-10-14 13:02:40 UTC
The bug has been closed via the following commit(s):

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

commit 6713fafe395a5489d3bea6880be339792e4a0519
Author:     Pacho Ramos <pacho@gentoo.org>
AuthorDate: 2018-10-14 12:46:37 +0000
Commit:     Pacho Ramos <pacho@gentoo.org>
CommitDate: 2018-10-14 13:02:26 +0000

    app-dicts/myspell-nl: Fix regions (#607080 by Erik Quaeghebeur)
    
    Closes: https://bugs.gentoo.org/607080
    Signed-off-by: Pacho Ramos <pacho@gentoo.org>
    Package-Manager: Portage-2.3.51, Repoman-2.3.11

 app-dicts/myspell-nl/myspell-nl-2.10g-r1.ebuild | 45 +++++++++++++++++++++++++
 1 file changed, 45 insertions(+)