1 |
On Thu, Jan 16, 2014 at 12:44 PM, Alexander Berntsen |
2 |
<alexander@××××××.net>wrote: |
3 |
|
4 |
> -----BEGIN PGP SIGNED MESSAGE----- |
5 |
> Hash: SHA256 |
6 |
> |
7 |
> On 16/01/14 18:24, Jesus Rivero (Neurogeek) wrote: |
8 |
> > So, how would this work with emails to this list, exactly? An |
9 |
> > email should be sent any time one of those fields is changed? |
10 |
> That's not necessary, in my opinion. We already send emails, "looks OK |
11 |
> to me" and similar. And most patches don't really need more than one |
12 |
> review and an ACK by the lead. |
13 |
> |
14 |
> > Do you have a more detailed plan on how would this work? |
15 |
> Not really. We're small enough to do this organically and on a |
16 |
> per-case basis. |
17 |
> |
18 |
> But basically, if someone authors a non-trivial patch, that person |
19 |
> should *never* push themselves. Whoever reviews it should push it, |
20 |
> adding the Reviewed-by field. The reviewer should also get an ACK by |
21 |
> the team lead (via IRC or whatever) and add that to the commit before |
22 |
> pushing. |
23 |
> |
24 |
|
25 |
Gotcha!, that makes sense to me. |
26 |
|
27 |
> |
28 |
> In a bigger project (or with a team lead with a lot of free time...), |
29 |
> I would argue that the reviewer should send the new commit, with the |
30 |
> Reviewed-by field added, to the team lead, which then adds the |
31 |
> Acked-by field themselves, before pushing. I'm not convinced the |
32 |
> benefits of this extra step outweighs the drawback in the overhead of |
33 |
> this small community of ours. |
34 |
> |
35 |
> - -- |
36 |
> Alexander |
37 |
> alexander@××××××.net |
38 |
> http://plaimi.net/~alexander |
39 |
> -----BEGIN PGP SIGNATURE----- |
40 |
> Version: GnuPG v2.0.22 (GNU/Linux) |
41 |
> Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ |
42 |
> |
43 |
> iF4EAREIAAYFAlLYGpkACgkQRtClrXBQc7WA4AEAmghIHMkNxiqJ79CONZzs/k/u |
44 |
> t0QoASddzlSruejiVaQA+QFOdbgMaA59hf9DInPAgpG7Kc6fbFENgkZn4jEY9NAq |
45 |
> =CrCK |
46 |
> -----END PGP SIGNATURE----- |
47 |
> |
48 |
> |
49 |
Thanks, |
50 |
|
51 |
-- |
52 |
Jesus Rivero (Neurogeek) |
53 |
Gentoo Developer |