On Dec 3, 2010, at 2:00 AM, Jonathan Nieder wrote: >> But most importantly, since $rest is echoed on the terminal, it MUST be >> derived from the recipe ($line). Rationale: I replace the commit subject >> in the recipe by a reminder what I intend to do when the "edit" command >> stops---I don't care so much what the commit subject is. > > Kevin, this sounds like a vote for the "replace commit message" output > format. The v3 patch will emit both a description of the commit it stopped on, as well as the comment. The rationale for extracting the first line of HEAD is for when the user doesn't provide any comment - e.g. they just add "edit". It may be worth doing this only in that case, and if the user did provide a comment, emit it in place of the first line of HEAD. Given the recipe pick bc17bb7 git-rebase--interactive.sh: extend "edit" command to be more useful edit - foo the edit command would print Stopped at bc17bb7... git-rebase--interactive.sh: extend "edit" command to be more useful foo You can amend the commit now... The alternative is to make that same recipe emit Stopped at bc17bb7... foo You can amend the commit now... I'm leaning towards making that change right now, but I'm not certain. Do either of you have a preference? -Kevin Ballard-- 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