Re: [PATCH v4 1/4] rebase: start implementing it as a builtin

[Duy Nguyen]

On Sun, Jul 8, 2018 at 8:03 PM Pratik Karki <predatoram...@gmail.com> wrote:
> +int cmd_rebase(int argc, const char **argv, const char *prefix)
> +{
> +       /*
> +        * NEEDSWORK: Once the builtin rebase has been tested enough
> +        * and git-legacy-rebase.sh is retired to contrib/, this preamble
> +        * can be removed.
> +        */
> +
> +       if (!use_builtin_rebase()) {
> +               const char *path = mkpath("%s/git-legacy-rebase",
> +                                         git_exec_path());
> +
> +               if (sane_execvp(path, (char **)argv) < 0)
> +                       die_errno("could not exec %s", path);

Please wrap all user visible strings in thi series in _().

> +               else
> +                       die("sane_execvp() returned???");

or if it's definitely a bug in the code, go with BUG()
-- 
Duy