vikrambohra commented on a change in pull request #3087:
URL: https://github.com/apache/incubator-gobblin/pull/3087#discussion_r474405733



##########
File path: gobblin-service/build.gradle
##########
@@ -91,12 +91,27 @@ dependencies {
 // Begin HACK to get around POM being depenendent on the (empty) 
gobblin-rest-api instead of gobblin-rest-api-rest-client
 def installer = install.repositories.mavenInstaller
 [installer]*.pom*.whenConfigured {pom ->
-    pom.dependencies.find {dep -> dep.groupId == project.group && 
dep.artifactId == 'gobblin-flow-config-service-api' }.artifactId = 
'gobblin-flow-config-service-api-rest-client'
+  def it = pom.dependencies
+  def s = it.find {dep -> dep.groupId == project.group && dep.artifactId == 
'gobblin-flow-config-service-api' }
+  if (s != null) {
+    def t = s.clone()
+    t.artifactId = 'gobblin-flow-config-service-api-data-template'
+    it.add(t)
+    s.artifactId = 'gobblin-flow-config-service-api-rest-client'
+  }
 }
+
 if (rootProject.publishToMaven || rootProject.publishToNexus) {
     def deployer = uploadArchives.repositories.mavenDeployer
     [deployer]*.pom*.whenConfigured {pom ->
-        pom.dependencies.find {dep -> dep.groupId == project.group && 
dep.artifactId == 'gobblin-flow-config-service-api' }.artifactId = 
'gobblin-flow-config-service-api-rest-client'
+      def it = pom.dependencies
+      def s = it.find {dep -> dep.groupId == project.group && dep.artifactId 
== 'gobblin-flow-config-service-api' }
+      if (s != null) {
+        def t = s.clone()
+        t.artifactId = 'gobblin-flow-config-service-api-data-template'

Review comment:
       good call adding gobblin-flow-config-service-api also. 




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to