Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 607080

Summary: app-dicts/myspell-nl: remove NL region, don't add it
Product: Gentoo Linux Reporter: Erik Quaeghebeur <gentoo>
Component: Current packagesAssignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it <maintainer-needed>
Status: RESOLVED FIXED    
Severity: normal    
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---

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(+)