Skip to content

Commit

Permalink
removed Dir[] for glob fallign back to Sass::Utils for glob support c…
Browse files Browse the repository at this point in the history
…loses#1036
  • Loading branch information
scottdavis committed Oct 11, 2012
1 parent d4db12f commit 58babac
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lib/compass/sprite_importer.rb
Expand Up @@ -16,7 +16,7 @@ class SpriteImporter < Sass::Importers::Base
def self.find_all_sprite_map_files(path)
hex = "[0-9a-f]"
glob = "*-s#{hex*10}{#{VALID_EXTENSIONS.join(",")}}"
Dir.glob(File.join(path, "**", glob))
Sass::Util.glob(File.join(path, "**", glob))
end

def find(uri, options)
Expand Down Expand Up @@ -75,7 +75,7 @@ def self.path(uri)
# Returns the Glob of image files for the uri
def self.files(uri)
Compass.configuration.sprite_load_path.compact.each do |folder|
files = Dir[File.join(folder, uri)].sort
files = Sass::Util.glob(File.join(folder, uri)).sort
next if files.empty?
return files
end
Expand Down

7 comments on commit 58babac

@telekineticyeti
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Scott! This fixed my windows sprite issue.

@juniorgodoi
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, fixed!

@j-steve
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! This fixed the sprite issue on my Windows machine too.

@sas48
Copy link

@sas48 sas48 commented on 58babac Nov 18, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hellow!

I'm testnig on windows.
I change the sprite_importer.rb file and what then? Do I have to rebuild compass or just repair sprite_importer.rb file and should it work?
Please, I need help!

Thanks!

@mvpriyank
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still not working for me. Any further steps to be followed after these changes?

@wzup
Copy link

@wzup wzup commented on 58babac Jan 23, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not working for me.
Windows 7 64bit
Ruby 1.9.3p545 (2014-02-24) [i386-mingw32]
Gem 2.4.5
Compass 0.12.6 (Alnilam)

@tharapon
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! It's work!
C:\Ruby200-x64\lib\ruby\gems\2.0.0\gems\compass-0.12.2\lib\compass\sprite_importer.rb

Please sign in to comment.