From 7228ca9bf651d9f06395419752139817511aabe1 Mon Sep 17 00:00:00 2001 From: Sebastian Thiel Date: Tue, 24 May 2016 17:52:55 +0200 Subject: fix(cmd): fix with_stdout implementation Admittedly this fix is solely based on the documentation provided for this parameter, which indicated a different intend than was actually implemented. Also I don't believe doing this will cause any harm. As a special note: the call to `open(os.devnull, 'wb')` does not seem leak the handle, apparently it is given as-is to the subprocess, which will then close it naturally. This was tested using an interactive session via `htop` on osx. Fixes #437 --- doc/source/changes.rst | 2 ++ 1 file changed, 2 insertions(+) (limited to 'doc/source/changes.rst') diff --git a/doc/source/changes.rst b/doc/source/changes.rst index 8cc9ca0d..b14df90a 100644 --- a/doc/source/changes.rst +++ b/doc/source/changes.rst @@ -7,6 +7,8 @@ Changelog * Fix: progress handler exceptions are not caught anymore, which would usually just hide bugs previously. +* Fix: The `Git.execute` method will now redirect `stdout` to `devnull` if `with_stdout` is false, + which is the intended behaviour based on the parameter's documentation. 2.0.2 - Fixes ============= -- cgit v1.2.3