Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Truffle] Dir.mktmpdir required for a lot of MRI tests #2837

Closed
bjfish opened this issue Apr 16, 2015 · 1 comment
Closed

[Truffle] Dir.mktmpdir required for a lot of MRI tests #2837

bjfish opened this issue Apr 16, 2015 · 1 comment

Comments

@bjfish
Copy link
Contributor

bjfish commented Apr 16, 2015

For reference, here is a patch of changes to temporarily workaround this issue (you may need to adjust tmp var to your local echo $TMPDIR):
https://gist.github.com/bjfish/013d9e650f9b17799dbf

Expected:

ruby -e 'require "tmpdir"; tmpdir = Dir.mktmpdir("autoload"); puts "Done"'
Done

Actual

~/Documents/jruby/bin/jruby -X+T -e 'require "tmpdir"; tmpdir = Dir.mktmpdir("autoload"); puts "Done"'
/Users/brandonfish/Documents/jruby/lib/ruby/truffle/mri/../../stdlib/tmpdir.rb:93:in `mkdir': wrong number of arguments (2 for 1) (ArgumentError)
    from /Users/brandonfish/Documents/jruby/lib/ruby/truffle/mri/../../stdlib/tmpdir.rb:93:in `block in mktmpdir'
    from /Users/brandonfish/Documents/jruby/lib/ruby/truffle/mri/../../stdlib/tmpdir.rb:146:in `create'
    from /Users/brandonfish/Documents/jruby/lib/ruby/truffle/mri/../../stdlib/tmpdir.rb:93:in `mktmpdir'
    from -e:1:in `<main>'

@chrisseaton chrisseaton added this to the truffle-dev milestone Apr 17, 2015
@chrisseaton
Copy link
Contributor

Fixed in b67e232

@enebo enebo added this to the Invalid or Duplicate milestone Dec 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants