> On 7 Apr 2022, at 13:01, Anna Vyalkova <cyber+gen...@sysrq.in> wrote: > > From: Thomas Bracht Laumann Jespersen <t...@laumann.xyz> > > * Drop EAPI 0, 1, 2 workarounds > * Move EXPORT_FUNCTIONS to end of file > * Add required @USAGE on functions > * Add _VIM_PLUGIN_ECLASS guard > > Bug: https://bugs.gentoo.org/830867 > Bug: https://bugs.gentoo.org/830866 > Signed-off-by: Thomas Bracht Laumann Jespersen <t...@laumann.xyz> > Signed-off-by: Anna Vyalkova <cyber+gen...@sysrq.in> > --- > eclass/vim-plugin.eclass | 28 +++++++++++++++++----------- > 1 file changed, 17 insertions(+), 11 deletions(-) > > diff --git a/eclass/vim-plugin.eclass b/eclass/vim-plugin.eclass > index 50e727e98f..0ee4ebe374 100644 > --- a/eclass/vim-plugin.eclass > +++ b/eclass/vim-plugin.eclass > @@ -1,10 +1,10 @@ > -# Copyright 1999-2021 Gentoo Authors > +# Copyright 1999-2022 Gentoo Authors > # Distributed under the terms of the GNU General Public License v2 > > # @ECLASS: vim-plugin.eclass > # @MAINTAINER: > # v...@gentoo.org > -# @SUPPORTED_EAPIS: 6 7 > +# @SUPPORTED_EAPIS: 6 7 8 > # @BLURB: used for installing vim plugins > # @DESCRIPTION: > # This eclass simplifies installation of app-vim plugins into > @@ -13,12 +13,13 @@ > # documentation, for which we make a special case via vim-doc.eclass. > > case ${EAPI} in > - 6|7);; > - *) die "EAPI ${EAPI:-0} unsupported (too old)";; > + 6|7|8);; > + *) die "${ECLASS}: EAPI ${EAPI:-0} unsupported (too old)";; > esac > > +if [[ ! ${_VIM_PLUGIN_ECLASS} ]]; then > + > inherit vim-doc > -EXPORT_FUNCTIONS src_install pkg_postinst pkg_postrm > > VIM_PLUGIN_VIM_VERSION="${VIM_PLUGIN_VIM_VERSION:-7.3}" > > @@ -32,13 +33,13 @@ fi > SLOT="0" > > # @FUNCTION: vim-plugin_src_install > +# @USAGE: > # @DESCRIPTION: > # Overrides the default src_install phase. In order, this function: > # * fixes file permission across all files in ${S}. > # * installs help and documentation files. > # * installs all files in "${ED}"/usr/share/vim/vimfiles. > vim-plugin_src_install() { > - has "${EAPI:-0}" 0 1 2 && ! use prefix && ED="${D}" > > # Install non-vim-help-docs > einstalldocs > @@ -53,6 +54,7 @@ vim-plugin_src_install() { > } >
Drop the now-blank line at start of function. > # @FUNCTION: vim-plugin_pkg_postinst > +# @USAGE: > # @DESCRIPTION: > # Overrides the pkg_postinst phase for this eclass. > # The following functions are called: > @@ -71,7 +73,6 @@ vim-plugin_pkg_postinst() { > # This function calls the update_vim_helptags and update_vim_afterscripts > # functions and eventually removes a bunch of empty directories. > vim-plugin_pkg_postrm() { > - has "${EAPI:-0}" 0 1 2 && ! use prefix && EPREFIX= > update_vim_helptags # from vim-doc > update_vim_afterscripts # see below > I'd fix the indentation (i.e. don't, just use a space) on the comment, but not a big deal. > @@ -82,25 +83,24 @@ vim-plugin_pkg_postrm() { > } > > # @FUNCTION: update_vim_afterscripts > +# @USAGE: > # @DESCRIPTION: > # Creates scripts in /usr/share/vim/vimfiles/after/* > # comprised of the snippets in /usr/share/vim/vimfiles/after/*/*.d > update_vim_afterscripts() { > - has "${EAPI:-0}" 0 1 2 && ! use prefix && EROOT="${ROOT}" > - has "${EAPI:-0}" 0 1 2 && ! use prefix && EPREFIX= > local d f afterdir="${EROOT}"/usr/share/vim/vimfiles/after > > # Nothing to do if the dir isn't there > [ -d "${afterdir}" ] || return 0 Bash tests. > > - einfo "Updating scripts in ${EPREFIX}/usr/share/vim/vimfiles/after" > + einfo "Updating scripts in ${afterdir}" > find "${afterdir}" -type d -name \*.vim.d | while read d; do > echo '" Generated by update_vim_afterscripts' > "${d%.d}" || die > find "${d}" -name \*.vim -type f -maxdepth 1 -print0 | sort -z > | \ > xargs -0 cat >> "${d%.d}" || die > "update_vim_afterscripts failed" > done > > - einfo "Removing dead scripts in ${EPREFIX}/usr/share/vim/vimfiles/after" > + einfo "Removing dead scripts in ${afterdir}" > find "${afterdir}" -type f -name \*.vim | \ > while read f; do > [[ "$(head -n 1 ${f})" == '" Generated by > update_vim_afterscripts' ]] \ > @@ -115,6 +115,7 @@ update_vim_afterscripts() { > } > > # @FUNCTION: display_vim_plugin_help > +# @USAGE: > # @DESCRIPTION: > # Displays a message with the plugin's help file if one is available. Uses the > # VIM_PLUGIN_HELPFILES env var. If multiple help files are available, they > @@ -160,3 +161,8 @@ display_vim_plugin_help() { > fi > fi > } > + > +_VIM_PLUGIN_ECLASS=1 > +fi > + > +EXPORT_FUNCTIONS src_install pkg_postinst pkg_postrm > -- > 2.35.1 > >
signature.asc
Description: Message signed with OpenPGP