search

mape / connect-assetmanager

Middleware for Connect (node.js) for handling your static assets.
MIT License
310 stars 48 forks source link

Fixed windows compatibility #32

Closed dawnerd closed 12 years ago

dawnerd commented 12 years ago

Changed fs.watchFile to use newer fs.watch api.

tig commented 12 years ago

Hi dawnerd: Are you sure this the only change needed? I get a "watch Unknown system errono 123" when I apply this 2 line fix.

dawnerd commented 12 years ago

That fix worked for me. Not sure why you're getting that error.

tig commented 12 years ago

@dawnerd Is the project where you are using -assetmanager public? If so, would you mind pointing me at it so I can have another example of how to use it correctly. I'm having trouble knowing if I'm even doing it right. thanks.

tig commented 12 years ago

@dawnerd - I figured it out. Your fix does, in fact, work fine. +1 on the pull request.

prabirshrestha commented 12 years ago

+1 would to have that pull in with a new npm package too.