| 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 |
Archives