Gentoo Archives: gentoo-dev

From: "Tomáš Chvátal" <scarabeus@g.o>
To: gentoo-dev@l.g.o
Subject: Re: [gentoo-dev] git-2: a bunch of patches to review
Date: Tue, 20 Sep 2011 20:46:56
Message-Id: CA+Nrkpd3ASNOtYjc-CRbbWtDvgPkcTaaJgJxjbNW0fGGkYd2OQ@mail.gmail.com
In Reply to: [gentoo-dev] git-2: a bunch of patches to review by "Michał Górny"
1 0001 - i had reason to put local definitions on the top, it is way
2 more readable to see right away what local vars function has, so
3 please stick to it.
4 0004 - Did you ever hear that executing another code in condition is
5 damn annoying to trace? :)
6 0007 - I placed it into the conditionals to be clear what is
7 happening, what if there will be added another if without the push...
8 0010 - 0011 - I was serious with getting crashes on some packages with
9 this approach (suprisingly i first really tried to make the eclass
10 backcompat as much as possible), did you get anyone else to ack this
11 thing? FWIW it is like fetching new packages, I can agree that
12 dowloading whole qt or libreoffice can make someone sad, but it is
13 just few megs compared to the rest of your weekly world update. -> You
14 are introducing possibility to nicely fail without any simple
15 resolution why for almost no benefit.
16
17 Cheers
18
19 Tom

Replies

Subject Author
Re: [gentoo-dev] git-2: a bunch of patches to review "Michał Górny" <mgorny@g.o>
Re: [gentoo-dev] git-2: a bunch of patches to review "Michał Górny" <mgorny@g.o>