ruby-dev (Japanese) list archive (unofficial mirror)
 help / color / mirror / Atom feed
* [ruby-dev:45977] [ruby-trunk - Feature #6757][Open] [PATCH] Add Process.getsid
@ 2012-07-20  2:47 fumiyas (Fumiyasu SATOH)
  2012-10-26 22:23 ` [ruby-dev:46283] [ruby-trunk - Feature #6757] " ko1 (Koichi Sasada)
                   ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: fumiyas (Fumiyasu SATOH) @ 2012-07-20  2:47 UTC (permalink / raw
  To: ruby developers list


Issue #6757 has been reported by fumiyas (Fumiyasu SATOH).

----------------------------------------
Feature #6757: [PATCH] Add Process.getsid
https://bugs.ruby-lang.org/issues/6757

Author: fumiyas (Fumiyasu SATOH)
Status: Open
Priority: Normal
Assignee: 
Category: ext
Target version: 2.0.0


setsid(2) に相当する Process.setsid はありますが、getsid(2) に
相当する Process.getsid がありません。

Process.getsid を追加するパッチを添付します。


-- 
http://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [ruby-dev:46283] [ruby-trunk - Feature #6757] [PATCH] Add Process.getsid
  2012-07-20  2:47 [ruby-dev:45977] [ruby-trunk - Feature #6757][Open] [PATCH] Add Process.getsid fumiyas (Fumiyasu SATOH)
@ 2012-10-26 22:23 ` ko1 (Koichi Sasada)
  2012-10-27  1:13 ` [ruby-dev:46292] [ruby-trunk - Feature #6757][Assigned] " mame (Yusuke Endoh)
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 6+ messages in thread
From: ko1 (Koichi Sasada) @ 2012-10-26 22:23 UTC (permalink / raw
  To: ruby developers list


Issue #6757 has been updated by ko1 (Koichi Sasada).

Assignee set to mame (Yusuke Endoh)

遠藤さん,どうしたもんでしょうか.
----------------------------------------
Feature #6757: [PATCH] Add Process.getsid
https://bugs.ruby-lang.org/issues/6757#change-31702

Author: fumiyas (Fumiyasu SATOH)
Status: Open
Priority: Normal
Assignee: mame (Yusuke Endoh)
Category: ext
Target version: 2.0.0


setsid(2) に相当する Process.setsid はありますが、getsid(2) に
相当する Process.getsid がありません。

Process.getsid を追加するパッチを添付します。


-- 
http://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [ruby-dev:46292] [ruby-trunk - Feature #6757][Assigned] [PATCH] Add Process.getsid
  2012-07-20  2:47 [ruby-dev:45977] [ruby-trunk - Feature #6757][Open] [PATCH] Add Process.getsid fumiyas (Fumiyasu SATOH)
  2012-10-26 22:23 ` [ruby-dev:46283] [ruby-trunk - Feature #6757] " ko1 (Koichi Sasada)
@ 2012-10-27  1:13 ` mame (Yusuke Endoh)
  2012-10-28  5:30   ` [ruby-dev:46304] " KOSAKI Motohiro
  2012-11-24  1:07 ` [ruby-dev:46599] [ruby-trunk - Feature #6757] " mame (Yusuke Endoh)
  2012-11-24  2:53 ` [ruby-dev:46605] " kosaki (Motohiro KOSAKI)
  3 siblings, 1 reply; 6+ messages in thread
From: mame (Yusuke Endoh) @ 2012-10-27  1:13 UTC (permalink / raw
  To: ruby developers list


Issue #6757 has been updated by mame (Yusuke Endoh).

Status changed from Open to Assigned
Assignee changed from mame (Yusuke Endoh) to akr (Akira Tanaka)

こういうシステム系の機能は判断する自信ないです。
一家言ありそうな akr さんに振ってみます。

テストがないですが、意味のあるテストは書けないものですかね。

-- 
Yusuke Endoh <mame@tsg•ne.jp>
----------------------------------------
Feature #6757: [PATCH] Add Process.getsid
https://bugs.ruby-lang.org/issues/6757#change-31737

Author: fumiyas (Fumiyasu SATOH)
Status: Assigned
Priority: Normal
Assignee: akr (Akira Tanaka)
Category: ext
Target version: 2.0.0


setsid(2) に相当する Process.setsid はありますが、getsid(2) に
相当する Process.getsid がありません。

Process.getsid を追加するパッチを添付します。


-- 
http://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [ruby-dev:46304] Re: [ruby-trunk - Feature #6757][Assigned] [PATCH] Add Process.getsid
  2012-10-27  1:13 ` [ruby-dev:46292] [ruby-trunk - Feature #6757][Assigned] " mame (Yusuke Endoh)
@ 2012-10-28  5:30   ` KOSAKI Motohiro
  0 siblings, 0 replies; 6+ messages in thread
From: KOSAKI Motohiro @ 2012-10-28  5:30 UTC (permalink / raw
  To: ruby developers list

> こういうシステム系の機能は判断する自信ないです。
> 一家言ありそうな akr さんに振ってみます。

こういう単に情報取得するだけの機能はノータイムで入れていいという認識。
逆にちゃんと考えないといけないのは、Cと同じ使い方をどうやっても提供できないケースで例えば

・IO: Ruby独自のバッファリングとエンコードの話があるのでsysread/syswriteが混ぜるな危険になる
・fork: ruby scriptをasync signal safeにする方法はないので、Cと同じ使い勝手を提供できない
・Trap: 同様に生のシグナルハンドラをRubyスクリプトでは書けないので、Cのシグナルハンドラと
  RubyのTrapハンドラは似て非なるもの

みたいなやつら。このケースは入れてしまっていいという認識。
しかし、あらためて見なおしてみると setsidとgetsidは本来 Process::Sys に入っているべき生の
操作なんじゃないかという気がしてきました。

ProcessとProcess::Sysの使い分けの指針は今どうなってるんでしょう?


> テストがないですが、意味のあるテストは書けないものですかね。

DLかsyscallつかって生のgetsid() 呼んで結果を比較する?

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [ruby-dev:46599] [ruby-trunk - Feature #6757] [PATCH] Add Process.getsid
  2012-07-20  2:47 [ruby-dev:45977] [ruby-trunk - Feature #6757][Open] [PATCH] Add Process.getsid fumiyas (Fumiyasu SATOH)
  2012-10-26 22:23 ` [ruby-dev:46283] [ruby-trunk - Feature #6757] " ko1 (Koichi Sasada)
  2012-10-27  1:13 ` [ruby-dev:46292] [ruby-trunk - Feature #6757][Assigned] " mame (Yusuke Endoh)
@ 2012-11-24  1:07 ` mame (Yusuke Endoh)
  2012-11-24  2:53 ` [ruby-dev:46605] " kosaki (Motohiro KOSAKI)
  3 siblings, 0 replies; 6+ messages in thread
From: mame (Yusuke Endoh) @ 2012-11-24  1:07 UTC (permalink / raw
  To: ruby developers list


Issue #6757 has been updated by mame (Yusuke Endoh).

Assignee changed from akr (Akira Tanaka) to kosaki (Motohiro KOSAKI)

kosaki さんが accept 気味なので、kosaki さんがよければ入れてください。
嫌なら next minor にしといてください。

-- 
Yusuke Endoh <mame@tsg•ne.jp>
----------------------------------------
Feature #6757: [PATCH] Add Process.getsid
https://bugs.ruby-lang.org/issues/6757#change-33703

Author: fumiyas (Fumiyasu SATOH)
Status: Assigned
Priority: Normal
Assignee: kosaki (Motohiro KOSAKI)
Category: ext
Target version: 2.0.0


setsid(2) に相当する Process.setsid はありますが、getsid(2) に
相当する Process.getsid がありません。

Process.getsid を追加するパッチを添付します。


-- 
http://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [ruby-dev:46605] [ruby-trunk - Feature #6757] [PATCH] Add Process.getsid
  2012-07-20  2:47 [ruby-dev:45977] [ruby-trunk - Feature #6757][Open] [PATCH] Add Process.getsid fumiyas (Fumiyasu SATOH)
                   ` (2 preceding siblings ...)
  2012-11-24  1:07 ` [ruby-dev:46599] [ruby-trunk - Feature #6757] " mame (Yusuke Endoh)
@ 2012-11-24  2:53 ` kosaki (Motohiro KOSAKI)
  3 siblings, 0 replies; 6+ messages in thread
From: kosaki (Motohiro KOSAKI) @ 2012-11-24  2:53 UTC (permalink / raw
  To: ruby developers list


Issue #6757 has been updated by kosaki (Motohiro KOSAKI).


r37825 で入れました。テストの追加と、引数を省略した時にカレントプロセスとみなす処理の追加を行いました(ほとんどのケースでカレントプロセスにしか興味ないはず)

佐藤さん、ありがとうございました。
----------------------------------------
Feature #6757: [PATCH] Add Process.getsid
https://bugs.ruby-lang.org/issues/6757#change-33753

Author: fumiyas (Fumiyasu SATOH)
Status: Closed
Priority: Normal
Assignee: kosaki (Motohiro KOSAKI)
Category: ext
Target version: 2.0.0


setsid(2) に相当する Process.setsid はありますが、getsid(2) に
相当する Process.getsid がありません。

Process.getsid を追加するパッチを添付します。


-- 
http://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2012-11-24  2:55 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-07-20  2:47 [ruby-dev:45977] [ruby-trunk - Feature #6757][Open] [PATCH] Add Process.getsid fumiyas (Fumiyasu SATOH)
2012-10-26 22:23 ` [ruby-dev:46283] [ruby-trunk - Feature #6757] " ko1 (Koichi Sasada)
2012-10-27  1:13 ` [ruby-dev:46292] [ruby-trunk - Feature #6757][Assigned] " mame (Yusuke Endoh)
2012-10-28  5:30   ` [ruby-dev:46304] " KOSAKI Motohiro
2012-11-24  1:07 ` [ruby-dev:46599] [ruby-trunk - Feature #6757] " mame (Yusuke Endoh)
2012-11-24  2:53 ` [ruby-dev:46605] " kosaki (Motohiro KOSAKI)

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).