Tempfile.open { ... } does not unlink the file

[eregon]

ruby -rtempfile -e 'Tempfile.open("txt") { |f| $path = f.path }; p File.exist?($path)'
true

but it should be false.

This means even after the block finishes to execute the file still exists on a disk
And this might or not be addressed by finalization much later on.

This is inconsistent with the resource block pattern and e.g. File.open.

Since the block creates the file, it should also delete it, so there are no leftovers after the block.

How about using close! instead of close in

tempfile/lib/tempfile.rb

Line 293 in ec3dd03

tempfile.close

?

Found by ruby/spec@d347e89 and the leak detector.

[shugo]

@eregon
Tempfile was originally designed as GC dependent, and it was an expected behavior that Tempfile.open {} doesn't unlink the file.
I'm not against this change, but there may be applications depending on the old behavior, so I think we should discuss this issue at bugs.ruby-lang.org.

[shugo]

FYI, Tempfile.create was introduced instead of Tempfile.open to avoid breaking change.

[znz]

Japanese reference manual has an example that use path after open with block.
This change breaks such usage.

https://docs.ruby-lang.org/ja/2.7.0/method/Tempfile/s/new.html

require 'tempfile'

tf = Tempfile.open("temp"){|fp|
  fp.puts "hoge"
  fp
}
# テンポラリファイルへのパスを表示
p tf.path
p File.read(tf.path) #=> "hoge\n"

[eregon]

OK, I'll create an issue on bugs.ruby-lang.org.

I think it's very strange that open doesn't fully release the resources it created.
So IMHO still good to have.
Also the English docs don't mention the file is kept on disk after the block:
https://docs.ruby-lang.org/en/2.7.0/Tempfile.html#method-c-open

In all of ruby/ruby I found only 2 usages depending on that, and 10+ usages which had to close! explicitly due to these surprising semantics, or forgot to.

[hsbt]

@eregon I and @ko1 discuss about the right place for discussing the feature change. Let's move to bugs.ruby-lang.org.

And I will disable the issue feature with unmaintained libraries on https://github.com/ruby/ruby/blob/master/doc/maintainers.rdoc.

[eregon]

https://bugs.ruby-lang.org/issues/17144