|| ||Linus Torvalds <torvalds-AT-linux-foundation.org>|
|| ||Andi Kleen <andi-AT-firstfloor.org>|
|| ||Re: Please pull ACPI updates|
|| ||Thu, 17 Jul 2008 08:03:26 -0700 (PDT)|
|| ||Jesse Barnes <jbarnes-AT-virtuousgeek.org>,
"Rafael J. Wysocki" <rjw-AT-sisk.pl>,
Linux Kernel Mailing List <linux-kernel-AT-vger.kernel.org>,
On Thu, 17 Jul 2008, Andi Kleen wrote:
> My plan was to keep everything in quilt and just regenerate for the pull.
> Please let me know if it's now not allowed anymore to use quilt.
End-point developers can use quilt all thei like.
But people cannot and *MUST NOT* destroy other peoples work with quilt,
nor make it harder for people to share commits.
Len had apparently left a nice topic tree for you. You took that work, and
then destroyed it. And yes, it is noticeable: Jesse had shared some of the
work from Len by pulling one of the branches (the 'suspend' branch), and
then you literally re-wrote _public_ history, so now tohose patches are
Now, duplicate patches happen, and that's not a huge technical issue (most
of the time it all merges cleanly, and it was just three patches this
time), but if you're going to rebase stuff, you're basically making it
impossible for people to share work with you. They can never rely on your
git trees, because your git threes are all throw-away.
Btw, being throw-away also means that they get essentially no testing.
They get rewritten each time, so even if you expose them to something like
linux-next and they get testing there, when you rebase them, the end
result is something *different*, and a lot of the test coverage goes away.
See the discussion from May about the x86 tree. I don't have it online in
my archives any more, but google finds
which has at least a big part of the discussion in one readable page.
So no, there's nothing wrong with a quilt interface. But there _is_
something wrong with a maintainer that makes it harder for other people
(in this case Jesse) to work with him, because he destroys history.
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to firstname.lastname@example.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
to post comments)