On 6 February 2011 19:54, Alberto Mardegan <[email protected]> wrote: > Hi Mohammad,
> The main problem with the patch is that it's big (and if I may say so, > also a bit messy). When Matan contributed some code to my project, I > asked him to rewrite the patches till they were of my liking, and he > was kind enough to do that. If he hadn't done that, I would have > either discarded the patches (if I was not interested in them) or > rewritten them myself. So this is more a personal style discussion than an actual "let's get things done" discussion. In the end, it doesn't really matter how many commits there are (more specifically, having 200 commit for 200 lines of change doesn't add any readability when it comes to repository maintenance, and we'll soon see complaints that merges should also include some form of rebasing in order to limit the amount of "noise" generated by overly trigger-happy committers. I fully support the idea of proper, apt and useful commit messages, but complaining that there are too few commits in a patch is just plain nonsense. git-blame allows to track what happened in a very useful manner. Whether that means being able to pinpoint commit index + 20 or commit index + 30 doesn't *really* impact a lot; most people won't take the time, or can't make sense of how people commit anyway. It also doesn't hamper git-bisect in any way -- well, it only means that you have to understand what is happening in code in order to find a bug, but frankly, if you don't, you shouldn't be reviewing code anyway. FYI: I'm using git at work, and often receive +3k line commits. It doesn't mean the coders or sloppy, it's just "one chunk of code that does one feature". I would like to see a coding style proposal, or a "how to use git effectively", or something of the like. I think it would make working with others a lot easier. My 2c, -S. -- question = ( to ) ? be : ! be; -- Wm. Shakespeare _______________________________________________ maemo-community mailing list [email protected] https://lists.maemo.org/mailman/listinfo/maemo-community
