Merge pull request #34811 from zakkak/2023-07-18-fail-upx-with-remote…

…-container Fail build when combining compression with remote-container-build
quarkusio · Oct 5, 2023 · 62ef5f6 · 62ef5f6
2 parents b71b1e0 + 9a53a94
commit 62ef5f6
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/core/deployment/src/main/java/io/quarkus/deployment/pkg/steps/UpxCompressionBuildStep.java b/core/deployment/src/main/java/io/quarkus/deployment/pkg/steps/UpxCompressionBuildStep.java
@@ -51,6 +51,10 @@ public void compress(NativeConfig nativeConfig, NativeImageRunnerBuildItem nativ
             if (!runUpxFromHost(upxPathFromSystem.get(), image.getPath().toFile(), nativeConfig)) {
                 throw new IllegalStateException("Unable to compress the native executable");
             }
+        } else if (nativeConfig.remoteContainerBuild()) {
+            log.errorf("Compression of native executables is not yet implemented for remote container builds.");
+            throw new IllegalStateException(
+                    "Unable to compress the native executable: Compression of native executables is not yet supported for remote container builds");
         } else if (nativeImageRunner.isContainerBuild()) {
             log.infof("Running UPX from a container using the builder image: " + effectiveBuilderImage);
             if (!runUpxInContainer(image, nativeConfig, effectiveBuilderImage)) {