Skip to content

Commit e86f511

Browse files
committed
Motivation:
The sql templates module does require strictly the codegen modules. These modules are only needed for the code generation phase. Changes: Set codegen and java.compiler module requirement to be static, so they are not required when using generated templates at runtime.
1 parent af6c72e commit e86f511

File tree

3 files changed

+34
-5
lines changed

3 files changed

+34
-5
lines changed

vertx-sql-client-templates/pom.xml

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -106,6 +106,33 @@
106106
</execution>
107107
</executions>
108108
</plugin>
109+
<plugin>
110+
<groupId>org.apache.maven.plugins</groupId>
111+
<artifactId>maven-failsafe-plugin</artifactId>
112+
<configuration>
113+
<useModulePath>true</useModulePath>
114+
<failIfNoTests>true</failIfNoTests>
115+
</configuration>
116+
<executions>
117+
<execution>
118+
<id>module-path-codegen</id>
119+
<goals>
120+
<goal>integration-test</goal>
121+
<goal>verify</goal>
122+
</goals>
123+
<configuration>
124+
<includes>
125+
<include>io/vertx/tests/sqlclient/templates/PgClientTest.java</include>
126+
</includes>
127+
<classpathDependencyExcludes>
128+
<classpathDependencyExclude>io.vertx:vertx-codegen-api</classpathDependencyExclude>
129+
<classpathDependencyExclude>io.vertx:vertx-codegen-processor</classpathDependencyExclude>
130+
<classpathDependencyExclude>io.vertx:vertx-codegen-json</classpathDependencyExclude>
131+
</classpathDependencyExcludes>
132+
</configuration>
133+
</execution>
134+
</executions>
135+
</plugin>
109136
</plugins>
110137
</build>
111138

vertx-sql-client-templates/src/main/java/module-info.java

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,13 @@
11
module io.vertx.sql.client.templates {
22

3+
requires static io.vertx.codegen.processor;
4+
requires static io.vertx.codegen.json;
5+
requires static io.vertx.codegen.api;
6+
requires static java.compiler;
7+
requires static io.vertx.docgen;
8+
39
requires io.vertx.sql.client;
4-
requires io.vertx.codegen.processor;
5-
requires io.vertx.codegen.api;
610
requires io.vertx.core;
7-
requires java.compiler;
8-
requires static io.vertx.docgen;
911

1012
exports io.vertx.sqlclient.templates;
1113
exports io.vertx.sqlclient.templates.annotations;

vertx-sql-client-templates/src/test/java/module-info.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
open module io.vertx.tests.sql.client.templates {
22
requires com.fasterxml.jackson.databind;
33
requires com.fasterxml.jackson.datatype.jsr310;
4-
requires io.vertx.codegen.api;
4+
requires static io.vertx.codegen.api;
55
requires io.vertx.core;
66
requires io.vertx.sql.client;
77
requires io.vertx.sql.client.mysql;

0 commit comments

Comments
 (0)