Gentoo Archives: gentoo-dev

From: "Michał Górny" <mgorny@g.o>
To: gentoo-dev@l.g.o
Cc: "Michał Górny" <mgorny@g.o>
Subject: [gentoo-dev] [PATCH eutils 2/2] Convert simple uses to dointo/newinto.
Date: Thu, 16 Aug 2012 20:21:13
Message-Id: 1345148385-27699-2-git-send-email-mgorny@gentoo.org
In Reply to: [gentoo-dev] [PATCH eutils 1/2] Add dointo && newinto. by "Michał Górny"
1 I will provide patches for more if the functions are accepted.
2 ---
3 eutils.eclass | 24 ++++++------------------
4 1 file changed, 6 insertions(+), 18 deletions(-)
5
6 diff --git a/eutils.eclass b/eutils.eclass
7 index 119fc32..f59b7ae 100644
8 --- a/eutils.eclass
9 +++ b/eutils.eclass
10 @@ -871,12 +871,8 @@ make_desktop_entry() {
11 fi
12 [[ -n ${fields} ]] && printf '%b\n' "${fields}" >> "${desktop}"
13
14 - (
15 - # wrap the env here so that the 'insinto' call
16 - # doesn't corrupt the env of the caller
17 - insinto /usr/share/applications
18 - doins "${desktop}"
19 - ) || die "installing desktop file failed"
20 + dointo /usr/share/applications "${desktop}" \
21 + || die "installing desktop file failed"
22 }
23
24 # @FUNCTION: validate_desktop_entries
25 @@ -932,12 +928,7 @@ make_session_desktop() {
26 Type=XSession
27 EOF
28
29 - (
30 - # wrap the env here so that the 'insinto' call
31 - # doesn't corrupt the env of the caller
32 - insinto /usr/share/xsessions
33 - doins "${desktop}"
34 - )
35 + dointo /usr/share/xsessions "${desktop}"
36 }
37
38 # @FUNCTION: domenu
39 @@ -973,12 +964,9 @@ domenu() {
40 # @DESCRIPTION:
41 # Like all other new* functions, install the specified menu as newname.
42 newmenu() {
43 - (
44 - # wrap the env here so that the 'insinto' call
45 - # doesn't corrupt the env of the caller
46 - insinto /usr/share/applications
47 - newins "$@"
48 - )
49 + [[ ${#} -eq 2 ]] || die 'Synopsis: newmenu <menu> <newname>'
50 +
51 + newinto /usr/share/applications "${@}"
52 }
53
54 # @FUNCTION: _iconins
55 --
56 1.7.11.1