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

glob acting in surprising manner #108

Closed
aj0strow opened this Issue Jun 5, 2013 · 4 comments

Comments

Projects
None yet
2 participants
@aj0strow

aj0strow commented Jun 5, 2013

I was expecting the line '/js/views/**/*.js' to come up with both js/views/application.js and js/views/dashboard/header.js, instead I have to do the following

'/js/views/*.js', '/js/views/**/*.js'

It should work like Dir#glob, no?

"Using sinatra-assetpack (0.2.3) "

@j15e

This comment has been minimized.

Collaborator

j15e commented Jun 5, 2013

What your code looks like exactly?

@aj0strow

This comment has been minimized.

aj0strow commented Jun 5, 2013

Here's the asset block:

app.assets do
  css :styles, '/css/application.css', [
    '/css/main.css'
  ]

  js :scripts, '/js/application.js', %w{
    /js/vendor/underscore*.js
    /js/vendor/jquery*.js
    /js/vendor/backbone*.js

    /js/modreal.js
    /js/models/*.js
    /js/models/**/*.js
    /js/collections/*.js
    /js/collections/**/*.js
    /js/views/*.js
    /js/views/**/*.js
    /js/routers/*.js

    /js/app.js
    /js/setup.js
    /js/vendor.js


  }

  css_compression :sass
  js_compression :yui
end

j15e added a commit that referenced this issue Jun 22, 2013

@j15e j15e closed this in 47d4ad1 Jun 22, 2013

@j15e

This comment has been minimized.

Collaborator

j15e commented Jun 22, 2013

Fixed in latest 0.2.7 release

@aj0strow

This comment has been minimized.

aj0strow commented Jun 22, 2013

Sweet, thanks!

j15e added a commit that referenced this issue Jun 24, 2013

j15e added a commit that referenced this issue Jun 24, 2013

Improve Css#preproc behaviour. Fix #108
- More explicit url match skip behaviour
- Split managed assets url building into a method
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment