Re: [PATCH v3 0/5] better document 'interactive.singlekey' and '--patch'

From: Junio C Hamano <gitster@pobox.com>
Date: May 5, 2011, 19:16 UTC
Message-ID: <7vbozh6jrj.fsf@alter.siamese.dyndns.org>
In-Reply-To: <1304621271-17107-1-git-send-email-valentin.haenel@gmx.de>

Valentin Haenel <valentin.haenel@gmx.de> writes:

> Note that the first patch is already included in pu (93556471), but I
> included it here again since its part of the series. Is that the right
> thing to do, or could I have dropped it?

To me it does not make much of a difference either way.

I usually try to apply a rerolled series at the same fork point as the previous round, and discard the earlier ones after verifying if they are identical between the two rounds (both trees and log message wording). It becomes irritating when the changes I find in this process only reverts what I fixed-up when I queued the previous round, but on the other hand, a complete re-send will give the patch a new chance to be reviewed by different people, so...