-
Notifications
You must be signed in to change notification settings - Fork 314
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
Failure to create test fixtures on JRuby #104
Comments
Thanks! I'll check into whether this is a JRuby bug or not. Feel free to ping me if you have any other JRuby issues. |
Opened this: jruby/jruby#1149 -- and I'll look into the new JRuby failures that popped up. |
githubmo
pushed a commit
to githubmo/rubyzip
that referenced
this issue
Jan 6, 2014
…ypt-perf * 'master' of https://github.com/rubyzip/rubyzip: Explicitly add the released 2.1.0 Ruby version Remove branch restriction Fix Rubinius by adding newly required gems, updating label in .travis.yml Update README.md Update README.md Make File.open_buffer support Tempfiles Version bump Update Changelog with Ruby 1.9 requirement Update README to reflect 1.9 requirement Fix rubyzip#106 Set options about restoring ownerships, permissions and times. restore permissions enabled by default. fix jRuby Building rubyzip#104 Fix rubyzip#28 and rubyzip#103 disable jRuby for a while Fix rubyzip#102 recover file permissions if zip file was exist Add missing Zip::Entry arguments to Zip::File#get_output_stream. Fixes rubyzip#100 fix string encoding of zip64 header ids for ruby 2.0 Add read/write support for zip64 extensions Conflicts: lib/zip/extra_field.rb
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Under JRuby, the
rake
command fails the first time. For example:But if I remove line 96 from
test/gentestfiles.rb
the task works and all tests pass. Also, if the fixtures where created by MRI, then the tests pass under JRuby.Are there any suggested work arounds? rubyzip is essential to Warbler, so I'm happy to help get it back under CI.
The text was updated successfully, but these errors were encountered: