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

Fix for Rails 4.1.0+ #95

Closed
wants to merge 1 commit into from

Conversation

bartoszkopinski
Copy link

cattr_* methods have been moved to the Module in this commit.
That resulted in syntax errors from re-defining extension methods:

/gems/aws-s3-0.6.3/lib/aws/s3/extensions.rb:223: `@@{' is not allowed as a class variable name
/gems/aws-s3-0.6.3/lib/aws/s3/extensions.rb:223: syntax error, unexpected end-of-input
unless defined? @@{:instance_writer=>true}
^
/gems/aws-s3-0.6.3/lib/aws/s3/extensions.rb:223:in `class_eval'
/gems/aws-s3-0.6.3/lib/aws/s3/extensions.rb:223:in `block in cattr_reader'
/gems/aws-s3-0.6.3/lib/aws/s3/extensions.rb:222:in `each'
/gems/aws-s3-0.6.3/lib/aws/s3/extensions.rb:222:in `cattr_reader'
/gems/aws-s3-0.6.3/lib/aws/s3/extensions.rb:258:in `cattr_accessor'
/gems/activesupport-4.1.0.rc1/lib/active_support/cache.rb:155:in `<class:Store>'
/gems/activesupport-4.1.0.rc1/lib/active_support/cache.rb:154:in `<module:Cache>'
/gems/activesupport-4.1.0.rc1/lib/active_support/cache.rb:14:in `<module:ActiveSupport>'
/gems/activesupport-4.1.0.rc1/lib/active_support/cache.rb:12:in `<top (required)>'
/gems/railties-4.1.0.rc1/lib/rails/application/bootstrap.rb:66:in `block in <module:Bootstrap>'
/gems/railties-4.1.0.rc1/lib/rails/initializable.rb:30:in `instance_exec'
/gems/railties-4.1.0.rc1/lib/rails/initializable.rb:30:in `run'
/gems/railties-4.1.0.rc1/lib/rails/initializable.rb:55:in `block in run_initializers'
/gems/railties-4.1.0.rc1/lib/rails/initializable.rb:54:in `run_initializers'
/gems/railties-4.1.0.rc1/lib/rails/application.rb:286:in `initialize!'
/gems/railties-4.1.0.rc1/lib/rails/railtie.rb:194:in `public_send'
/gems/railties-4.1.0.rc1/lib/rails/railtie.rb:194:in `method_missing'
/app/current/config/environment.rb:5:in `<top (required)>'
/gems/activesupport-4.1.0.rc1/lib/active_support/dependencies.rb:247:in `require'
/gems/activesupport-4.1.0.rc1/lib/active_support/dependencies.rb:247:in `block in require'
/gems/activesupport-4.1.0.rc1/lib/active_support/dependencies.rb:232:in `load_dependency'
/gems/activesupport-4.1.0.rc1/lib/active_support/dependencies.rb:247:in `require'
/gems/railties-4.1.0.rc1/lib/rails/application.rb:262:in `require_environment!'
/gems/railties-4.1.0.rc1/lib/rails/application.rb:346:in `block in run_tasks_blocks'
/gems/sprockets-rails-2.0.1/lib/sprockets/rails/task.rb:54:in `block (2 levels) in define'

This change should be backward compatible and work with Rails 4.1.0.
It also makes more sense to check for inherited methods as well.

@davetron5000
Copy link

👍 on this.

@fnando
Copy link

fnando commented Apr 11, 2014

+1

@scotthelm
Copy link

👍

@georgedrummond
Copy link

👍 can this get merged?

@deevis
Copy link

deevis commented Jun 2, 2014

+1 yes - please merge this. Also, how do I get a thumbs-up icon?!?

@halrogers-claritas
Copy link

👍

@lucasmartins
Copy link

👍

@adrianstainforth
Copy link

👍

@Shelvak
Copy link

Shelvak commented Jun 9, 2014

👍

1 similar comment
@akeem
Copy link

akeem commented Jun 9, 2014

👍

@itsterry
Copy link

Thumbs up

@robertomiranda
Copy link

👍 pelase merge this!

@mool
Copy link

mool commented Jul 15, 2014

👍

1 similar comment
@stasl
Copy link

stasl commented Jul 21, 2014

+1

@nyku
Copy link

nyku commented Jul 29, 2014

can anyone merge this? 👍

@mrkamel
Copy link

mrkamel commented Aug 6, 2014

+1

2 similar comments
@lsylvester
Copy link

👍

@springerigor
Copy link

👍

@bhgames
Copy link

bhgames commented Sep 10, 2014

+1

4 similar comments
@achiinto
Copy link

👍

@denisahearn
Copy link

+1

@c4ssio
Copy link

c4ssio commented Apr 14, 2015

+1

@shime
Copy link

shime commented Apr 16, 2015

👍

@lukeasrodgers
Copy link

Seems like this gem is no longer maintained, and the author has no public github activity over the last year. Anyone interested in collaborating on a fork?

@davetron5000
Copy link

Amazon has released a gem for all of AWS and includes S3. It works similarly to this. We've moved over to that and no longer use this gem.

@fnando
Copy link

fnando commented Jul 24, 2015

I'm now using Fog.

pstare added a commit to Shopify/aws-s3 that referenced this pull request May 20, 2017
…rsion 0.6.3, and apply Rails 4.1.0+ syntax fix from marcel#95
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.