44214: <(...) substitutions shouldn't grab the terminal

ericfreese · Peter Stephenson · commit 5c165453ab1d · 2019-04-10T09:44:58.000+01:00
diff --git a/ChangeLog b/ChangeLog
@@ -1,5 +1,8 @@
 2019-04-10  Peter Stephenson  <p.stephenson@samsung.com>
 
+	* Eric Freese: 44214: Src/exec.c: <(...) substitutions shouldn't
+	grab the terminal.
+
 	* Roman Perepelitsa: 44215: Doc/Zsh/zle.yo,
 	Src/Zle/iwidgets.list, Src/Zle/zle.h, Src/Zle/zle_main.c,
 	Src/Zle/zle_thingy.c, Src/Zle/zle_utils.c, Src/Zle/zle_vi.c:
diff --git a/Src/exec.c b/Src/exec.c
@@ -4984,7 +4984,7 @@ getpipe(char *cmd, int nullexec)
 	procsubstpid = pid;
 	return pipes[!out];
     }
-    entersubsh(ESUB_PGRP, NULL);
+    entersubsh(ESUB_ASYNC|ESUB_PGRP, NULL);
     redup(pipes[out], out);
     closem(FDT_UNUSED, 0);	/* this closes pipes[!out] as well */
     cmdpush(CS_CMDSUBST);

Original file line number	Diff line number	Diff line change
`@@ -4984,7 +4984,7 @@ getpipe(char *cmd, int nullexec)`
`4984`	`4984`	`procsubstpid = pid;`
`4985`	`4985`	`return pipes[!out];`
`4986`	`4986`	`}`
`4987`		`- entersubsh(ESUB_PGRP, NULL);`
	`4987`	`+ entersubsh(ESUB_ASYNC\|ESUB_PGRP, NULL);`
`4988`	`4988`	`redup(pipes[out], out);`
`4989`	`4989`	`closem(FDT_UNUSED, 0); /* this closes pipes[!out] as well */`
`4990`	`4990`	`cmdpush(CS_CMDSUBST);`