Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stderr of job is redirected to stdout #89

Closed

Conversation

MartinMystikJonas
Copy link
Contributor

In test output show both output of stdout and stderr not only stdout because some errors are written to stderr.

@MartinMystikJonas
Copy link
Contributor Author

Previously discussed in #86

@milo
Copy link
Member

milo commented Feb 12, 2014

@MartinMystikJonas I discovered that it is not so simple. With CGI and code:

echo "STDOUT\n";
file_put_contents('php://stderr', "STDERR\n");
echo "STDOUT\n";
file_put_contents('php://stderr', "STDERR\n");

you get

STDERR
STDERR
X-Powered-By: PHP/5.4.4-14+deb7u7
Content-type: text/html

STDOUT
STDOUT

And it is hard to split to stdout and stderr.

@JanTvrdik
Copy link
Contributor

@milo Can't you disable PHP output buffers?

@MartinMystikJonas
Copy link
Contributor Author

@milo: Even if you can not disable output buffer I think it is still better to have all messages in "wrong order" than some error messages missing.

@dg
Copy link
Member

dg commented Jan 8, 2015

Have you tried it on Windows? I think it doesn't work. 2>&1 works from command line, but not from proc_open.

@JanTvrdik
Copy link
Contributor

Would setting bypass_shell to FALSE help?

@dg
Copy link
Member

dg commented Jan 8, 2015

Yes, it helps. But bypass_shell must be enabled…

dg added a commit to dg/nette-tester that referenced this pull request Jan 8, 2015
dg added a commit to dg/nette-tester that referenced this pull request Jan 8, 2015
dg added a commit to dg/nette-tester that referenced this pull request Jan 8, 2015
dg added a commit to dg/nette-tester that referenced this pull request Jan 8, 2015
dg added a commit to dg/nette-tester that referenced this pull request Jan 27, 2015
dg added a commit to dg/nette-tester that referenced this pull request Feb 10, 2015
dg added a commit to dg/nette-tester that referenced this pull request Mar 18, 2015
dg added a commit to dg/nette-tester that referenced this pull request May 28, 2015
dg added a commit to dg/nette-tester that referenced this pull request May 28, 2015
dg added a commit to dg/nette-tester that referenced this pull request Jun 19, 2015
dg added a commit to dg/nette-tester that referenced this pull request Jun 19, 2015
dg added a commit to dg/nette-tester that referenced this pull request Jul 13, 2015
@dg dg closed this in #193 Jul 13, 2015
dg added a commit that referenced this pull request Jul 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants