Skip to content

Commit

Permalink
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Remove config customizer
Browse files Browse the repository at this point in the history
This is no longer used anywhere, so let's remove it

Resolves: quarkusio#5332
geoand committed Jan 11, 2022

Verified

This commit was signed with the committer’s verified signature.
maksim-paskal Maksim Paskal
1 parent 8eadd79 commit 2fb4c7f
Showing 2 changed files with 2 additions and 14 deletions.
Original file line number Diff line number Diff line change
@@ -128,8 +128,7 @@ private static boolean isRecorder(AnnotatedElement element) {
* @throws ClassNotFoundException if a build step class is not found
*/
public static Consumer<BuildChainBuilder> loadStepsFrom(ClassLoader classLoader, Properties buildSystemProps,
ApplicationModel appModel, LaunchMode launchMode, DevModeType devModeType,
Consumer<ConfigBuilder> configCustomizer)
ApplicationModel appModel, LaunchMode launchMode, DevModeType devModeType)
throws IOException, ClassNotFoundException {
// populate with all known types
List<Class<?>> roots = new ArrayList<>();
@@ -167,9 +166,6 @@ public static Consumer<BuildChainBuilder> loadStepsFrom(ClassLoader classLoader,
builder.withSources(ds1, ds2, platformConfigSource, pcs);
}

if (configCustomizer != null) {
configCustomizer.accept(builder);
}
final SmallRyeConfig src = builder.build();

// install globally
Original file line number Diff line number Diff line change
@@ -57,7 +57,6 @@ public class QuarkusAugmentor {
private final Path targetDir;
private final ApplicationModel effectiveModel;
private final String baseName;
private final Consumer<ConfigBuilder> configCustomizer;
private final boolean rebuild;
private final boolean auxiliaryApplication;
private final Optional<DevModeType> auxiliaryDevModeType;
@@ -76,7 +75,6 @@ public class QuarkusAugmentor {
this.targetDir = builder.targetDir;
this.effectiveModel = builder.effectiveModel;
this.baseName = builder.baseName;
this.configCustomizer = builder.configCustomizer;
this.deploymentClassLoader = builder.deploymentClassLoader;
this.rebuild = builder.rebuild;
this.devModeType = builder.devModeType;
@@ -106,7 +104,7 @@ public BuildResult run() throws Exception {
//in additional stuff from the deployment leaking in, this is unlikely but has a bit of a smell.
ExtensionLoader.loadStepsFrom(deploymentClassLoader,
buildSystemProperties == null ? new Properties() : buildSystemProperties,
effectiveModel, launchMode, devModeType, configCustomizer)
effectiveModel, launchMode, devModeType)
.accept(chainBuilder);

Thread.currentThread().setContextClassLoader(classLoader);
@@ -202,7 +200,6 @@ public static final class Builder {

ApplicationModel effectiveModel;
String baseName = "quarkus-application";
Consumer<ConfigBuilder> configCustomizer;
ClassLoader deploymentClassLoader;
DevModeType devModeType;
boolean test;
@@ -341,10 +338,5 @@ public Builder setDeploymentClassLoader(ClassLoader deploymentClassLoader) {
this.deploymentClassLoader = deploymentClassLoader;
return this;
}

public Builder setConfigCustomizer(Consumer<ConfigBuilder> configCustomizer) {
this.configCustomizer = configCustomizer;
return this;
}
}
}

0 comments on commit 2fb4c7f

Please sign in to comment.