You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by lc...@apache.org on 2017/11/17 22:29:43 UTC

[12/50] [abbrv] beam git commit: Add a task dependency visualization plugin. Resolve a spurious error that gogradle was not adding a task dependency between resolving build dependencies and building the dependency

Add a task dependency visualization plugin.
Resolve a spurious error that gogradle was not adding a task dependency between resolving build dependencies and building the dependency


Project: http://git-wip-us.apache.org/repos/asf/beam/repo
Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/cd713b16
Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/cd713b16
Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/cd713b16

Branch: refs/heads/master
Commit: cd713b169abf07a7377eae91c58de45989f1f1e7
Parents: 31785b3
Author: Lukasz Cwik <lc...@google.com>
Authored: Sat Nov 4 00:03:24 2017 -0700
Committer: Luke Cwik <lc...@google.com>
Committed: Fri Nov 17 14:27:15 2017 -0800

----------------------------------------------------------------------
 build.gradle                       | 1 +
 build_rules.gradle                 | 6 +++++-
 sdks/python/container/build.gradle | 5 +++++
 3 files changed, 11 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/beam/blob/cd713b16/build.gradle
----------------------------------------------------------------------
diff --git a/build.gradle b/build.gradle
index f8cefa5..14d9bc5 100644
--- a/build.gradle
+++ b/build.gradle
@@ -151,6 +151,7 @@ buildscript {
     classpath "com.diffplug.spotless:spotless-plugin-gradle:3.6.0"
     classpath "gradle.plugin.com.github.blindpirate:gogradle:0.7.0"
     classpath "gradle.plugin.com.palantir.gradle.docker:gradle-docker:0.13.0"
+    classpath "cz.malohlava:visteg:1.0.3"
   }
 }
 

http://git-wip-us.apache.org/repos/asf/beam/blob/cd713b16/build_rules.gradle
----------------------------------------------------------------------
diff --git a/build_rules.gradle b/build_rules.gradle
index 37b93a8..7155ef5 100644
--- a/build_rules.gradle
+++ b/build_rules.gradle
@@ -45,7 +45,11 @@ repositories {
 task allDeps(type: DependencyReportTask) {
 }
 
-apply plugin: 'project-report'
+apply plugin: "project-report"
+
+// Apply a task dependency visualization plugin which creates a ".dot" file in the build directory
+// giving the task dependencies for that project
+apply plugin: "cz.malohlava.visteg"
 
 /*************************************************************************************************/
 

http://git-wip-us.apache.org/repos/asf/beam/blob/cd713b16/sdks/python/container/build.gradle
----------------------------------------------------------------------
diff --git a/sdks/python/container/build.gradle b/sdks/python/container/build.gradle
index eb201b4..0d091d4 100644
--- a/sdks/python/container/build.gradle
+++ b/sdks/python/container/build.gradle
@@ -22,6 +22,11 @@ applyDockerNature()
 
 description = "Apache Beam :: SDKs :: Python :: Container"
 
+// Figure out why the golang plugin does not add a build dependency between projects.
+// Without the line below, we get spurious errors about not being able to resolve
+// "./github.com/apache/beam/sdks/go"
+resolveBuildDependencies.dependsOn ":beam-sdks-parent:beam-sdks-go:build"
+
 dependencies {
     golang {
         // TODO(herohde): use "./" prefix to prevent gogradle use base github path, for now.