1 |
On 04/26/2010 01:42 AM, Alistair Bush wrote: |
2 |
>> On 04/24/2010 09:14 PM, Alexis Ballier wrote: |
3 |
>>> On Sat, 24 Apr 2010 20:40:54 +0300 |
4 |
>>> |
5 |
>>> Petteri Räty <betelgeuse@g.o> wrote: |
6 |
>>>> 17:34 < Betelgeuse> robbat2|na: how easy to it to prevent commits to |
7 |
>>>> CVS if the commit message doesn't match a certain pattern? |
8 |
>>>> 17:36 <@robbat2|na> go and checkout the CVSROOT and there should be an |
9 |
>>>> example there |
10 |
>>>> 17:37 < Betelgeuse> robbat2|na: Ok so doable then. Thanks. |
11 |
>>>> |
12 |
>>>> What do you think about not allowing commits to eclasses without |
13 |
>>>> mentioning an another developer who has reviewed and approved the diff |
14 |
>>>> in the commit message? There's enough people on gentoo-dev for urgent |
15 |
>>>> stuff too. |
16 |
>>> |
17 |
>>> no thanks; we already have the policy to require that major changes to |
18 |
>>> broad impact eclasses have gone through -dev, no need to add more |
19 |
>>> bureaucracy. |
20 |
>> |
21 |
>> But the policy is not tested by the quizzes and we have had cases lately |
22 |
>> where large diffs have been committed without gentoo-dev review. With |
23 |
>> peer review it's likely that the reviewer is familiar with what should |
24 |
>> be sent to gentoo-dev as hesitant/new people won't give their approval |
25 |
>> that easily. |
26 |
> |
27 |
> 1) Why is it of any relevance whether or not the quizzes test this policy? |
28 |
|
29 |
I doubt recruits read all of our documentation while answering the |
30 |
quizzes. This would just enforce behavior. |
31 |
|
32 |
> 2) Where is this policy recorded, and why does devmanual.g.o seem to |
33 |
> (possibly) contradict it? [1] I'm not sure of the nature of the commits but |
34 |
> were they non-general? |
35 |
> |
36 |
|
37 |
http://devmanual.gentoo.org/eclass-writing/index.html |
38 |
|
39 |
"Before updating eutils or a similar widely used eclass, it is best to |
40 |
email the gentoo-dev list." |
41 |
|
42 |
> |
43 |
> [1] "It is not usually necessary to email the gentoo-dev list before making |
44 |
> changes to a non-general eclass which you maintain. Use common sense here." |
45 |
|
46 |
Yeah it's not spelled that clearly there. |
47 |
|
48 |
Regards, |
49 |
Petteri |