yogthos / lein-asset-minifier

Leiningen plugin for CSS/Js asset minifcation
Eclipse Public License 1.0
41 stars 7 forks source link

Ensure relative asset paths are resolved relative to lein root direct… #5

Closed publicmayhem closed 8 years ago

publicmayhem commented 8 years ago

yogthos, can you check my commit. It fixes assets given as relative paths, by ensuring they are resolved with respect to lein project root (:root project). This is needed for nested projects that use plugins such as lein-sub or lein-modules. Please see issue https://github.com/kumarshantanu/lein-sub/issues/13 which discusses the problem.

normalize-path was taken from leiningen.core.classpath

With this fix, I can then compile and install sub modules that include assets to minify via lein sub install.

R

yogthos commented 8 years ago

Ah perfect, I'll push out a new version tonight.

publicmayhem commented 8 years ago

Thx yogthos, great plugin btw!!

yogthos commented 8 years ago

thanks :)

yogthos commented 8 years ago

and new version's up on clojars