Gentoo Archives: gentoo-pms

From: Ulrich Mueller <ulm@g.o>
To: gentoo-pms@l.g.o
Cc: council@g.o
Subject: [gentoo-pms] [PATCH] src-install-4: Check for empty DOCS variable.
Date: Thu, 22 Aug 2013 11:18:44
Message-Id: 21013.62348.374840.160365@a1i15.kph.uni-mainz.de
In Reply to: [gentoo-pms] Re: [PATCH] Installation commands must be called with at least one parameter. by Ulrich Mueller
1 This arises from bug 481664. There seems to be consensus that the
2 default src_install function of EAPI 4 (and 5) is flawed, because it
3 cannot handle the case where an ebuild specifies DOCS="" or DOCS=().
4 The question is whether we wait with a fix until EAPI 6, or if we
5 change it retroactively.
6
7 See https://bugs.gentoo.org/show_bug.cgi?id=481664#c29 and following
8 (in particular, comment 37) for a discussion of the issue. It looks
9 like no ebuild is currently affected by this.
10
11 Ulrich
12
13
14 From c865af98e08f721383b1ece2a6f310794662d3f6 Mon Sep 17 00:00:00 2001
15 From: =?UTF-8?q?Ulrich=20M=C3=BCller?= <ulm@g.o>
16 Date: Thu, 22 Aug 2013 12:38:02 +0200
17 Subject: [PATCH] src-install-4: Check for empty DOCS variable.
18
19 ---
20 ebuild-functions.tex | 4 ++--
21 1 file changed, 2 insertions(+), 2 deletions(-)
22
23 diff --git a/ebuild-functions.tex b/ebuild-functions.tex
24 index 84847e9..46e6f3b 100644
25 --- a/ebuild-functions.tex
26 +++ b/ebuild-functions.tex
27 @@ -318,9 +318,9 @@ src_install() {
28 [[ -s "${d}" ]] && dodoc "${d}"
29 done
30 elif declare -p DOCS | grep -q "^declare -a " ; then
31 - dodoc "${DOCS[@]}"
32 + [[ ${DOCS[@]} ]] && dodoc "${DOCS[@]}"
33 else
34 - dodoc ${DOCS}
35 + [[ ${DOCS} ]] && dodoc ${DOCS}
36 fi
37 }
38 \end{verbatim}
39 --
40 1.8.3.2

Replies