[rkward-cvs] SF.net SVN: rkward:[4130] trunk/rkward

[tfry at users.sourceforge.net]

Revision: 4130
          http://rkward.svn.sourceforge.net/rkward/?rev=4130&view=rev
Author:   tfry
Date:     2011-12-16 15:44:22 +0000 (Fri, 16 Dec 2011)
Log Message:
-----------
Actually capture stderr, too.

Modified Paths:
--------------
    trunk/rkward/ChangeLog
    trunk/rkward/rkward/rbackend/rkrbackend.cpp

Modified: trunk/rkward/ChangeLog
===================================================================
--- trunk/rkward/ChangeLog	2011-12-15 08:33:22 UTC (rev 4129)
+++ trunk/rkward/ChangeLog	2011-12-16 15:44:22 UTC (rev 4130)
@@ -1,3 +1,4 @@
+- Fixed: Would not show output of system() commands to stderr on the console (on Unix-like systems)
 - Allow some markup inside <text> elements in plugins, and auto-add breaks only for duplicate newlines.
 - Reorganized t-test plugin, and add support for single sample t-tests			TODO: adjust test(s)
 - Fixed: Function argument hints for the second half of the parameter list would not be quoted, correctly

Modified: trunk/rkward/rkward/rbackend/rkrbackend.cpp
===================================================================
--- trunk/rkward/rkward/rbackend/rkrbackend.cpp	2011-12-15 08:33:22 UTC (rev 4129)
+++ trunk/rkward/rkward/rbackend/rkrbackend.cpp	2011-12-16 15:44:22 UTC (rev 4130)
@@ -1009,8 +1009,8 @@
 	// re-direct stdout / stderr to a pipe, so we can read output from system() calls
 	int pfd[2];
 	pipe (pfd);
-	dup2 (STDOUT_FILENO, STDERR_FILENO);		// single channel to avoid interleaving hell, for now.
 	dup2 (pfd[1], STDOUT_FILENO);
+	dup2 (pfd[1], STDERR_FILENO);		// forward both to a single channel to avoid interleaving hell, for now.
 	close (pfd[1]);
 	stdout_stderr_fd = pfd[0];
 #endif

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.