# HG changeset patch
# User Christian Brabandt <cb@256bit.org>
# Date 1526139905 -7200
# Node ID ea0e6c71ba51acdc5f108f10c44b97fc8b231087
# Parent  d74b3df1aea3fa6d884c0bb6089a974130ff10d7
patch 8.0.1820: terminal window redirecting stdout does not show stderr

commit https://github.com/vim/vim/commit/cd8fb449d6486a1a5a09c8c098ea3a38c19e8dc5
Author: Bram Moolenaar <Bram@vim.org>
Date:   Sat May 12 17:42:42 2018 +0200

    patch 8.0.1820: terminal window redirecting stdout does not show stderr

    Problem:    Terminal window redirecting stdout does not show stderr. (Mat?o
                Zanibelli)
    Solution:   When stdout is not connected to pty_master_fd then use it for
                stderr. (closes #2903)

diff --git a/src/os_unix.c b/src/os_unix.c
--- a/src/os_unix.c
+++ b/src/os_unix.c
@@ -5645,7 +5645,12 @@ mch_job_start(char **argv, job_T *job, j
 	/* When using pty_master_fd only set it for stdout, do not duplicate it
 	 * for stderr, it only needs to be read once. */
 	int err_fd = use_out_for_err || use_file_for_err || use_null_for_err
-		      ? INVALID_FD : fd_err[0] < 0 ? INVALID_FD : fd_err[0];
+		      ? INVALID_FD
+		      : fd_err[0] >= 0
+		         ? fd_err[0]
+		         : (out_fd == pty_master_fd
+				 ? INVALID_FD
+				 : pty_master_fd);
 
 	channel_set_pipes(channel, in_fd, out_fd, err_fd);
 	channel_set_job(channel, job, options);
diff --git a/src/testdir/test_terminal.vim b/src/testdir/test_terminal.vim
--- a/src/testdir/test_terminal.vim
+++ b/src/testdir/test_terminal.vim
@@ -1484,3 +1484,25 @@ func Test_terminal_termwinkey()
   call feedkeys("\<C-L>\<C-C>", 'tx')
   call WaitForAssert({-> assert_equal("dead", job_status(job))})
 endfunc
+
+func Test_terminal_out_err()
+  if !has('unix')
+    return
+  endif
+  call writefile([
+	\ '#!/bin/sh',
+	\ 'echo "this is standard error" >&2',
+	\ 'echo "this is standard out" >&1',
+	\ ], 'Xechoerrout.sh')
+  call setfperm('Xechoerrout.sh', 'rwxrwx---')
+
+  let outfile = 'Xtermstdout'
+  let buf = term_start(['./Xechoerrout.sh'], {'out_io': 'file', 'out_name': outfile})
+  call WaitForAssert({-> assert_inrange(1, 2, len(readfile(outfile)))})
+  call assert_equal("this is standard out", readfile(outfile)[0])
+  call assert_equal('this is standard error', term_getline(buf, 1))
+
+  exe buf . 'bwipe'
+  call delete('Xechoerrout.sh')
+  call delete(outfile)
+endfunc
diff --git a/src/version.c b/src/version.c
--- a/src/version.c
+++ b/src/version.c
@@ -762,6 +762,8 @@ static char *(features[]) =
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    1820,
+/**/
     1819,
 /**/
     1818,