Gentoo Archives: gentoo-dev

From: "Michał Górny" <mgorny@g.o>
To: gentoo-dev@l.g.o
Cc: vim@g.o
Subject: Re: [gentoo-dev] [PATCH 2/3] vim-plugin.eclass: use Portage internal variables.
Date: Sat, 04 Nov 2017 23:26:46
Message-Id: 1509837993.10754.0.camel@gentoo.org
In Reply to: [gentoo-dev] [PATCH 2/3] vim-plugin.eclass: use Portage internal variables. by Patrice Clement
1 W dniu sob, 04.11.2017 o godzinie 22∶48 +0100, użytkownik Patrice
2 Clement napisał:
3 > Closes: https://bugs.gentoo.org/469400
4 > ---
5 > eclass/vim-plugin.eclass | 6 +++---
6 > 1 file changed, 3 insertions(+), 3 deletions(-)
7 >
8 > diff --git a/eclass/vim-plugin.eclass b/eclass/vim-plugin.eclass
9 > index a05cdb4520b..3c809768cf8 100644
10 > --- a/eclass/vim-plugin.eclass
11 > +++ b/eclass/vim-plugin.eclass
12 > @@ -39,11 +39,11 @@ vim-plugin_src_install() {
13 > ebegin "Fixing file permissions"
14 > # Make sure perms are good
15 > chmod -R a+rX "${S}" || die "chmod failed"
16 > - find "${S}" -user 'portage' -exec chown root '{}' \; || die "chown failed"
17 > + find "${S}" -user "${PORTAGE_USERNAME}" -exec chown root '{}' \; || die "chown failed"
18 > if use userland_BSD || [[ ${CHOST} == *-darwin* ]] ; then
19 > - find "${S}" -group 'portage' -exec chgrp wheel '{}' \; || die "chgrp failed"
20 > + find "${S}" -group "${PORTAGE_GRPNAME}" -exec chgrp wheel '{}' \; || die "chgrp failed"
21 > else
22 > - find "${S}" -group 'portage' -exec chgrp root '{}' \; || die "chgrp failed"
23 > + find "${S}" -group "${PORTAGE_GRPNAME}" -exec chgrp root '{}' \; || die "chgrp failed"
24 > fi
25 > eend $?
26 > fi
27
28 This is going to die on every non-Portage package manager.
29
30 --
31 Best regards,
32 Michał Górny

Replies