Skip to content

Commit

Permalink
Fix #220: When generating a new project, restx-server-jetty dependenc…
Browse files Browse the repository at this point in the history
…y should be considered optional in order :

- To be able ot use it at compile time (particularly in AppServer)
- To be able to have IDE like IntelliJ to provide jetty libs to classpath when running AppServer
- To be able to not package it (nor its dependencies) when building the war
Note that by removing restx-server-jetty from packaged libs, we need to bring back restx-servlet artefact (which was initially pulled from restx-server-jetty)
directly at the dependency root level, in order to have it packaged into the war (otherwise, we wouldn't be able to use
restx servlet-specific stuff when deploying on standard servlet containers such as tomcat)
  • Loading branch information
fcamblor committed Oct 23, 2015
1 parent dc9fc40 commit 8a75daa
Showing 1 changed file with 2 additions and 1 deletion.
Expand Up @@ -33,7 +33,8 @@
{{#includeStatsModule}} {{#includeStatsModule}}
"io.restx:restx-stats-admin:${restx.version}", "io.restx:restx-stats-admin:${restx.version}",
{{/includeStatsModule}} {{/includeStatsModule}}
"io.restx:restx-server-jetty:${restx.version}", "io.restx:restx-servlet:${restx.version}",
"io.restx:restx-server-jetty:${restx.version}!optional",
"io.restx:restx-apidocs:${restx.version}", "io.restx:restx-apidocs:${restx.version}",
"io.restx:restx-specs-admin:${restx.version}", "io.restx:restx-specs-admin:${restx.version}",
"io.restx:restx-admin:${restx.version}", "io.restx:restx-admin:${restx.version}",
Expand Down

0 comments on commit 8a75daa

Please sign in to comment.