Paul Tan <pyokagan@xxxxxxxxx> writes: > +int cmd_am(int argc, const char **argv, const char *prefix) > +{ > + if (!getenv("_GIT_USE_BUILTIN_AM")) { > + const char *path = mkpath("%s/git-am", git_exec_path()); > + > + if (sane_execvp(path, (char**) argv) < 0) Style: if (sane_execvp(path, (char**)argv) < 0) > + die_errno("could not exec %s", path); > + } > + > + return 0; > +} > diff --git a/git.c b/git.c > index 44374b1..42328ed 100644 > --- a/git.c > +++ b/git.c > @@ -370,6 +370,7 @@ static int run_builtin(struct cmd_struct *p, int argc, const char **argv) > > static struct cmd_struct commands[] = { > { "add", cmd_add, RUN_SETUP | NEED_WORK_TREE }, > + { "am", cmd_am, RUN_SETUP | NEED_WORK_TREE }, Would this, especially having RUN_SETUP, keep the same behaviour when the command is run from a subdirectory of a working tree? e.g. save messages to ./inbox $ cd sub/dir $ git am ../../inbox > { "annotate", cmd_annotate, RUN_SETUP }, > { "apply", cmd_apply, RUN_SETUP_GENTLY }, > { "archive", cmd_archive }, -- 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