ruby-list

なかだです。

At Fri, 29 Jun 2001 23:42:37 +0900,
matz@ruby-lang.org (Yukihiro Matsumoto) wrote:
> 結局pidを取り出せればよいので、open3の返すIOのpidメソッドを
> 置き換えて、プロセスのpidを返すようにしてやれば良いかなと。
> でも、そうするとwaitを強要することになりますねえ。
> 
> 結局、取り出せないままの方が良いのかなって気になってきました。

(1) status を得るためには自動的に wait されてはいけない
(2) ゾンビを残さないためには必ず wait しなければならない

という条件があるわけですが、pid を取り出せるだけだと前者しか満
たしません。finalizer を使っても、今までのように終了したら即座
に取り除かれるのではなくて、無期限にゾンビが残ることはない、と
いうところで妥協が必要ですが。

  [ruby-list:30417]はやっぱりダメダメでした。最初の close で
wait してしまうと、場合によって(stdin を読まないプログラムの
stdin を close したときとか)デッドロックしてしまいます。やっぱ
り自動的に wait するなら全部 close したあとでないとまずいようで
す。つまり基本形としては[ruby-list:30388]ということになります。
ただ最後の close で暗黙のうちに wait っていうのはすごく分かりづ
らい気がして来ましたので、

>   * pipeに特異メソッドを付加

これと finalizer を使ったパターンのパッチです。今のところこれが
一番マシに思えます。でなければ、

>   * 現状のまま(ステータスは無視)

と、ステータスを得られるが wait が必要なものと二つに分けるか。

# ところで 1.6 の open3.rb は動かなくなってますが。


Index: lib/open3.rb
===================================================================
RCS file: /cvs/ruby/src/ruby/lib/open3.rb,v
retrieving revision 1.7
diff -u -2 -p -w -r1.7 open3.rb
--- lib/open3.rb	2001/05/16 09:05:54	1.7
+++ lib/open3.rb	2001/06/30 16:15:53
@@ -17,6 +17,4 @@ module Open3
     pid = fork{
       # child
-      fork{
-	# grandchild
 	pw[1].close
 	STDIN.reopen(pw[0])
@@ -33,12 +31,11 @@ module Open3
 	exec(*cmd)
       }
-      exit!
-    }
 
     pw[0].close
     pr[1].close
     pe[1].close
-    Process.waitpid(pid)
+    get_pid = proc {pid}
     pi = [pw[1], pr[0], pe[0]]
+    pi.each {|p| class << p; self; end.module_eval {define_method(:pid, get_pid)}}
     if defined? yield
       begin
@@ -46,9 +43,32 @@ module Open3
       ensure
 	pi.each{|p| p.close unless p.closed?}
+	begin Process.waitpid(pid); rescue Errno::ECHILD; end
       end
     end
+    Sweeper.new(pi, pid)
     pi
   end
   module_function :popen3
+
+  class Sweeper < Hash
+    def initialize(deps, arg)
+      super()
+      finalizer = method(:finalize).to_proc
+      ObjectSpace.define_finalizer(self, finalizer)
+      deps.each do |obj| 
+	self[obj.__id__] = arg
+	ObjectSpace.define_finalizer(obj, finalizer)
+      end
+    end
+
+    private
+
+    def finalize(id)
+      id = delete(id) {return}
+      empty? or return
+      ObjectSpace.undefine_finalizer(self)
+      Process.waitpid(id)
+    end
+  end
 end
 


-- 
--- 僕の前にBugはない。
--- 僕の後ろにBugはできる。
    中田 伸悦

Thread

Prev Next

In This Thread

Prev Next

