weavejester / lein-ring

Ring plugin for Leiningen
Eclipse Public License 1.0
501 stars 100 forks source link

Error with :init in 0.9.0 #143

Closed arohner closed 9 years ago

arohner commented 9 years ago

In project.clj:

:ring {:init foo.main/-main
       :handler foo.system/ring-handler}

lein ring war works fine with this on 0.8.13, but fails on 0.9.0 with

Exception in thread "main" java.lang.Exception: Found lib name 'foo.system' containing period with prefix 'foo.main'. lib names inside prefix lists must not contain periods

Commenting out :init makes the war compile on 0.9.0

MichaelBlume commented 9 years ago

Thanks for the report, digging now

On Sat Jan 10 2015 at 1:58:13 PM Allen Rohner notifications@github.com wrote:

In project.clj:

:ring {:init foo.main/-main :handler foo.system/ring-handler }

lein ring war works fine with this on 0.8.13, but fails on 0.9.0 with

Exception in thread "main" java.lang.Exception: Found lib name 'foo.system' containing period with prefix 'foo.main'. lib names inside prefix lists must not contain periods

Commenting out :init makes the war compile on 0.9.0

— Reply to this email directly or view it on GitHub https://github.com/weavejester/lein-ring/issues/143.

MichaelBlume commented 9 years ago

Fixed in master, thanks again for the report.

@weavejester can I just push a commit with 0.9.1 in the project.clj and get a new release or is there more that needs to be done?

weavejester commented 9 years ago

I'll push out a new release tomorrow. In future, could you also make sure that your commit messages have the same format as the other commits in the repository?