commit: bb7f724185f344a656bbaf1df19f00f8265ea76d Author: Patrice Clement <monsieurp <AT> gentoo <DOT> org> AuthorDate: Sun Nov 5 21:16:43 2017 +0000 Commit: Patrice Clement <monsieurp <AT> gentoo <DOT> org> CommitDate: Sun Nov 5 21:17:32 2017 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=bb7f7241
vim-plugin.eclass: simplify permission setting logic. We don't use Portage internal variables. Instead, we do away with that chunk of code and invoke fperms recursively on the directory with the right permission flags. Closes: https://bugs.gentoo.org/469400 eclass/vim-plugin.eclass | 17 ++--------------- 1 file changed, 2 insertions(+), 15 deletions(-) diff --git a/eclass/vim-plugin.eclass b/eclass/vim-plugin.eclass index abe9c7b3661..a0ba714f6e6 100644 --- a/eclass/vim-plugin.eclass +++ b/eclass/vim-plugin.eclass @@ -35,19 +35,6 @@ vim-plugin_src_install() { has "${EAPI:-0}" 0 1 2 && ! use prefix && ED="${D}" local f - if use !prefix && [[ ${EUID} -eq 0 ]] ; then - ebegin "Fixing file permissions" - # Make sure perms are good - chmod -R a+rX "${S}" || die "chmod failed" - find "${S}" -user 'portage' -exec chown root '{}' \; || die "chown failed" - if use userland_BSD || [[ ${CHOST} == *-darwin* ]] ; then - find "${S}" -group 'portage' -exec chgrp wheel '{}' \; || die "chgrp failed" - else - find "${S}" -group 'portage' -exec chgrp root '{}' \; || die "chgrp failed" - fi - eend $? - fi - # When globbing, if nothing exists, the shell literally returns the glob # pattern. So turn on nullglob and extglob options to avoid this. eshopts_push -s extglob @@ -85,8 +72,8 @@ vim-plugin_src_install() { mv "${S}" "${ED}"/usr/share/vim/vimfiles || die \ "couldn't move ${S} to ${ED}/usr/share/vim/vimfiles" - # Fix remaining bad permissions - chmod -R -x+X "${ED}"/usr/share/vim/vimfiles/ || die "chmod failed" + # Set permissions + fperms -R a+rX /usr/share/vim/vimfiles } # @FUNCTION: vim-plugin_pkg_postinst
