Add optional S3 dependency to py embedded-server (#5180)

Also, adds overlooked sql and s3 to server-jetty-app-custom example.
deephaven · Feb 28, 2024 · 3fb6fd8 · 3fb6fd8
1 parent fc87685
commit 3fb6fd8
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 0 deletions.
diff --git a/py/embedded-server/java-runtime/build.gradle b/py/embedded-server/java-runtime/build.gradle
@@ -29,6 +29,10 @@ dependencies {
     if (!hasProperty('excludeSql')) {
         runtimeOnly project(':engine-sql')
     }
+
+    if (!hasProperty('excludeS3')) {
+        runtimeOnly project(':extensions-s3')
+    }
 }
 
 // making a dir here isn't optimal, but without it we need to make py-embedded-server be a java and a python

diff --git a/server/jetty-app-custom/build.gradle b/server/jetty-app-custom/build.gradle
@@ -46,6 +46,18 @@ if (!hasProperty('excludeClockImpl')) {
     extraJvmArgs += ['--add-exports', 'java.base/jdk.internal.misc=ALL-UNNAMED']
 }
 
+if (!hasProperty('excludeSql')) {
+    dependencies {
+        runtimeOnly project(':engine-sql')
+    }
+}
+
+if (!hasProperty('excludeS3')) {
+    dependencies {
+        runtimeOnly project(':extensions-s3')
+    }
+}
+
 if (hasProperty('devCerts') || hasProperty('devMTLS')) {
     extraJvmArgs += [
             '-Dhttp.port=8443',