Drop long deprecated native config options

zakkak · zakkak · commit 2fa551627f03 · 2025-10-06T11:57:14.000+03:00
diff --git a/core/deployment/src/main/java/io/quarkus/deployment/pkg/NativeConfig.java b/core/deployment/src/main/java/io/quarkus/deployment/pkg/NativeConfig.java
@@ -64,31 +64,6 @@ public interface NativeConfig {
     @WithDefault("false")
     boolean enableHttpsUrlHandler();
 
-    /**
-     * If all security services should be added to the native image
-     *
-     * @deprecated {@code --enable-all-security-services} was removed in GraalVM 21.1 https://github.com/oracle/graal/pull/3258
-     */
-    @WithDefault("false")
-    @Deprecated
-    boolean enableAllSecurityServices();
-
-    /**
-     * If {@code -H:+InlineBeforeAnalysis} flag will be added to the native-image run
-     *
-     * @deprecated inlineBeforeAnalysis is always enabled starting from GraalVM 21.3.
-     */
-    @Deprecated
-    @WithDefault("true")
-    boolean inlineBeforeAnalysis();
-
-    /**
-     * @deprecated JNI is always enabled starting from GraalVM 19.3.1.
-     */
-    @Deprecated
-    @WithDefault("true")
-    boolean enableJni();
-
     /**
      * The default value for java.awt.headless JVM option.
      * Switching this option affects linking of awt libraries.
@@ -173,16 +148,6 @@ public interface NativeConfig {
     @WithDefault("true")
     boolean publishDebugBuildProcessPort();
 
-    /**
-     * If the native image server should be restarted.
-     *
-     * @deprecated Since GraalVM 20.2.0 the native image server has become an experimental feature and is disabled by
-     *             default.
-     */
-    @Deprecated
-    @WithDefault("false")
-    boolean cleanupServer();
-
     /**
      * If isolates should be enabled
      */
@@ -196,18 +161,6 @@ public interface NativeConfig {
     @WithDefault("false")
     boolean enableFallbackImages();
 
-    /**
-     * If the native image server should be used. This can speed up compilation but can result in changes not always
-     * being picked up due to cache invalidation not working 100%
-     *
-     * @deprecated This used to be the default prior to GraalVM 20.2.0 and this configuration item was used to disable
-     *             it as it was not stable. Since GraalVM 20.2.0 the native image server has become an experimental
-     *             feature.
-     */
-    @Deprecated
-    @WithDefault("false")
-    boolean enableServer();
-
     /**
      * If all META-INF/services entries should be automatically registered
      */
@@ -339,15 +292,6 @@ default String getEffectiveImage() {
      */
     Optional<List<MonitoringOption>> monitoring();
 
-    /**
-     * If full stack traces are enabled in the resulting image
-     *
-     * @deprecated GraalVM 23.1+ will always build with full stack traces.
-     */
-    @WithDefault("true")
-    @Deprecated
-    boolean fullStackTraces();
-
     /**
      * If the reports on call paths and included packages/classes/methods should be generated
      */
diff --git a/core/deployment/src/main/java/io/quarkus/deployment/pkg/steps/NativeImageBuildStep.java b/core/deployment/src/main/java/io/quarkus/deployment/pkg/steps/NativeImageBuildStep.java
@@ -246,14 +246,6 @@ public NativeImageBuildItem build(NativeConfig nativeConfig, LocalesBuildTimeCon
         checkGraalVMVersion(graalVMVersion);
 
         try {
-            if (nativeConfig.cleanupServer()) {
-                log.warn(
-                        "Your application is setting the deprecated 'quarkus.native.cleanup-server' configuration key"
-                                + " to true. Please consider removing this configuration key as it is ignored"
-                                + " (The Native image build server is always disabled) and it will be removed in a"
-                                + " future Quarkus version.");
-            }
-
             NativeImageInvokerInfo commandAndExecutable = new NativeImageInvokerInfo.Builder()
                     .setNativeConfig(nativeConfig)
                     .setLocalesBuildTimeConfig(localesBuildTimeConfig)
@@ -717,7 +709,6 @@ public Builder setNativeMonitoringOptions(List<NativeMonitoringBuildItem> option
             public NativeImageInvokerInfo build() {
                 List<String> nativeImageArgs = new ArrayList<>();
                 boolean enableSslNative = false;
-                boolean inlineBeforeAnalysis = nativeConfig.inlineBeforeAnalysis();
                 boolean addAllCharsets = nativeConfig.addAllCharsets();
                 boolean enableHttpsUrlHandler = nativeConfig.enableHttpsUrlHandler();
                 for (NativeImageSystemPropertyBuildItem prop : nativeImageProperties) {
@@ -735,8 +726,6 @@ public NativeImageInvokerInfo build() {
                                         + " will be removed in a future Quarkus version.");
                     } else if (prop.getKey().equals("quarkus.native.enable-all-charsets") && prop.getValue() != null) {
                         addAllCharsets |= Boolean.parseBoolean(prop.getValue());
-                    } else if (prop.getKey().equals("quarkus.native.inline-before-analysis") && prop.getValue() != null) {
-                        inlineBeforeAnalysis = Boolean.parseBoolean(prop.getValue());
                     } else {
                         // todo maybe just -D is better than -J-D in this case
                         if (prop.getValue() == null) {
@@ -922,29 +911,13 @@ public NativeImageInvokerInfo build() {
                 if (!protocols.isEmpty()) {
                     nativeImageArgs.add("--enable-url-protocols=" + String.join(",", protocols));
                 }
-                if (!inlineBeforeAnalysis) {
-                    addExperimentalVMOption(nativeImageArgs, "-H:-InlineBeforeAnalysis");
-                }
                 if (!pie.isEmpty()) {
                     nativeImageArgs.add("-H:NativeLinkerOption=" + pie);
                 }
 
                 if (!nativeConfig.enableIsolates()) {
                     addExperimentalVMOption(nativeImageArgs, "-H:-SpawnIsolates");
                 }
-                if (!nativeConfig.enableJni()) {
-                    log.warn(
-                            "Your application is setting the deprecated 'quarkus.native.enable-jni' configuration key to false."
-                                    + " Please consider removing this configuration key as it is ignored (JNI is always enabled) and it"
-                                    + " will be removed in a future Quarkus version.");
-                }
-                if (nativeConfig.enableServer()) {
-                    log.warn(
-                            "Your application is setting the deprecated 'quarkus.native.enable-server' configuration key to true."
-                                    + " Please consider removing this configuration key as it is ignored"
-                                    + " (The Native image build server is always disabled) and it"
-                                    + " will be removed in a future Quarkus version.");
-                }
                 if (nativeConfig.enableVmInspection()) {
                     addExperimentalVMOption(nativeImageArgs, "-H:+AllowVMInspection");
                 }
@@ -1000,14 +973,6 @@ public NativeImageInvokerInfo build() {
                 } else {
                     addExperimentalVMOption(nativeImageArgs, "-H:-UseServiceLoaderFeature");
                 }
-                // This option has no effect on GraalVM 23.1+
-                if (graalVMVersion.compareTo(GraalVM.Version.VERSION_23_1_0) < 0) {
-                    if (nativeConfig.fullStackTraces()) {
-                        nativeImageArgs.add("-H:+StackTrace");
-                    } else {
-                        nativeImageArgs.add("-H:-StackTrace");
-                    }
-                }
 
                 if (nativeConfig.enableDashboardDump()) {
                     addExperimentalVMOption(nativeImageArgs,
diff --git a/core/deployment/src/test/java/io/quarkus/deployment/pkg/TestNativeConfig.java b/core/deployment/src/test/java/io/quarkus/deployment/pkg/TestNativeConfig.java
@@ -50,21 +50,6 @@ public boolean enableHttpsUrlHandler() {
         return false;
     }
 
-    @Override
-    public boolean enableAllSecurityServices() {
-        return false;
-    }
-
-    @Override
-    public boolean inlineBeforeAnalysis() {
-        return false;
-    }
-
-    @Override
-    public boolean enableJni() {
-        return false;
-    }
-
     @Override
     public boolean headless() {
         return false;
@@ -115,11 +100,6 @@ public boolean publishDebugBuildProcessPort() {
         return false;
     }
 
-    @Override
-    public boolean cleanupServer() {
-        return false;
-    }
-
     @Override
     public boolean enableIsolates() {
         return false;
@@ -130,11 +110,6 @@ public boolean enableFallbackImages() {
         return false;
     }
 
-    @Override
-    public boolean enableServer() {
-        return false;
-    }
-
     @Override
     public boolean autoServiceLoaderRegistration() {
         return false;
@@ -190,11 +165,6 @@ public Optional<List<MonitoringOption>> monitoring() {
         return Optional.empty();
     }
 
-    @Override
-    public boolean fullStackTraces() {
-        return false;
-    }
-
     @Override
     public boolean enableReports() {
         return false;
