luminus-framework / luminus-template

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

Unrecognized feature error with +dailycred or +site-dailycred #50

Closed limist closed 11 years ago

limist commented 11 years ago

With the command lein new luminus mysite +cljs +site +dailycred +http-kit there's a java.lang.Exception: unrecognized feature: +dailycred thrown with a long stacktrace.

And with the command lein new luminus mysite +cljs +site-dailycred +http-kit there's a calmer error message of simply Unrecognized options: +site-dailycred

Looking in src/leiningen/new/luminus.clj I'm not sure why either option causes problems, they're both in the file...

limist commented 11 years ago

Further tinkering shows it's the combination of +site +dailycred (crashes) or +site-dailycred (Unrecognized) that causes problems; +dailycred as the only option, or with other options besides +site doesn't cause problems.

yogthos commented 11 years ago

Yeah, that's a bug in the template generation, I'll take a look at cleaning that up.

yogthos commented 11 years ago

Fixed in the latest version, +dailycred will now correctly pull in the +site feature when selected, if +site is also specified then the project will still be generated correctly.