Hi, On Thu, 7 Jun 2007, Junio C Hamano wrote: > Johannes Schindelin <Johannes.Schindelin@xxxxxx> writes: > > > When commit-filter echoes just "skip", just skip that commit by mapping > > its object name to the same (possibly rewritten) object name(s) its > > parent(s) are mapped to. > > > > IOW, given A-B-C, if commit-filter says "skip" upon B, the rewritten > > branch will look like this: A'-C'. > > > > Signed-off-by: Johannes Schindelin <johannes.schindelin@xxxxxx> > > --- > > > > Of course, if you think of "patchsets", this behaviour might > > be unexpected, since the children will still contain everything > > which was changed in the skipped revisions, and not changed in > > _them_. > > I think that is fine; in effect, by saying "skip" B, you are > squashing B-C into C'. > > Does this mean that, given > > C---D---E > / / > A---B > > and if commit-filter says "skip" on D, the written history would > look like this? > > C'------E' > / / > A'--B'--' > > The new commit E' would become an evil merge that has difference > between D and E in the original history? > > I am not objecting; just trying to get a mental picture. Yeah, but you called it "squash" instead of "skip". So, maybe it should accept "squash" to do that operation instead? Ciao, Dscho - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html