Paul Tan <pyoka...@gmail.com> 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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to