search

luminus-framework / luminus-template

a template project for the Luminus framework
http://www.luminusweb.net/
MIT License
648 stars 147 forks source link

possible issue with +site #75

Closed upgradingdave closed 10 years ago

upgradingdave commented 10 years ago

Hi @yogthos, while I was working on +static, I noticed that +site might have an issue on this line:

https://github.com/yogthos/luminus-template/blob/master/src/leiningen/new/luminus.clj#L169

I think there was a (if-let ...) expression added to layout.clj and the regex here maybe needs to be updated? In addition, I also added a try/catch for +static to layout.clj that might need to be included in the regex here as well. I haven't needed +site yet, but if/when I have time to test it out, I'll look deeper.

yogthos commented 10 years ago

That's a good catch, there definitely should be a try/catch around the context for site.

yogthos commented 10 years ago

should be fixed