Re: [gentoo-dev] [PATCH] freedict.eclass: rename dict.eclass, generalize

[Michał Górny]

W dniu wto, 24.04.2018 o godzinie 23∶03 -0500, użytkownik Marty E.
Plummer napisał:
> Reworked to be usable with app-dicts/dictd-* ebuilds to avoid code
> duplication
>
> You can reference this pull request to get an idea as to the usage.
>
> https://github.com/gentoo/gentoo/pull/8106
>
> Package-Manager: Portage-2.3.31, Repoman-2.3.9
> ---
>  eclass/dict.eclass     | 75 ++++++++++++++++++++++++++++++++++++++++++
>  eclass/freedict.eclass | 51 ----------------------------
>  2 files changed, 75 insertions(+), 51 deletions(-)
>  create mode 100644 eclass/dict.eclass
>  delete mode 100644 eclass/freedict.eclass
>
> diff --git a/eclass/dict.eclass b/eclass/dict.eclass
> new file mode 100644
> index 00000000000..8d523e3863e
> --- /dev/null
> +++ b/eclass/dict.eclass
> @@ -0,0 +1,75 @@
> +# Copyright 1999-2018 Gentoo Foundation
> +# Distributed under the terms of the GNU General Public License v2
> +
> +# @ECLASS: dict.eclass
> +# @MAINTAINER:
> +# maintainer-needed@g.o
> +# @AUTHOR:
> +# Original author: Seemant Kulleen
> +# @BLURB: Ease the installation of dict and freedict translation dictionaries
> +# @DESCRIPTION:
> +# This eclass exists to ease the installation of dictd and freedictd translation
> +# dictionaries.  The only variables which need to be defined in the actual
> +# ebuilds are FORLANG and TOLANG for the source and target languages,
> +# respectively, and DICTS if the package ships more than one dictionary
> +# and cannot be determined from ${PN}.
> +
> +# @ECLASS-VARIABLE: DICTS
> +# @DESCRIPTION:
> +# Array of dictionary files (foo.dict.dz and foo.index) to be installed during
> +# dict_src_install.
> +
> +# @ECLASS-VARIABLE: FORLANG
> +# @DESCRIPTION:
> +# Please see above for a description.
> +
> +# @ECLASS-VARIABLE: TOLANG
> +# @DESCRIPTION:
> +# Please see above for a description.

I don't see those two being used anywhere.

> +
> +if [[ -z ${_DICT_ECLASS} ]]; then
> +_DICT_ECLASS=1
> +
> +case ${EAPI:-0} in
> +	6) ;;
> +	*) die "${ECLASS}.eclass is banned in EAPI=${EAPI}" ;;
> +esac
> +
> +if [[ ${PN} == *freedict-* ]]; then
> +	MY_P=${PN/freedict-/}
> +	DICTS=( ${MY_P} )
> +
> +	DESCRIPTION="Freedict for language translation from ${FORLANG} to ${TOLANG}"
> +	HOMEPAGE="http://freedict.sourceforge.net/"
> +	SRC_URI="http://freedict.sourceforge.net/download/linux/${MY_P}.tar.gz"
> +elif [[ ${PN} == *dictd-* ]]; then
> +	MY_P=${PN/dictd-/}
> +	DICTS=( ${MY_P} )
> +
> +	DESCRIPTION="${MY_P} dictionary for dictd"
> +	HOMEPAGE="http://www.dict.org/"
> +fi
> +
> +LICENSE="GPL-2+"
> +SLOT="0"
> +IUSE=""
> +
> +RDEPEND="app-text/dictd"

Why the RDEPEND?  The packages seem only to install data files (without
calling any extra tools or anything), so I don't see why they need to
depend on anything.

>
> +
> +S="${WORKDIR}"
> +
> +# @FUNCTION: dict_src_install
> +# @DESCRIPTION:
> +# The freedict src_install function, which is exported
> +dict_src_install() {
> +	insinto /usr/$(get_libdir)/dict
> +	for dict in "${DICTS[@]}"; do
> +		doins ${dict}.dict.dz
> +		doins ${dict}.index
> +	done
> +	einstalldocs
> +}
> +
> +EXPORT_FUNCTIONS src_install
> +
> +fi

That said, I don't really see any gain from having this eclass, really.

-- 
Best regards,
Michał Górny