Gentoo Archives: gentoo-dev

From: "René 'Necoro' Neumann" <lists@××××××.eu>
To: gentoo-dev@l.g.o
Subject: Re: [gentoo-dev] Re: New eclass: cmake-utils.eclass
Date: Fri, 09 Nov 2007 16:15:28
Message-Id: 4734866D.20203@necoro.eu
In Reply to: Re: [gentoo-dev] Re: New eclass: cmake-utils.eclass by "Wulf C. Krueger"
1 -----BEGIN PGP SIGNED MESSAGE-----
2 Hash: SHA1
3
4 Wulf C. Krueger schrieb:
5 > On Friday, 09. November 2007 10:10:42 René 'Necoro' Neumann wrote:
6 >> But as I think, that the uppercase version is the common behavior here,
7 >> it should not need this extra "PYTHON". :) That's why the patch ;)
8 >
9 > Actually, the mixed-case is what we have encountered in most cases.
10 >
11 > Furthermore, as you stated correctly yourself, cmake is case-sensitive and
12 > a patch that works around that fact only to have one parameter less for a
13 > function doesn't really make much sense in my book.
14 >
15 Hmm ... ok - if you say, that more applications used the mixed case
16 versions, the current version is ok :)
17 I did not want to reduce one parameter, but when I first used this
18 eclass function, I assumed, that it will do the right thing (that is:
19 make it uppercase). It did not do so - that's why the patch ;).
20
21 Another way would be to enhance the comment and state explicitly that it
22 takes the useflag literally and does not do any case transition :)
23 -----BEGIN PGP SIGNATURE-----
24 Version: GnuPG v1.4.7 (GNU/Linux)
25 Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
26
27 iD8DBQFHNIZt4UOg/zhYFuARAhuNAJ97EaX5W2ffNUrtPsFLLY1ZzTQFFQCffyCE
28 mThno69KazBAWmnsifjxM8E=
29 =7Xbk
30 -----END PGP SIGNATURE-----
31 --
32 gentoo-dev@g.o mailing list

Replies

Subject Author
Re: [gentoo-dev] Re: New eclass: cmake-utils.eclass "Marijn Schouten (hkBst)" <hkBst@g.o>