Show stderr in test output about tester HOT 11 CLOSED

from tester.

It should be cross-platform solution

On what platform it does not work?

from tester.

No way! It works for Windows too! 😃 Well, I'm shut up. I don't see a reason why not to do that in this way.

from tester.

Sent pull request #89

from tester.

We were discussing it with @dg and we agreed that stderr should be captured by Job separately. I tried to implement it, but there is really a lot of proc_open()'s bugs espacially on Win.

The main problem is with stderr as pipe and large output. Bugs #65650, #51800, #64438, #60120. For some I found workaround, but for some not on PHP 5.3.3. The bug behavior is that the process get stuck when reading even single byte from stdout or stderr.

One solution is use php://temp for stderr, but it is not the way since temp file is opened and discarted with every Job. But it works.

Next solution is use it and just pray for stderr shorter 2048 bytes ;-)

The safest way is close the stderr immediately after proc_open() as it is done now.

The next is use proposed 2>&1, mix the stdout with stderr and somehow parse valid HTTP headers.

I have no more ideas...

from tester.

Btw. @MartinMystikJonas When is the stderr hidden? If I use file_put_contents('php://stderr', 'test'); in tests now, I get E_WARNING: file_put_contents(): Only 0 of 4 bytes written, possibly out of free disk space because stderr is closed.

from tester.

@milo Did you manage to solve this issus on Windows or did you give up?

I have one idea how to solve this. It's not strictly clean but if we need workaround bug I think it is acceptable.

My idea is to output stderr to custom temp file (similarly to lock file used in Environment::lock) and then just read this file.

What do you think? Could this solution work?

from tester.

@MartinMystikJonas I didn't solve this, din't give up yet.

My idea is to output stderr to custom temp file

That's I wrote above with the php://temp. Problem is, this temporary file is created by every job, even stderr is not used.

Maybe solution is just mix stdin and stdout together as proposed. Who cares about wrong HTTP headers parsing. If is header needed by test, it just fails.

from tester.

Well what about use just 2> stderr-uniqid? File will be created only if there is some output

from tester.

@MartinMystikJonas Not so on Linux. But interesting idea...

from tester.

@milo My bad. I did not try it :-)

from tester.