1 |
On Tue, Sep 18, 2012 at 12:11 PM, Ulrich Mueller <ulm@g.o> wrote: |
2 |
>>>>>> On Tue, 18 Sep 2012, Brian Harring wrote: |
3 |
> |
4 |
>>> > from diffball (under current EAPIs) |
5 |
>>> |
6 |
>>> > """ |
7 |
>>> > RDEPEND=">=sys-libs/zlib-1.1.4 |
8 |
>>> > >=app-arch/bzip2-1.0.2 |
9 |
>>> > app-arch/xz-utils" |
10 |
>>> > DEPEND="${RDEPEND} |
11 |
>>> > virtual/pkgconfig" |
12 |
>>> > """ |
13 |
>>> |
14 |
>>> > becomes the following under the proposal: |
15 |
>>> |
16 |
>>> > """ |
17 |
>>> > DEPENDENCIES=">=sys-libs/zlib-1.1.4 |
18 |
>>> > >=app-arch/bzip2-1.0.2 |
19 |
>>> > app-arch/xz-utils" |
20 |
>>> > dep:build? ( virtual/pkgconfig )" |
21 |
>>> > """ |
22 |
>>> |
23 |
>>> Which is longer than the original. ;-) |
24 |
> |
25 |
>> I see 5 lines in the first version, and 4 in the second. I also see |
26 |
>> either someone who counted wrong, or basing that statement purely on |
27 |
>> byte count (which is frankly arguing to argue on your part). |
28 |
> |
29 |
> Can we agree that both counting of lines and characters is silly? ;-) |
30 |
> My point was that the new syntax isn't significantly more compact than |
31 |
> the present one. In one case there is another variable assignment, |
32 |
> in the other case you need an additional "dep:build? ( |
33 |
> virtual/pkgconfig )" group. |
34 |
> |
35 |
> Readability is more important, and there I still don't buy the |
36 |
> argument that the new syntax is better, and that any gain would |
37 |
> outweigh the cost of changing. After all, the existing variables for |
38 |
> dependency specification won't disappear, so devs would have to |
39 |
> remember both. |
40 |
|
41 |
I agree it is a con, but is it a blocker? I mean basically any change |
42 |
proposed requires know the old way, and the new way..that is how |
43 |
changes work... |
44 |
|
45 |
> |
46 |
> Ulrich |
47 |
> |