Hi, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Stephan Beyer <s-beyer@xxxxxxx> writes: > > (Well, I think the > > http://thread.gmane.org/gmane.comp.version-control.git/86985 > > thread died in the last days, but I hope some responses will come and > > also that Junio's patch for cherry-picking root commits will be > > included. I try to be patient...) ;-) > > Please don't be patient but actively re-review what you sent out. > > I _really_ wanted to merge the basic bits and rewrite of "am" at least to > pu tonight, but I had to drop them after noticing that it does not seem to > handle --rebasing at all (it parses to set $rebasing but after that where > does that bit go? Yes, you are right that am --rebasing is a no-op. That option was a little mystery to me, because it seemed to do nothing special, but I'll check again (bash-completion etc) and do appropriate changes. > About the "rewrite rebase to use sequencer" bits, because we've dropped > the older rebase-i change, I do not want your series to depend on it. Yes, I made the mistake that I started developing on "next" and I did not expect that js/rebase-i-sequencer could be dropped. (I somehow expected the opposite, that it would be migrated to master some day.) Then I wanted to make it work on master, and thought this is simply done by removing the "additional features" (-f and -p option using extended todo list), but then I noticed that master also has a -p which works entirely different and I wanted to see that stuff on the list before July 1st and I had no idea what the status of js/rebase-i-sequencer is. So I just tried and sent that last patch to the list in that way. But now I have a clear statement. So I'm going to send a new patchset (also with an EXAMPLES section in the docs) to the list in a few days. Big thanks for the feedback, Stephan -- Stephan Beyer <s-beyer@xxxxxxx>, PGP 0x6EDDD207FCC5040F -- 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
