Hi Junio, On Sun, 15 Jan 2017, Junio C Hamano wrote: > * js/prepare-sequencer-more (2017-01-09) 38 commits I think that it adds confusion rather than value to specifically use a different branch name than I indicated in my submission, unless there is a really good reason to do so (which I fail to see here). > - sequencer (rebase -i): write out the final message > - sequencer (rebase -i): write the progress into files > - sequencer (rebase -i): show the progress > - sequencer (rebase -i): suggest --edit-todo upon unknown command > - sequencer (rebase -i): show only failed cherry-picks' output > - sequencer (rebase -i): show only failed `git commit`'s output > - sequencer: use run_command() directly > - sequencer: make reading author-script more elegant > - sequencer (rebase -i): differentiate between comments and 'noop' > - sequencer (rebase -i): implement the 'drop' command > - sequencer (rebase -i): allow rescheduling commands > - sequencer (rebase -i): respect strategy/strategy_opts settings > - sequencer (rebase -i): respect the rebase.autostash setting > - sequencer (rebase -i): run the post-rewrite hook, if needed > - sequencer (rebase -i): record interrupted commits in rewritten, too > - sequencer (rebase -i): copy commit notes at end > - sequencer (rebase -i): set the reflog message consistently > - sequencer (rebase -i): refactor setting the reflog message > - sequencer (rebase -i): allow fast-forwarding for edit/reword > - sequencer (rebase -i): implement the 'reword' command > - sequencer (rebase -i): leave a patch upon error > - sequencer (rebase -i): update refs after a successful rebase > - sequencer (rebase -i): the todo can be empty when continuing > - sequencer (rebase -i): skip some revert/cherry-pick specific code path > - sequencer (rebase -i): remove CHERRY_PICK_HEAD when no longer needed > - sequencer (rebase -i): allow continuing with staged changes > - sequencer (rebase -i): write an author-script file > - sequencer (rebase -i): implement the short commands > - sequencer (rebase -i): add support for the 'fixup' and 'squash' commands > - sequencer (rebase -i): write the 'done' file > - sequencer (rebase -i): learn about the 'verbose' mode > - sequencer (rebase -i): implement the 'exec' command > - sequencer (rebase -i): implement the 'edit' command > - sequencer (rebase -i): implement the 'noop' command > - sequencer: support a new action: 'interactive rebase' > - sequencer: use a helper to find the commit message > - sequencer: move "else" keyword onto the same line as preceding brace > - sequencer: avoid unnecessary curly braces > > The sequencer has further been extended in preparation to act as a > back-end for "rebase -i". > > Waiting for review comments to be addressed. The only outstanding review comments I know about are your objection to the name of the read_env_script() function (which I shot down as bogus), and the rather real bug fix I sent out as a fixup! which you may want to squash in (in the alternative, I can mailbomb v4 of the entire sequencer-i patch series, that is your choice). Ciao, Johannes