Fix build

aws · Oct 27, 2022 · 9bcb3e6 · 9bcb3e6
1 parent 8a92319
commit 9bcb3e6
Show file tree

Hide file tree

Showing 8 changed files with 40 additions and 23 deletions.
diff --git a/codegen-lite-maven-plugin/pom.xml b/codegen-lite-maven-plugin/pom.xml
@@ -74,10 +74,6 @@
             <artifactId>maven-project</artifactId>
             <groupId>org.apache.maven</groupId>
         </dependency>
-        <dependency>
-            <artifactId>plexus-utils</artifactId>
-            <groupId>org.codehaus.plexus</groupId>
-        </dependency>
     </dependencies>
 
     <build>

diff --git a/codegen-maven-plugin/pom.xml b/codegen-maven-plugin/pom.xml
@@ -79,10 +79,6 @@
             <artifactId>maven-project</artifactId>
             <groupId>org.apache.maven</groupId>
         </dependency>
-        <dependency>
-            <artifactId>plexus-utils</artifactId>
-            <groupId>org.codehaus.plexus</groupId>
-        </dependency>
     </dependencies>
 
     <build>

diff --git a/...src/main/java/software/amazon/awssdk/codegen/poet/builder/BaseClientBuilderInterface.java b/...src/main/java/software/amazon/awssdk/codegen/poet/builder/BaseClientBuilderInterface.java
@@ -73,11 +73,14 @@ public TypeSpec poetSpec() {
             builder.addMethod(serviceConfigurationConsumerBuilderMethod());
         }
 
-        if (hasClientContextParams() && endpointRulesSpecUtils.isEndpointRulesEnabled()) {
+        if (endpointRulesSpecUtils.isEndpointRulesEnabled()) {
             builder.addMethod(endpointProviderMethod());
-            model.getClientContextParams().forEach((n, m) -> {
-                builder.addMethod(clientContextParamSetter(n, m));
-            });
+
+            if (hasClientContextParams()) {
+                model.getClientContextParams().forEach((n, m) -> {
+                    builder.addMethod(clientContextParamSetter(n, m));
+                });
+            }
         }
 
         if (generateTokenProviderMethod()) {

diff --git a/pom.xml b/pom.xml
@@ -489,7 +489,9 @@
                         <ignoredUnusedDeclaredDependency>org.junit.vintage:*</ignoredUnusedDeclaredDependency>
                         <ignoredUnusedDeclaredDependency>software.amazon.awssdk:aws-sdk-java</ignoredUnusedDeclaredDependency>
                         <!-- Declared by the codegen maven plugins (lite and normal). Not used directly but used to override a transitive dependecy -->
-                        <ignoredUnusedDeclaredDependency>org.codehaus.plexus:plexus-utils</ignoredUnusedDeclaredDependency>
+                        <ignoredUnusedDeclaredDependency>software.amazon.awssdk:json-utils</ignoredUnusedDeclaredDependency>
+                        <ignoredUnusedDeclaredDependency>software.amazon.awssdk:endpoints-spi</ignoredUnusedDeclaredDependency>
+                        <ignoredUnusedDeclaredDependency>software.amazon.awssdk:aws-sdk-java</ignoredUnusedDeclaredDependency>
                     </ignoredUnusedDeclaredDependencies>
                 </configuration>
             </plugin>

diff --git a/services/eventbridge/pom.xml b/services/eventbridge/pom.xml
@@ -56,6 +56,16 @@
             <artifactId>aws-json-protocol</artifactId>
             <version>${awsjavasdk.version}</version>
         </dependency>
+        <dependency>
+            <groupId>software.amazon.awssdk</groupId>
+            <artifactId>json-utils</artifactId>
+            <version>${awsjavasdk.version}</version>
+        </dependency>
+        <dependency>
+            <groupId>software.amazon.awssdk</groupId>
+            <artifactId>endpoints-spi</artifactId>
+            <version>${awsjavasdk.version}</version>
+        </dependency>
         <!-- Test Dependencies -->
         <dependency>
             <groupId>software.amazon.awssdk</groupId>

diff --git a/services/pom.xml b/services/pom.xml
@@ -368,11 +368,6 @@
             <artifactId>http-client-spi</artifactId>
             <version>${awsjavasdk.version}</version>
         </dependency>
-        <dependency>
-            <groupId>software.amazon.awssdk</groupId>
-            <artifactId>endpoints-spi</artifactId>
-            <version>${awsjavasdk.version}</version>
-        </dependency>
         <dependency>
             <groupId>software.amazon.awssdk</groupId>
             <artifactId>regions</artifactId>
@@ -388,11 +383,6 @@
             <artifactId>utils</artifactId>
             <version>${awsjavasdk.version}</version>
         </dependency>
-        <dependency>
-            <groupId>software.amazon.awssdk</groupId>
-            <artifactId>json-utils</artifactId>
-            <version>${awsjavasdk.version}</version>
-        </dependency>
         <dependency>
             <artifactId>aws-core</artifactId>
             <groupId>software.amazon.awssdk</groupId>

diff --git a/services/s3/pom.xml b/services/s3/pom.xml
@@ -75,6 +75,16 @@
             <artifactId>profiles</artifactId>
             <version>${awsjavasdk.version}</version>
         </dependency>
+        <dependency>
+            <groupId>software.amazon.awssdk</groupId>
+            <artifactId>json-utils</artifactId>
+            <version>${awsjavasdk.version}</version>
+        </dependency>
+        <dependency>
+            <groupId>software.amazon.awssdk</groupId>
+            <artifactId>endpoints-spi</artifactId>
+            <version>${awsjavasdk.version}</version>
+        </dependency>
         <dependency>
             <groupId>software.amazon.awssdk.crt</groupId>
             <artifactId>aws-crt</artifactId>

diff --git a/services/s3control/pom.xml b/services/s3control/pom.xml
@@ -71,6 +71,16 @@
             <artifactId>profiles</artifactId>
             <version>${awsjavasdk.version}</version>
         </dependency>
+        <dependency>
+            <groupId>software.amazon.awssdk</groupId>
+            <artifactId>json-utils</artifactId>
+            <version>${awsjavasdk.version}</version>
+        </dependency>
+        <dependency>
+            <groupId>software.amazon.awssdk</groupId>
+            <artifactId>endpoints-spi</artifactId>
+            <version>${awsjavasdk.version}</version>
+        </dependency>
         <!-- Test Dependencies -->
         <dependency>
             <artifactId>commons-io</artifactId>