gerrit-acceptance-framework: Simplify build rules

Change-Id: I7000f5ebd569c5c3a5d5bed2da9e65d267a8b886
This commit is contained in:
David Ostrovsky
2016-10-27 08:23:55 +02:00
parent 1e15271ad3
commit 98f906b8af
2 changed files with 38 additions and 44 deletions

View File

@@ -1,25 +1,5 @@
SRCS = glob(['src/test/java/com/google/gerrit/acceptance/*.java']) SRCS = glob(['src/test/java/com/google/gerrit/acceptance/*.java'])
DEPS = [
'//gerrit-antlr:query_exception',
'//gerrit-gpg:gpg',
'//gerrit-launcher:launcher',
'//gerrit-openid:openid',
'//gerrit-pgm:daemon',
'//gerrit-pgm:http-jetty',
'//gerrit-pgm:util-nodep',
'//gerrit-server/src/main/prolog:common',
'//gerrit-server:testutil',
'//lib/auto:auto-value',
'//lib/httpcomponents:fluent-hc',
'//lib/httpcomponents:httpclient',
'//lib/httpcomponents:httpcore',
'//lib/jetty:servlet',
'//lib/jgit/org.eclipse.jgit.junit:junit',
'//lib/log:impl_log4j',
'//lib/log:log4j',
]
PROVIDED = [ PROVIDED = [
'//gerrit-common:annotations', '//gerrit-common:annotations',
'//gerrit-common:server', '//gerrit-common:server',
@@ -55,7 +35,24 @@ genrule(
java_library( java_library(
name = 'lib', name = 'lib',
srcs = SRCS, srcs = SRCS,
exported_deps = DEPS + [ exported_deps = [
'//gerrit-antlr:query_exception',
'//gerrit-gpg:gpg',
'//gerrit-launcher:launcher',
'//gerrit-openid:openid',
'//gerrit-pgm:daemon',
'//gerrit-pgm:http-jetty',
'//gerrit-pgm:util-nodep',
'//gerrit-server/src/main/prolog:common',
'//gerrit-server:testutil',
'//lib/auto:auto-value',
'//lib/httpcomponents:fluent-hc',
'//lib/httpcomponents:httpclient',
'//lib/httpcomponents:httpcore',
'//lib/jetty:servlet',
'//lib/jgit/org.eclipse.jgit.junit:junit',
'//lib/log:impl_log4j',
'//lib/log:log4j',
'//lib:truth', '//lib:truth',
], ],
provided_deps = PROVIDED + [ provided_deps = PROVIDED + [
@@ -79,7 +76,8 @@ java_doc(
pkgs = [' com.google.gerrit.acceptance'], pkgs = [' com.google.gerrit.acceptance'],
source_jar = ':acceptance-framework-src', source_jar = ':acceptance-framework-src',
srcs = SRCS, srcs = SRCS,
deps = DEPS + PROVIDED + [ deps = PROVIDED + [
':lib',
'//lib:guava', '//lib:guava',
'//lib/guice:guice-assistedinject', '//lib/guice:guice-assistedinject',
'//lib/guice:guice_library', '//lib/guice:guice_library',
@@ -87,7 +85,6 @@ java_doc(
'//lib/guice:javax-inject', '//lib/guice:javax-inject',
'//lib:gwtorm_client', '//lib:gwtorm_client',
'//lib:junit', '//lib:junit',
'//lib:truth',
], ],
visibility = ['PUBLIC'], visibility = ['PUBLIC'],
) )

View File

@@ -2,26 +2,6 @@ load('//tools/bzl:java.bzl', 'java_library2')
SRCS = glob(['src/test/java/com/google/gerrit/acceptance/*.java']) SRCS = glob(['src/test/java/com/google/gerrit/acceptance/*.java'])
DEPS = [
'//gerrit-antlr:query_exception',
'//gerrit-gpg:gpg',
'//gerrit-launcher:launcher',
'//gerrit-openid:openid',
'//gerrit-pgm:daemon',
'//gerrit-pgm:http-jetty',
'//gerrit-pgm:util-nodep',
'//gerrit-server/src/main/prolog:common',
'//gerrit-server:testutil',
'//lib/auto:auto-value',
'//lib/httpcomponents:fluent-hc',
'//lib/httpcomponents:httpclient',
'//lib/httpcomponents:httpcore',
'//lib/jetty:servlet',
'//lib/jgit/org.eclipse.jgit.junit:junit',
'//lib/log:impl_log4j',
'//lib/log:log4j',
]
PROVIDED = [ PROVIDED = [
'//gerrit-common:annotations', '//gerrit-common:annotations',
'//gerrit-common:server', '//gerrit-common:server',
@@ -48,7 +28,24 @@ java_binary(
java_library2( java_library2(
name = 'lib', name = 'lib',
srcs = SRCS, srcs = SRCS,
exported_deps = DEPS + [ exported_deps = [
'//gerrit-antlr:query_exception',
'//gerrit-gpg:gpg',
'//gerrit-launcher:launcher',
'//gerrit-openid:openid',
'//gerrit-pgm:daemon',
'//gerrit-pgm:http-jetty',
'//gerrit-pgm:util-nodep',
'//gerrit-server/src/main/prolog:common',
'//gerrit-server:testutil',
'//lib/auto:auto-value',
'//lib/httpcomponents:fluent-hc',
'//lib/httpcomponents:httpclient',
'//lib/httpcomponents:httpcore',
'//lib/jetty:servlet',
'//lib/jgit/org.eclipse.jgit.junit:junit',
'//lib/log:impl_log4j',
'//lib/log:log4j',
'//lib:truth', '//lib:truth',
], ],
deps = PROVIDED + [ # We want these deps to be exported_deps deps = PROVIDED + [ # We want these deps to be exported_deps