Compass / compass

Compass is no longer actively maintained. Compass is a Stylesheet Authoring Environment that makes your website design simpler to implement and easier to maintain.
http://compass-style.org
Other
6.72k stars 1.18k forks source link

warning: circular argument reference - browsers with mri 2.2.0 #1902

Open salimane opened 9 years ago

salimane commented 9 years ago

After upgrading to mri 2.2.0, I'm getting some warnings :

/usr/local/rvm/gems/ruby-2.2.0-rc1@project/gems/compass-core-1.0.1/lib/compass/core/caniuse.rb:72: warning: circular argument reference - browsers

This happens because of https://github.com/Compass/compass/blob/stable/core/lib/compass/core/caniuse.rb#L72

  # returns the prefixes needed by the list of browsers given
  def prefixes(browsers = browsers)
    result = browsers.map{|b| all_prefixes(b) }
    result.flatten!
    result.uniq!
    result.sort!
    result
  end

Thanks

jurriaan commented 9 years ago

browsers = browsers should be changed to browsers = browsers(), see https://bugs.ruby-lang.org/issues/10314

salimane commented 9 years ago

the pr #1903 should fix it.

davegson commented 9 years ago

same problem here

gastonmorixe commented 9 years ago

:+1:

meiokubo-zz commented 9 years ago

I've got same error and then just did bundle update is fixed this issue. Result is down below. $ bundle update Fetching gem metadata from https://rubygems.org/............. Fetching version metadata from https://rubygems.org/... Fetching dependency metadata from https://rubygems.org/.. Resolving dependencies... Installing chunky_png 1.3.5 (was 1.3.3) Using multi_json 1.11.2 (was 1.10.1) Using sass 3.4.19 (was 3.4.9) Installing compass-core 1.0.3 (was 1.0.1) Using compass-import-once 1.0.5 Using rb-fsevent 0.9.6 (was 0.9.4) Using ffi 1.9.10 (was 1.9.6) Using rb-inotify 0.9.5 Installing compass 1.0.3 (was 1.0.1) Using bundler 1.10.6 Bundle updated!

adyz commented 8 years ago

same here after update

damienmckenna commented 8 years ago

Another vote for "gem update" to fix the problem - I had this problem and updating the gems fixed it.

mkoller commented 8 years ago

Ran into this problem and this was my solution

$ gem uninstall compass $ gem update --system $ gem install compass