Gentoo Archives: gentoo-dev

From: Michael Palimaka <kensington@g.o>
To: gentoo-dev@l.g.o
Subject: [gentoo-dev] Re: [PATCH 07/15] cmake-utils.eclass: replace replace comment_add_subdirectory with a namespaced version
Date: Fri, 22 Jan 2016 09:45:20
Message-Id: n7st69$rgp$2@ger.gmane.org
In Reply to: Re: [gentoo-dev] [PATCH 07/15] cmake-utils.eclass: replace replace comment_add_subdirectory with a namespaced version by "Michał Górny"
1 On 01/22/2016 03:29 AM, Michał Górny wrote:
2 > Dnia 20 stycznia 2016 11:43:05 CET, Michael Palimaka <kensington@g.o> napisał(a):
3 >> ---
4 >> eclass/cmake-utils.eclass | 15 +++++++++++++--
5 >> 1 file changed, 13 insertions(+), 2 deletions(-)
6 >>
7 >> diff --git a/eclass/cmake-utils.eclass b/eclass/cmake-utils.eclass
8 >> index 1de863f..e8b24bd 100644
9 >> --- a/eclass/cmake-utils.eclass
10 >> +++ b/eclass/cmake-utils.eclass
11 >> @@ -250,11 +250,11 @@ _generator_to_use() {
12 >> echo ${generator_name}
13 >> }
14 >>
15 >> -# @FUNCTION: comment_add_subdirectory
16 >> +# @FUNCTION: cmake_comment_add_subdirectory
17 >> # @USAGE: <subdirectory>
18 >> # @DESCRIPTION:
19 >> # Comment out an add_subdirectory call in CMakeLists.txt in the current
20 >> directory
21 >> -comment_add_subdirectory() {
22 >> +cmake_comment_add_subdirectory() {
23 >> if [[ -z ${1} ]]; then
24 >> die "comment_add_subdirectory must be passed the directory name to
25 >> comment"
26 >> fi
27 >> @@ -265,6 +265,17 @@ comment_add_subdirectory() {
28 >> fi
29 >> }
30 >>
31 >> +# @FUNCTION: comment_add_subdirectory
32 >> +# @USAGE: <subdirectory>
33 >> +# @DESCRIPTION:
34 >> +# Comment out an add_subdirectory call in CMakeLists.txt in the
35 >> current directory
36 >> +# Banned in EAPI 6 and later - use cmake_comment_add_subdirectory
37 >> instead.
38 >> +comment_add_subdirectory() {
39 >> + has "${EAPI:-0}" 2 3 4 5 || die "comment_add_subdirectory is banned
40 >> in EAPI 6 and later - use cmake_comment_add_subdirectory instead"
41 >> +
42 >> + cmake_comment_add_subdirectory "$@"
43 >> +}
44 >> +
45 >> # @FUNCTION: cmake-utils_use_with
46 >> # @USAGE: <USE flag> [flag name]
47 >> # @DESCRIPTION:
48 >
49 > Enough to 'replace' once :-P (commit message).
50 >
51
52 Thanks, fixed.