Skip to content

Commit 3858f60

Browse files
committed
Revert workaround from rc1
1 parent e748280 commit 3858f60

File tree

3 files changed

+4
-8
lines changed

3 files changed

+4
-8
lines changed

build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/EmbeddedProviderExtension.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,8 +39,7 @@ void impl(String implName, Project implProject) {
3939
String capitalName = capitalize(projectName);
4040

4141
Dependency implDependency = project.getDependencies().create(implProject);
42-
Configuration implConfig = project.getConfigurations().detachedConfiguration();
43-
implConfig.getDependencies().add(implDependency);
42+
Configuration implConfig = project.getConfigurations().detachedConfiguration(implDependency);
4443
implConfig.attributes(attrs -> {
4544
attrs.attribute(ARTIFACT_TYPE_ATTRIBUTE, DIRECTORY_TYPE);
4645
attrs.attribute(EmbeddedProviderPlugin.IMPL_ATTR, true);

build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/distribution/ElasticsearchDistributionExtension.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,7 @@ public ElasticsearchDistributionExtension(Project project) {
3434
private Configuration moduleZip(Project module) {
3535
var moduleConfigurationCoords = Map.of("path", module.getPath(), "configuration", EXPLODED_BUNDLE_CONFIG);
3636
var dep = project.getDependencies().project(moduleConfigurationCoords);
37-
Configuration detachedConfiguration = project.getConfigurations().detachedConfiguration();
38-
detachedConfiguration.getDependencies().add(dep);
37+
Configuration detachedConfiguration = project.getConfigurations().detachedConfiguration(dep);
3938
return detachedConfiguration;
4039
}
4140

build-tools/src/main/java/org/elasticsearch/gradle/testclusters/ElasticsearchCluster.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -247,8 +247,7 @@ public void setTestDistribution(TestDistribution distribution) {
247247

248248
private void registerExtractedConfig(Provider<RegularFile> pluginProvider) {
249249
Dependency pluginDependency = this.project.getDependencies().create(project.files(pluginProvider));
250-
Configuration extractedConfig = project.getConfigurations().detachedConfiguration();
251-
extractedConfig.getDependencies().add(pluginDependency);
250+
Configuration extractedConfig = project.getConfigurations().detachedConfiguration(pluginDependency);
252251
extractedConfig.getAttributes().attribute(ArtifactTypeDefinition.ARTIFACT_TYPE_ATTRIBUTE, ArtifactTypeDefinition.DIRECTORY_TYPE);
253252
extractedConfig.getAttributes().attribute(BUNDLE_ATTRIBUTE, true);
254253
pluginAndModuleConfiguration.from(extractedConfig);
@@ -297,8 +296,7 @@ Collection<Configuration> getPluginAndModuleConfigurations() {
297296
private Provider<RegularFile> maybeCreatePluginOrModuleDependency(String path, String consumingConfiguration) {
298297
var configuration = pluginAndModuleConfigurations.computeIfAbsent(path, key -> {
299298
var bundleDependency = this.project.getDependencies().project(Map.of("path", path, "configuration", consumingConfiguration));
300-
Configuration detachedConfiguration = project.getConfigurations().detachedConfiguration();
301-
detachedConfiguration.getDependencies().add(bundleDependency);
299+
Configuration detachedConfiguration = project.getConfigurations().detachedConfiguration(bundleDependency);
302300
return detachedConfiguration;
303301
});
304302

0 commit comments

Comments
 (0)