commit:     374c53ac8519cd064bbfb83d158b07ac2e238cbc
Author:     Eli Schwartz <eschwartz93 <AT> gmail <DOT> com>
AuthorDate: Sun Feb 11 19:38:24 2024 +0000
Commit:     Eli Schwartz <eschwartz <AT> gentoo <DOT> org>
CommitDate: Wed Aug 21 19:07:30 2024 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=374c53ac

app-editors/vim: sync trivial details with gvim ebuild

A lot of the logic between these packages is identical. And some of it
should be -- but because of things like variance between order of
pkg_postinst, whether or not to include "#" before a bug reference, or
capitalization in comments, the diff is larger than it has to be.

Larger diffs == harder to compare which changes are relevant. Make these
differences disappear by using consistent style between both.

It fixes style warts such as unsorted dependencies or IUSE as a bonus.

Signed-off-by: Eli Schwartz <eschwartz93 <AT> gmail.com>
Signed-off-by: Eli Schwartz <eschwartz <AT> gentoo.org>

 app-editors/vim/vim-9999.ebuild | 40 ++++++++++++++++++++++------------------
 1 file changed, 22 insertions(+), 18 deletions(-)

diff --git a/app-editors/vim/vim-9999.ebuild b/app-editors/vim/vim-9999.ebuild
index 91f87fbcb9a3..32a908dc0b81 100644
--- a/app-editors/vim/vim-9999.ebuild
+++ b/app-editors/vim/vim-9999.ebuild
@@ -13,9 +13,9 @@ PYTHON_COMPAT=( python3_{10..12} )
 PYTHON_REQ_USE="threads(+)"
 USE_RUBY="ruby31 ruby32"
 
-inherit vim-doc flag-o-matic bash-completion-r1 lua-single python-single-r1 
ruby-single toolchain-funcs desktop xdg-utils
+inherit bash-completion-r1 flag-o-matic lua-single desktop python-single-r1 
ruby-single toolchain-funcs vim-doc xdg-utils
 
-if [[ ${PV} == 9999* ]] ; then
+if [[ ${PV} == 9999* ]]; then
        inherit git-r3
        EGIT_REPO_URI="https://github.com/vim/vim.git";
 else
@@ -29,7 +29,7 @@ HOMEPAGE="https://www.vim.org https://github.com/vim/vim";
 
 LICENSE="vim"
 SLOT="0"
-IUSE="X acl crypt cscope debug gpm lua minimal nls perl python racket ruby 
selinux sound tcl terminal vim-pager"
+IUSE="acl crypt cscope debug gpm lua minimal nls perl python racket ruby 
selinux sound tcl terminal vim-pager X"
 REQUIRED_USE="
        lua? ( ${LUA_REQUIRED_USE} )
        python? ( ${PYTHON_REQUIRED_USE} )
@@ -37,18 +37,18 @@ REQUIRED_USE="
 "
 
 RDEPEND="
+       ~app-editors/vim-core-${PV}
        >=app-eselect/eselect-vi-1.1
        >=sys-libs/ncurses-5.2-r2:0=
-       nls? ( virtual/libintl )
        acl? ( kernel_linux? ( sys-apps/acl ) )
        crypt? ( dev-libs/libsodium:= )
        cscope? ( dev-util/cscope )
        gpm? ( >=sys-libs/gpm-1.19.3 )
-       lua? ( ${LUA_DEPS}
+       lua? (
+               ${LUA_DEPS}
                $(lua_gen_impl_dep 'deprecated' lua5-1)
        )
-       ~app-editors/vim-core-${PV}
-       vim-pager? ( app-editors/vim-core[-minimal] )
+       nls? ( virtual/libintl )
        perl? ( dev-lang/perl:= )
        python? ( ${PYTHON_DEPS} )
        racket? ( dev-scheme/racket )
@@ -56,6 +56,7 @@ RDEPEND="
        selinux? ( sys-libs/libselinux )
        sound? ( media-libs/libcanberra )
        tcl? ( dev-lang/tcl:0= )
+       vim-pager? ( app-editors/vim-core[-minimal] )
        X? ( x11-libs/libXt )
 "
 DEPEND="${RDEPEND}
@@ -76,7 +77,7 @@ if [[ ${PV} != 9999* ]]; then
        )
 fi
 
-# platform-specific checks (bug #898452):
+# platform-specific checks (bug #898450 #898452):
 # - acl()     -- Solaris
 # - statacl() -- AIX
 QA_CONFIG_IMPL_DECL_SKIP=(
@@ -153,7 +154,8 @@ src_prepare() {
        # (2) Rebuild auto/configure
        # (3) Notice auto/configure is newer than auto/config.mk
        # (4) Run ./configure (with wrong args) to remake auto/config.mk
-       sed -i 's# auto/config\.mk:#:#' src/Makefile || die "Makefile sed 
failed"
+       sed -i -e \
+               's# auto/config\.mk:#:#' src/Makefile || die "Makefile sed 
failed"
        rm src/auto/configure || die "rm failed"
 
        # bug 908961
@@ -177,8 +179,9 @@ src_configure() {
 
        # This should fix a sandbox violation (see bug #24447). The hvc
        # things are for ppc64, see bug #86433.
+       local file
        for file in /dev/pty/s* /dev/console /dev/hvc/* /dev/hvc*; do
-               if [[ -e "${file}" ]]; then
+               if [[ -e ${file} ]]; then
                        addwrite ${file}
                fi
        done
@@ -316,6 +319,7 @@ src_test() {
        # Hangs.
        export 
TEST_SKIP_PAT='\(Test_expand_star_star\|Test_exrc\|Test_job_tty_in_out\|Test_spelldump_bang\|Test_fuzzy_completion_env\|Test_term_mouse_multiple_clicks_to_select_mode\|Test_spelldump\)'
 
+       # Don't do additional GUI tests.
        emake -j1 -C src/testdir nongui
 }
 
@@ -355,23 +359,23 @@ src_install() {
 }
 
 pkg_postinst() {
-       # Update documentation tags (from vim-doc.eclass)
+       # update documentation tags (from vim-doc.eclass)
        update_vim_helptags
 
-       # Call eselect vi update
-       eselect_vi_update
-
        # update desktop file mime cache
        xdg_desktop_database_update
+
+       # call eselect vi update
+       eselect_vi_update
 }
 
 pkg_postrm() {
-       # Update documentation tags (from vim-doc.eclass)
+       # update documentation tags (from vim-doc.eclass)
        update_vim_helptags
 
-       # Call eselect vi update
-       eselect_vi_update
-
        # update desktop file mime cache
        xdg_desktop_database_update
+
+       # call eselect vi update
+       eselect_vi_update
 }

Reply via email to