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

[dmd-cxx] Fix Issue 17844 - std.process.execute should allow not capturing stderr #7984

Merged
merged 8 commits into from
Apr 19, 2021

Conversation

ibuclaw
Copy link
Member

@ibuclaw ibuclaw commented Apr 19, 2021

From #5742

@dlang-bot
Copy link
Contributor

Thanks for your pull request, @ibuclaw!

Bugzilla references

Auto-close Bugzilla Severity Description
17844 enhancement std.process.execute should allow not capturing stderr

Testing this PR locally

If you don't have a local development environment setup, you can use Digger to test this PR:

dub run digger -- build "dmd-cxx + phobos#7984"

@ibuclaw
Copy link
Member Author

ibuclaw commented Apr 19, 2021

FYI, requested here: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98494

std/process.d Outdated Show resolved Hide resolved
@ibuclaw
Copy link
Member Author

ibuclaw commented Apr 19, 2021

RC is imminent, so getting this in now. :-)

@ibuclaw
Copy link
Member Author

ibuclaw commented Apr 19, 2021

Auto-merge toggled on

@ibuclaw ibuclaw merged commit e6907ff into dlang:dmd-cxx Apr 19, 2021
@ibuclaw ibuclaw deleted the dmd-cxx-passthru branch April 19, 2021 16:49
nstester pushed a commit to nstester/gcc that referenced this pull request Apr 19, 2021
Phobos changes:

 - Synchronize C bindings with the latest port fixes in upstream
   druntime.

 - Add Config.stderrPassThrough to std.process (PR98494).

Reviewed-on: dlang/druntime#3448
	     dlang/phobos#7984

libphobos/ChangeLog:

	PR d/98494
	* libdruntime/MERGE: Merge upstream druntime 89f870b7.
	* src/MERGE: Merge upstream phobos e6907ff3e.
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.

5 participants