On Thu, Jul 17, 2008 at 12:49 PM, Andi Kleen <andi@firstfloor.org> wrote:Because sometimes merges are non-trivial. If you roll that merge conflict resolution back into the original patch, then the patch is now different. And in all these rebasings, who's to say there won't be a typo that accidentally changes the original patch that has had more testing? We'er all human, y'know? Surely you agree that more testing is better? A rebased patch has had less testing than the original patch, by definition. Andi, no offense was intended here. I'm just asking you to walk through a thought experiment with me, okay? If you rebase, you're creating new patches that have less testing than the originals. You're also tossing away a history of the changesets, which means that any external testers can no longer point to a historical potion of a tree and say "I tested that". Maybe the merging is trivial in all the cases you've hit so far, great. What happens when it isn't? That's the larger point here. Uhm, the history is the whole point of using source control and git. If all we cared about was the end result and not the history, there'd be little point to using source management other than to help speed merging. --
| Max Krasnyansky | Re: Inquiry: Should we remove "isolcpus= kernel boot option? (may have realtime us... |
| Jeremy Allison | Re: [RFC] Heads up on sys_fallocate() |
| Randy Dunlap | Re: -mm merge plans for 2.6.23 (pcmcia) |
| Damien Wyart | ACPI power off regression in 2.6.23-rc8 (NOT in rc7) |
git: | |
| Josip Rodin | Re: bnx2_poll panicking kernel |
| Linus Torvalds | Re: [GIT]: Networking |
| Denys Fedoryshchenko | thousands of classes, e1000 TX unit hang |
