On Nov 4, 2010, at 2:23 AM, Ævar Arnfjörð Bjarmason wrote: > On Thu, Nov 4, 2010 at 09:53, Kevin Ballard <kevin@xxxxxx> wrote: >> On Nov 4, 2010, at 1:42 AM, Matthieu Moy wrote: >> >>> Kevin Ballard <kevin@xxxxxx> writes: >>> >>>> Add a new command "shell", which takes an option commit. It simply exits >>>> to the shell with the commit (if given) and a message telling the user how >>>> to resume the rebase. >>> >>> "shell" sounds like you're going to execute something in a shell, not >>> that you're going back to the shell. Looking at the commit message, I >>> thought you had missed the "exec" command and re-implemented it. >>> >>> What about "pause", abbreviated as "p" for the command name? >> >> That sounds like a reasonable suggestion, except "p" is already taken by "pick". >> I suppose this command could simply omit the short version. > > I thought "shell" would do exactly what your patch does. And it has > the "s" short version. > > So +1 for "shell" from me and -1 for "pause", which *does* confuse me. > I'd expect that > to just sleep for a few seconds. "s" is actually taken by "squash". That's why my original patch used "!", though a user might actually expect "!" to do what "x" does. -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