From ec35190cfc920dc816b446f1845060d819d5a7c2 Mon Sep 17 00:00:00 2001 From: Mitchell Hashimoto Date: Sun, 13 Apr 2014 14:24:05 -0700 Subject: [PATCH] core: use process.alive? to avoid exceptions --- CHANGELOG.md | 1 + lib/vagrant/util/subprocess.rb | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 43cc9eaeb..04ebeaed5 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -19,6 +19,7 @@ BUG FIXES: - core: Setting an ID on a provisioner now works. [GH-3424] - core: All synced folder paths containing symlinks are fully expanded before sharing. [GH-3444] + - core: Windows no longer sees "process not started" errors rarely. - commands/box/repackage: Works again. [GH-3372] - commands/box/update: Update should check for updates from latest version. [GH-3452] diff --git a/lib/vagrant/util/subprocess.rb b/lib/vagrant/util/subprocess.rb index db2130691..37490452f 100644 --- a/lib/vagrant/util/subprocess.rb +++ b/lib/vagrant/util/subprocess.rb @@ -204,7 +204,7 @@ module Vagrant # Return an exit status container return Result.new(process.exit_code, io_data[:stdout], io_data[:stderr]) ensure - if process && !process.exited? + if process && process.alive? # Make sure no matter what happens, the process exits process.stop(2) end