1 |
On 09/14/2010 05:50 PM, Arfrever Frehtes Taifersar Arahesis wrote: |
2 |
> 2010-09-13 00:53:19 Mark Loeser napisał(a): |
3 |
>> "Arfrever Frehtes Taifersar Arahesis (arfrever)" <arfrever@g.o> said: |
4 |
>>> arfrever 10/09/12 20:43:13 |
5 |
>>> |
6 |
>>> Removed: pygtkhelpers-0.4.1.ebuild |
7 |
>>> Log: |
8 |
>>> Delete older ebuild. |
9 |
>> |
10 |
>> Please update the Changelog when you remove versions of the package. |
11 |
> |
12 |
> When multiple commits in one package are made during e.g. 10 minutes, then it should |
13 |
> be sufficient to describe all changes in one commit in ChangeLog. Deletion of |
14 |
> pygtkhelpers-0.4.1.ebuild is mentioned in ChangeLog of dev-python/pygtkhelpers. |
15 |
> |
16 |
|
17 |
It's hard to review commits if they are spread across multiple emails. |
18 |
Why not just commit with a single repoman invocation? At least the way I |
19 |
do things is I take my CVS commit message from the ChangeLog entry and |
20 |
if it has something like "Version bump and remove old ebuild" then it |
21 |
first fine as a cvs commit message. |
22 |
|
23 |
> By the way, 2 QA members usually don't update ChangeLog when they delete old |
24 |
> ebuilds, so maybe the policy should be changed. |
25 |
> |
26 |
|
27 |
If you have a problem with a policy then start a thread to change the |
28 |
policy to see where the majority opinion lies. Jaywalking with other |
29 |
people does not make your action any more legal (assuming a country |
30 |
where its never legal). Rules should be changed/improved when they don't |
31 |
make sense any more instead of ignored (for example some countries allow |
32 |
jaywalking if there's no traffic in sight). |
33 |
|
34 |
Regards, |
35 |
Petteri |