noir-clojure / lib-noir

A set of libraries for ring apps, including stateful sessions.
Eclipse Public License 1.0
483 stars 47 forks source link

noir.io/resource-path #99

Closed misty000 closed 10 years ago

misty000 commented 10 years ago

It has a problem in _Windows_, (io/resource-path) => "/E:/Devel/web/liberator-service/resources/public%5c"

use "/" instead File/separator is ok (if-let [path (resource (str "public" "/"))](.getPath path)) => "/E:/Devel/web/liberator-service/resources/public/"

yogthos commented 10 years ago

I don't have a windows machine handy to test this with, but if you'd like to make a pr with a change I can test that it doesn't break existing behavior and push it out.

yoctozepto commented 10 years ago

I have made a pull request with a fix for this issue.

yogthos commented 10 years ago

@yoctozepto thanks, just pushed out a new version with the fix