Skip to content

Commit

Permalink
Merge pull request teampoltergeist#329 from Ajedi32/fix_windows_pgrou…
Browse files Browse the repository at this point in the history
…p_error

Fix process spawn pgroup error on Windows
  • Loading branch information
jonleighton committed Jun 8, 2013
2 parents 7dee775 + 14f6fdc commit ef695f1
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 1 deletion.
2 changes: 2 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -334,6 +334,8 @@ Include as much information as possible. For example:

* Fix `within_window` finding window after close/open
(Ryan Schlesinger) [Issue #312]
* Fix "wrong exec option symbol: pgroup" error on windows (Andrew Meyer)
[Issue #314]

### 1.3.0 ###

Expand Down
6 changes: 5 additions & 1 deletion lib/capybara/poltergeist/client.rb
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,11 @@ def start
}

redirect_stdout(write) do
@pid = Process.spawn(*command.map(&:to_s), pgroup: true)
if Capybara::Poltergeist.windows?
@pid = Process.spawn(*command.map(&:to_s))
else
@pid = Process.spawn(*command.map(&:to_s), pgroup: true)
end
end
end

Expand Down

0 comments on commit ef695f1

Please sign in to comment.