Skip to content

Commit

Permalink
fix: don't build uber JARs
Browse files Browse the repository at this point in the history
Instead we rely on classpath.txt to include everything we need.
  • Loading branch information
alecthomas committed Nov 10, 2023
1 parent 5f74abb commit b8ae866
Show file tree
Hide file tree
Showing 6 changed files with 5 additions and 26 deletions.
2 changes: 1 addition & 1 deletion Bitfile
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ COMMON_LOG_IN = backend/common/log/api.go
COMMON_LOG_OUT = backend/common/log/log_level_string.go

KT_RUNTIME_IN = kotlin-runtime/ftl-runtime/**/*.{kt,kts} pom.xml kotlin-runtime/ftl-runtime/**/pom.xml
KT_RUNTIME_OUT = kotlin-runtime/ftl-runtime/target/ftl-runtime-1.0-SNAPSHOT-jar-with-dependencies.jar
KT_RUNTIME_OUT = kotlin-runtime/ftl-runtime/target/ftl-runtime-1.0-SNAPSHOT.jar
KT_RUNTIME_RUNNER_TEMPLATE_OUT = build/template/ftl/jars/ftl-runtime.jar

KT_GENERATOR_IN = kotlin-runtime/ftl-generator/**/*.{kt,kts} pom.xml kotlin-runtime/ftl-runtime/**/pom.xml %{KT_RUNTIME_OUT}
Expand Down
1 change: 0 additions & 1 deletion examples/echo-kotlin/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,6 @@
<groupId>xyz.block</groupId>
<artifactId>ftl-runtime</artifactId>
<version>${ftl.version}</version>
<classifier>jar-with-dependencies</classifier>
<destFileName>ftl-runtime.jar</destFileName>
</artifactItem>
</artifactItems>
Expand Down
1 change: 0 additions & 1 deletion examples/online-boutique/services/ad-kotlin/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,6 @@
<groupId>xyz.block</groupId>
<artifactId>ftl-runtime</artifactId>
<version>${ftl.version}</version>
<classifier>jar-with-dependencies</classifier>
<destFileName>ftl-runtime.jar</destFileName>
</artifactItem>
</artifactItems>
Expand Down
24 changes: 3 additions & 21 deletions kotlin-runtime/ftl-runtime/pom.xml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<project xmlns="http://maven.apache.org/POM/4.0.0"
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">

<modelVersion>4.0.0</modelVersion>

Expand Down Expand Up @@ -76,23 +76,6 @@
<groupId>org.codehaus.mojo</groupId>
<artifactId>build-helper-maven-plugin</artifactId>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-assembly-plugin</artifactId>
<executions>
<execution>
<phase>package</phase>
<goals>
<goal>single</goal>
</goals>
<configuration>
<descriptorRefs>
<descriptorRef>jar-with-dependencies</descriptorRef>
</descriptorRefs>
</configuration>
</execution>
</executions>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId>
Expand Down Expand Up @@ -121,5 +104,4 @@
<url>https://maven.pkg.github.com/TBD54566975/ftl</url>
</repository>
</distributionManagement>
</project>

</project>
1 change: 0 additions & 1 deletion kotlin-runtime/scaffolding/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,6 @@
<groupId>xyz.block</groupId>
<artifactId>ftl-runtime</artifactId>
<version>${ftl.version}</version>
<classifier>jar-with-dependencies</classifier>
<destFileName>ftl-runtime.jar</destFileName>
</artifactItem>
</artifactItems>
Expand Down
2 changes: 1 addition & 1 deletion scripts/integration-tests
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ build_release() {
bit build/release/ftl-controller \
build/release/ftl-runner \
build/release/ftl \
kotlin-runtime/ftl-runtime/target/ftl-runtime-1.0-SNAPSHOT-jar-with-dependencies.jar \
kotlin-runtime/ftl-runtime/target/ftl-runtime-1.0-SNAPSHOT.jar \
kotlin-runtime/ftl-generator/target/ftl-generator-1.0-SNAPSHOT-jar-with-dependencies.jar \
build/template/ftl/jars/ftl-runtime.jar
}
Expand Down

0 comments on commit b8ae866

Please sign in to comment.