[#29986] ruby 1.6.4 to be released — matz@... (Yukihiro Matsumoto)

[#29987] ライセンス論点整理 — Tacos <ozaki@...>

[#30018] FSF 案（先後してごめん） — Tacos <ozaki@...>

[#30021] シュリンクラップ契約について（米判例） — Tacos <ozaki@...>

[#30023] 「オープンソースのつどい 2001 in 名大」開催のお知らせ — Matsuura Hirosato <matswra@...>

[#30025] ライセンス／基礎編（笑） — Tacos <ozaki@...>

[#30026] ライセンス／論点整理メモ ver0.91 — Tacos <ozaki@...>

[#30030] Ruby license discussion — Masayuki Hatta <mhatta@...>

[#30032] ｍ（。− _ −。）ｍス・スイマセーン — 熊谷のAkira@... <aan82580@...>

[#30041] Shellwords — "Akinori MUSHA" <knu@...>

[#30048] RTtool and RDtool — Takahiro Kambe <taca@...>

[#30067] [Q]drb で提供されている object のメソッド一覧を得るには ? — eda@...

[#30080] module#private — "Shin'ya Adzumi" <adzumi@...>

[#30092] Ruby 1.6.4 is released — matz@... (Yukihiro Matsumoto)

[#30093] Ruby manual license — Kazuhiro NISHIYAMA <zn@...>

[#30096] グラフ処理エンジン — "Shigeo Kobayashi" <shigeo@...>

[#30097] グラフ処理エンジン — "Shigeo Kobayashi" <shigeo@...>

[#30098] Drag and Drop on Windows — nobu.nakada@...

[#30101] 「パイプ」がわかりません。 — "伊藤 壮一" <soichi-i@...>

[#30108] 製品に組み入れた場合のライセンスは ? — "D.Miyawaki" <aaf18900@...06.odn.ne.jp>

[#30110] extmk.rb doesn't take care of LDFLAGS — YANAGAWA Kazuhisa <kjana@...>

[#30111] みなさんありがとうございます。 — "伊藤 壮一" <soichi-i@...>

[#30112] Ann: NETRuby (NOTRuby for .NET) pre alpha — arton <arton@...>

[#30114] 補足 & 訂正 — "D.Miyawaki" <aaf18900@...06.odn.ne.jp>

[#30115] Oracle 使用時のエラーについて — ISIHARA HIROSI <ishihara-h@...>

[#30117] xmlparser について（初心者） — Makoto Sato <makotosato@...>

[#30120] Oracle7 用拡張モジュール +Oracle 7.2(Solaris) — SAITO Shukaku <shukaku@...>

[#30121] Re: Oracle 使用時のエラーについて — ISIHARA HIROSI <ishihara-h@...>

[#30122] [Q]"2001 年 01 月 02 日 03 時 05 分 " を "2001 年 1 月 2 日 3 時 5 分 " のようにするには ? — eda@...

[#30129] ライセンス FACT SHEET ver0.95 — Tacos <ozaki@...>

[#30132] ライセンスＱ＆Ａ — Tacos <ozaki@...>

[#30141] 数字の 3 桁区切り — Makoto Sato <makotosato@...>

[#30150] TkLabel.new のブロック — Masatoshi SEKI <m_seki@...>

[#30155] How do we treat local static variables? — IKEGAMI Daisuke <daisu-ik@...>

[#30157] 「訴えられらくない」をまじめに考える。 — Tacos <ozaki@...>

[#30163] FW: [arg1:119] RE: [RubyUnit:4203] Re: インストーラ・プロジェクト — "KANEMITSU Masao" <masao-k@...>

[#30183] Eruby erb および本について — "Inoue" <inoue@...>

[#30188] Net::HTTP の new にパラメタを追加したい — matsui@...

[#30190] 呼び出し元のクラスを知る方法？ — "Shin'ya Adzumi" <adzumi@...>

[#30198] Ruby/Xlib の ArgumentError — FUJIMOTO Hisakuni <hisa@...>

[#30206] .dup がされてないらしい ? — Atsushi Kanaya <todo-a2c@...>

[#30212] ruby/ODBC — "Makoto Sato" <makotosato@...>

[#30215] パス文字列を操作するライブラリはありませんか？ — "Kaoru Shirai" <shirai@...1jp.com>

[#30220] rnet.rb 0.1.1 — rubikitch <rubikitch@...>

[#30224] bug?: $= and string hash key — genta@... (IHA, genta)

[#30225] Ruby インストールガイド — Kazuhiro NISHIYAMA <zn@...>

[#30227] TCPSocket.open でエラー？ — "Inoue" <inoue@...>

[#30234] Cookie について — "Toshiaki YAMASHITA" <yamashita@...>

[#30236] Re: Is there someone who could help with the newgroup problem? — dave@...

[#30253] 物権／債権（ re ：ライセンス） — Tacos <ozaki@...>

[#30265] Why NameError raises when module name equals class name? — IKEGAMI Daisuke <daisu-ik@...>

[#30270] setup.rb — rubikitch@...

[#30271] rd2html-ext-lib.rb — rubikitch <rubikitch@...>

[#30272] Hash の Hash — ABE Shigeru <shiger-a@...>

[#30281] オブジェクトのプロパティの名前の取得 — "Inoue" <inoue@...>

[#30301] round の仕様 — Shin-ichi TAKAYA <takaya@...>

[#30305] TMarshal — rubikitch@...

[#30312] composing a mail with a big file attached — "Akinori MUSHA" <knu@...>

[#30333] 共同著作物の謎（ re ：ライセンス） — Tacos <ozaki@...>

[#30337] インクルードしたクラスで元のモジュールにあったクラスを上書きしたい — Yuichiro MASUI <masui@...>

[#30345] PS2Linux への Ruby インストール — "T.Shimomura" <redbugml@...>

[#30359] sablot-0.5.0 — TAKAHASHI Masayoshi <maki@...>

[#30361] RWiki-1.2.1 — m_seki@...

[#30362] 1.6.4 と open3 — "Toshihisa Tsuji" <dayan-ml-ruby-list@...>

[#30365] Samba+File.stat+ 奇数秒ファイル — Take_tk <ggb03124@...>

[#30369] メーリングリストの検索 — Takahiro Kambe <taca@...>

[#30374] rdtool's bug? — Takahiro Kambe <taca@...>

[#30379] Win32API について — "okajima" <okajima@...>

[#30380] Float 演算は遅い？ — ABE Shigeru <shiger-a@...>

[#30389] Re: Win32API について — "okajima" <okajima@...>

[#30391] Root 権限をもたせたい — "takashi Yamaoka" <t_yamashita@...>

[#30404] setup.rb: pre_*, post_*, task 'all' — rubikitch@...

[#30405] Re: 1.6.4 と open3 — Tomoyuki Konowa <mld25999@...>

[#30407] MQ の環境設定について — "okajima" <okajima@...>

[#30413] WIN32OLE で msoxxx の定数を得るには ? — 岡 保嗣 <oka@...>

[#30420] Re: WIN32OLE で msoxxx の定数を得るには ? — tamra@...

[#30423] Ruby package on NetBSD has updated. — Takahiro Kambe <taca@...>

[ruby-list:30424] Re: 1.6.4 と open3

Thread

In This Thread

[#30101] 「パイプ」がわかりません。 — "伊藤　壮一" <soichi-i@...>

[#30111] みなさんありがとうございます。 — "伊藤　壮一" <soichi-i@...>

[#30404] setup.rb: pre_, post_, task 'all' — rubikitch@...

[#30413] WIN32OLE で msoxxx の定数を得るには ? — 岡保嗣 <oka@...>