Paul Tan <pyokagan@xxxxxxxxx> writes: > On Mon, Jun 15, 2015 at 6:08 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote: >> Paul Tan <pyokagan@xxxxxxxxx> writes: >>> 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 >> > > Yes, in 05/19, where the splitting of patches is implemented, we > prefix the mbox paths with the path to the working tree. I wasn't wondering about your new code. The scripted Porcelain is spawned after applying patches 1-3 from here, when you do not have _GIT_USE_BUILTIN_AM exported. Haven't RUN_SETUP code did its thing by that time? -- 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