Skip to content

Commit 8a75daa

Browse files
committed
Fix #220: When generating a new project, restx-server-jetty dependency 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)
1 parent dc9fc40 commit 8a75daa

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

restx-core-shell/src/main/resources/templates/srv/main/_md.restx.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,8 @@
3333
{{#includeStatsModule}}
3434
"io.restx:restx-stats-admin:${restx.version}",
3535
{{/includeStatsModule}}
36-
"io.restx:restx-server-jetty:${restx.version}",
36+
"io.restx:restx-servlet:${restx.version}",
37+
"io.restx:restx-server-jetty:${restx.version}!optional",
3738
"io.restx:restx-apidocs:${restx.version}",
3839
"io.restx:restx-specs-admin:${restx.version}",
3940
"io.restx:restx-admin:${restx.version}",

0 commit comments

Comments
 (0)