diff options
| author | Jacob Vosmaer <contact@jacobvosmaer.nl> | 2015-01-22 15:53:16 +0100 |
|---|---|---|
| committer | Jacob Vosmaer <contact@jacobvosmaer.nl> | 2015-01-22 15:53:16 +0100 |
| commit | a63187f28b18e2feea16681b313166a982254e4e (patch) | |
| tree | e62e584be1f15c48a6da7088202286eeb8e4ee16 /doc/development/shell_commands.md | |
| parent | f937e059493037f3e18896a81693de81cf6a69a1 (diff) | |
| download | gitlab-ce-a63187f28b18e2feea16681b313166a982254e4e.tar.gz | |
Don't create zombies with IO.popen
The previous recommend incantation would leave the process we read from
hanging around, even though it had finished. That gives you a
'defunct'/'zombie' process.
Diffstat (limited to 'doc/development/shell_commands.md')
| -rw-r--r-- | doc/development/shell_commands.md | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/doc/development/shell_commands.md b/doc/development/shell_commands.md index 1e51ad73e32..42f17e19536 100644 --- a/doc/development/shell_commands.md +++ b/doc/development/shell_commands.md @@ -108,7 +108,7 @@ In other repositories, such as gitlab-shell you can also use `IO.popen`. ```ruby # Safe IO.popen example -logs = IO.popen(%W(git log), chdir: repo_dir).read +logs = IO.popen(%W(git log), chdir: repo_dir) { |p| p.read } ``` Note that unlike `Gitlab::Popen.popen`, `IO.popen` does not capture standard error. |