This is an automated email from the ASF dual-hosted git repository.

huijun pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-heron.git


The following commit(s) were added to refs/heads/master by this push:
     new 7e011f7  refactor downlaoder build (#2911)
7e011f7 is described below

commit 7e011f73acf1367cc4e8e566466f287859c03ee6
Author: bed debug <[email protected]>
AuthorDate: Wed May 30 18:04:06 2018 -0700

    refactor downlaoder build (#2911)
---
 heron/downloaders/src/java/BUILD | 33 +++++++++++++++++++++++----------
 1 file changed, 23 insertions(+), 10 deletions(-)

diff --git a/heron/downloaders/src/java/BUILD b/heron/downloaders/src/java/BUILD
index 63c5945..c819ecf 100644
--- a/heron/downloaders/src/java/BUILD
+++ b/heron/downloaders/src/java/BUILD
@@ -1,18 +1,36 @@
+licenses(["notice"])
+
 package(default_visibility = ["//visibility:public"])
 
+files = glob(
+  ["**/*.java"],
+  exclude = ["**/DownloadRunner.java"],
+)
+
 downloader_deps = [
-  "@org_apache_distributedlog_core//jar",
-  "@commons_cli_commons_cli//jar",
-  "//heron/spi/src/java:common-spi-java",
   "//heron/io/dlog/src/java:dlog-lib",
+  "//heron/spi/src/java:common-spi-java",
   "//third_party/java:commons-compress",
+  "@commons_cli_commons_cli//jar",
+  "@org_apache_distributedlog_core//jar",
 ]
 
+java_library(
+  name = "downloader-java",
+  srcs = files,
+  deps = downloader_deps,
+)
+
+filegroup(
+  name = "downloader-main",
+  srcs = glob(["**/DownloadRunner.java"]),
+)
+
 java_binary(
   name = 'heron-downloader-unshaded',
-  srcs = glob(["**/downloader/*.java"]),
+  srcs = [":downloader-main"],
   main_class = "org.apache.heron.downloader.DownloadRunner",
-  deps = downloader_deps,
+  deps = downloader_deps + [":downloader-java"],
 )
 
 genrule(
@@ -21,8 +39,3 @@ genrule(
   outs = ["heron-downloader.jar"],
   cmd  = "cp $< $@",
 )
-
-filegroup(
-  name = "exported-downloader-java-src",
-  srcs = glob(["**/*.java"]),
-)

-- 
To stop receiving notification emails like this one, please contact
[email protected].

Reply via email to