[ruby-dev:31573] Re: Dir.mktmpdir for 1.8

At Tue, 21 Aug 2007 17:33:06 +0900,
Tanaka Akira wrote:
> In article <[e-mail]>,
>   "Akinori MUSHA" <[e-mail]> writes:
>
> > 　ブランチポリシーに記載の条件を満たしていればよく、凍結期間外に
> > 私から個別の了承を取る必要はありません。
>
> はい、了解です。
> (ブランチポリシーから了承が必要な時には - 想定しているのは
> securerandom.rb の追加ですが - 了承をお願いすることになりま
> すが)

　わかりました。MLでの議論を読み返しておきます。

> > 　Tempfile#make_tmpname (private)のようにファイル名生成部分を
> > オーバーライドする手段がないことが多少気になりますが、作った
> > 一時ディレクトリの中に好きな名前のディレクトリを作ればいいので
> > 問題はなさそうですね。
>
> prefix は引数で指定できるので、問題は suffix ですが、ディレ
> クトリの suffix を指定する必要性は薄いだろうという判断です。
> ディレクトリに拡張子をつけたいケースはあまりないであろうとい
> うか。

　例として、 Mac OS X ではディレクトリにも拡張子を付けて種別の
判定に用いています。

　アプリケーションは .app, Keynote 書類は .key, 多国語名を持つ
フォルダは .localized 等です。

　こうした「ファイル」の一時コピーを作って処理したりするときは、
サフィックスを指定できないとそのままでは使えません。

> もし、つけたくなったら、そうですねぇ、今考えましたが prefix
> のところに [prefix, suffix] と指定するとかですかね。必要性が
> 感じられてからの話になりますが、拡張に無理があるということは
> なさそうです。

　なるほど。[prefix, suffix] で十分かもしれませんね。Tempfile も
そうしましょうか。

　ついでに $prefix.$$.$n という生成規則もいまいち(. を使っている,
乱数を使っていない, 等)なので変えたいが、ご丁寧にドキュメントに
"basename.pid.n" なんて明記してしまっているからなあ…。

Index: tempfile.rb
===================================================================
--- tempfile.rb	(revision 13119)
+++ tempfile.rb	(working copy)
@@ -15,9 +15,15 @@
   @@cleanlist = []
   @@lock = Mutex.new

-  # Creates a temporary file of mode 0600 in the temporary directory
-  # whose name is basename.pid.n and opens with mode "w+".  A Tempfile
-  # object works just like a File object.
+  # Creates a temporary file of mode 0600 in the temporary directory,
+  # opens it with mode "w+", and returns a Tempfile object which
+  # represents the created temporary file.  A Tempfile object can be
+  # treated just like a normal File object.
+  #
+  # The basename parameter is used to determine the name of a
+  # temporary file.  If an Array is given, the first element is used
+  # as prefix and the second as suffix, respectively.  Otherwise it is
+  # treated as a string and used as prefix.
   #
   # If tmpdir is omitted, the temporary directory is determined by
   # Dir::tmpdir provided by 'tmpdir.rb'.
@@ -65,7 +71,12 @@
   end

   def make_tmpname(basename, n)
-    sprintf('%s.%d.%d', basename, $$, n)
+    case basename
+    when Array
+      sprintf('%s.%d.%d%s', basename[0], $$, n, basename[1])
+    else
+      sprintf('%s.%d.%d', basename, $$, n)
+    end
   end
   private :make_tmpname


--
                     /
                    /__  __            Akinori.org / MUSHA.org
                   / )  )  ) )  /     FreeBSD.org / Ruby-lang.org
Akinori MUSHA aka / (_ /  ( (__(  @ iDaemons.org / and.or.jp

"Different eyes see different things,
    Different hearts beat on different strings --
       But there are times for you and me when all such things agree"