diff options
author | Hans de Graaff <graaff@gentoo.org> | 2019-04-29 20:15:44 +0200 |
---|---|---|
committer | Hans de Graaff <graaff@gentoo.org> | 2019-04-29 20:33:35 +0200 |
commit | 657fe39493c136eb4cc66121f4b15cd6adbf84cb (patch) | |
tree | c1d448368ba3ca707316093798e3e01227996bf9 /www-servers/puma/files | |
parent | dev-ruby/selenium-webdriver: add ruby26 (diff) | |
download | gentoo-657fe39493c136eb4cc66121f4b15cd6adbf84cb.tar.gz gentoo-657fe39493c136eb4cc66121f4b15cd6adbf84cb.tar.bz2 gentoo-657fe39493c136eb4cc66121f4b15cd6adbf84cb.zip |
www-servers/puma: add ruby26
Backport upstream patch to fix compatibility with ruby 2.6.
Signed-off-by: Hans de Graaff <graaff@gentoo.org>
Package-Manager: Portage-2.3.62, Repoman-2.3.11
Diffstat (limited to 'www-servers/puma/files')
-rw-r--r-- | www-servers/puma/files/puma-3.12.1-ruby26-waitpid.patch | 47 |
1 files changed, 47 insertions, 0 deletions
diff --git a/www-servers/puma/files/puma-3.12.1-ruby26-waitpid.patch b/www-servers/puma/files/puma-3.12.1-ruby26-waitpid.patch new file mode 100644 index 000000000000..b28c69fa4d4b --- /dev/null +++ b/www-servers/puma/files/puma-3.12.1-ruby26-waitpid.patch @@ -0,0 +1,47 @@ +From b94c3e34faff024a5b1930af36e4d64bd6dde57f Mon Sep 17 00:00:00 2001 +From: MSP-Greg <MSP-Greg@users.noreply.github.com> +Date: Fri, 15 Mar 2019 17:26:20 -0500 +Subject: [PATCH] Puma::Cluster#stop_workers - use WNOHANG with nil return + tests + +Ruby 2.6 introduced a bug that affects worker shutdown (waitpid). + +Added code using Process::WNOHANG along with needed logic. Adds worker status (via $?) and total shutdown time to log. + +Co-authored-by: MSP-Greg <greg.mpls@gmail.com> +Co-authored-by: guilleiguaran <guilleiguaran@gmail.com> +--- + lib/puma/cluster.rb | 20 +++++++++++++++++++- + 1 file changed, 19 insertions(+), 1 deletion(-) + +diff --git a/lib/puma/cluster.rb b/lib/puma/cluster.rb +index 93d65131..0879c96c 100644 +--- a/lib/puma/cluster.rb ++++ b/lib/puma/cluster.rb +@@ -37,7 +37,25 @@ def stop_workers + @workers.each { |x| x.term } + + begin +- @workers.each { |w| Process.waitpid(w.pid) } ++ if RUBY_VERSION < '2.6' ++ @workers.each { |w| Process.waitpid(w.pid) } ++ else ++ # below code is for a bug in Ruby 2.6+, above waitpid call hangs ++ t_st = Process.clock_gettime(Process::CLOCK_MONOTONIC) ++ pids = @workers.map(&:pid) ++ loop do ++ pids.reject! do |w_pid| ++ if Process.waitpid(w_pid, Process::WNOHANG) ++ log " worker status: #{$?}" ++ true ++ end ++ end ++ break if pids.empty? ++ sleep 0.5 ++ end ++ t_end = Process.clock_gettime(Process::CLOCK_MONOTONIC) ++ log format(" worker shutdown time: %6.2f", t_end - t_st) ++ end + rescue Interrupt + log "! Cancelled waiting for workers" + end |