am -i: fix "v"iew
authorJunio C Hamano <gitster@pobox.com>
Tue, 16 Feb 2016 22:46:39 +0000 (14:46 -0800)
committerJunio C Hamano <gitster@pobox.com>
Wed, 17 Feb 2016 17:52:20 +0000 (09:52 -0800)
The 'v'iew subcommand of the interactive mode of "git am -i" was
broken by the rewrite to C we did at around 2.6.0 timeframe at
7ff26832 (builtin-am: implement -i/--interactive, 2015-08-04); we
used to spawn the pager via the shell, accepting things like

PAGER='less -S'

in the environment, but the rewrite forgot and tried to directly
spawn a command whose name is the entire string.

The previous refactoring of the new helper function makes it easier
for us to do the right thing.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
builtin/am.c
index 1399c8dd880f86190534b9c9e0a5e84f48ebeceb..56cf26e8b7dd794c34c95524caaf5c86aa3252ba 100644 (file)
@@ -1740,7 +1740,7 @@ static int do_interactive(struct am_state *state)
 
                        if (!pager)
                                pager = "cat";
-                       argv_array_push(&cp.args, pager);
+                       prepare_pager_args(&cp, pager);
                        argv_array_push(&cp.args, am_path(state, "patch"));
                        run_command(&cp);
                }