Abdelrazak Younes <[EMAIL PROTECTED]> writes:
| [EMAIL PROTECTED] wrote:
| > On Sun, 12 Aug 2007, Jean-Marc Lasgouttes wrote:
| >
| >> Abdelrazak Younes <[EMAIL PROTECTED]> writes:
| >>
| >>> Lars, as always, I simply cannot understand you and vice-versa. So
| >>> please, work as you like and let me work as I like. I won't listen to
| >>> you in any case...
| >>
| >> Hello there,
| >>
| >> I am always impressed to see how you manage to speak past each other
| >> :) But if I may intervene, I'd say that there is a fairly strong
| >> consensus here in Bromarv that a branch is not done to be dumped on
| >> the trunk.
| > My impression as well. I'd go further and say that committing things
| > without previous review[*] is _not_ appreciated. As I understand it
| > (which may well be wrong of course), patches allow others to review
| > the changes before committing. If a single patch is big, it's much
| > more difficult to look at it :-(.
|
| For fuck sake, the commit logs are there to look at! Sorry Christian,
| it's not against you but I've already gone through the lengthy process
| of proving myself to Lars and others. I am simply not going to do it
| again.
Do you really think that this is about you having to prove
yourself?
It is about having all developers on the same page, understanding the
changes done in the lyx source and agreeing that this is the best
approach. A large un-split, patch are hard to grasp and
understand, and makes a decent review practically impossible,
especially when there are little information about the goal of the
patch-set.
And I have looked at the commit logs... did not tell me much.
There are quite a few small things there as well that has nothing to
do with mvc at all...
--
Lgb
