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

Unlock mime-type gem version to support Ruby 3 #1124

Closed
wants to merge 1 commit into from
Closed

Unlock mime-type gem version to support Ruby 3 #1124

wants to merge 1 commit into from

Conversation

yahonda
Copy link

@yahonda yahonda commented Oct 27, 2020

This pull request addresses these syntax errors which has been fixed by mime-types/ruby-mime-types#146

$ ruby -v
ruby 3.0.0dev (2020-10-24T13:53:53Z master 148961adcd) [x86_64-linux]
$ bundle exec rake spec

An error occurred while loading ./spec/delayed/command_spec.rb.
Failure/Error: require 'coveralls'

SyntaxError:
  /home/yahonda/.rbenv/versions/3.0.0-dev/lib/ruby/gems/3.0.0/gems/mime-types-2.6.2/lib/mime/types/logger.rb:26: _1 is reserved for numbered parameter
  /home/yahonda/.rbenv/versions/3.0.0-dev/lib/ruby/gems/3.0.0/gems/mime-types-2.6.2/lib/mime/types/logger.rb:26: _2 is reserved for numbered parameter
  /home/yahonda/.rbenv/versions/3.0.0-dev/lib/ruby/gems/3.0.0/gems/mime-types-2.6.2/lib/mime/types/logger.rb:26: _3 is reserved for numbered parameter
/home/yahonda/.rbenv/versions/3.0.0-dev/lib/ruby/gems/3.0.0/gems/mime-types-2.6.2/lib/mime/types/columnar.rb:16: warning: already initialized constant MIME::Types::Columnar::LOAD_MUTEX
/home/yahonda/.rbenv/versions/3.0.0-dev/lib/ruby/gems/3.0.0/gems/mime-types-2.6.2/lib/mime/types/columnar.rb:16: warning: previous definition of LOAD_MUTEX was here

This pull request addresses these syntax errors which has been fixed
by mime-types/ruby-mime-types#146

```ruby
$ ruby -v
ruby 3.0.0dev (2020-10-24T13:53:53Z master 148961adcd) [x86_64-linux]
$ bundle exec rake spec

An error occurred while loading ./spec/delayed/command_spec.rb.
Failure/Error: require 'coveralls'

SyntaxError:
  /home/yahonda/.rbenv/versions/3.0.0-dev/lib/ruby/gems/3.0.0/gems/mime-types-2.6.2/lib/mime/types/logger.rb:26: _1 is reserved for numbered parameter
  /home/yahonda/.rbenv/versions/3.0.0-dev/lib/ruby/gems/3.0.0/gems/mime-types-2.6.2/lib/mime/types/logger.rb:26: _2 is reserved for numbered parameter
  /home/yahonda/.rbenv/versions/3.0.0-dev/lib/ruby/gems/3.0.0/gems/mime-types-2.6.2/lib/mime/types/logger.rb:26: _3 is reserved for numbered parameter
/home/yahonda/.rbenv/versions/3.0.0-dev/lib/ruby/gems/3.0.0/gems/mime-types-2.6.2/lib/mime/types/columnar.rb:16: warning: already initialized constant MIME::Types::Columnar::LOAD_MUTEX
/home/yahonda/.rbenv/versions/3.0.0-dev/lib/ruby/gems/3.0.0/gems/mime-types-2.6.2/lib/mime/types/columnar.rb:16: warning: previous definition of LOAD_MUTEX was here
```
@albus522
Copy link
Member

albus522 commented Dec 9, 2020

I think you have something odd on your system. That gem line is for jruby not MRI. It would not apply to MRI ruby 3.0. Nothing in delayed job's gemfile for MRI ruby pulls in mime-types.

@yahonda
Copy link
Author

yahonda commented Dec 11, 2020

Thanks for the comment. It does not reproduce anymore. I might be doing something wrong.

@yahonda yahonda closed this Dec 11, 2020
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

Successfully merging this pull request may close these issues.

2 participants