opendev/gerrit
Code Issues Proposed changes

ElasticTestUtils needs dependency to "//gerrit-reviewdb:client"

Browse Source 
The Gerrit build of the Bazel team fails with:

ERROR: /home/ci/workspace/gerrit/BAZEL_VERSION/HEAD/PLATFORM_NAME/linux-x86_64/gerrit-elasticsearch/BUILD:32:1:
Java compilation in rule '//gerrit-elasticsearch:elasticsearch_test_utils' failed:
Worker process sent response with exit code: 1..
gerrit-elasticsearch/src/test/java/com/google/gerrit/elasticsearch/ElasticTestUtils.java:162:
error: [strict] Using type com.google.gerrit.reviewdb.client.AccountGroup from an indirect
dependency (TOOL_INFO: "//gerrit-reviewdb:client"). See command below **
    Schema<AccountGroup> groupSchema =
           ^
1 error

Change-Id: I6e3a9988becf087d9b0f9bca6c04721daa4468fc
Signed-off-by: Edwin Kempin <ekempin@google.com>
This commit is contained in:
Edwin Kempin
2017-01-12 13:46:44 +01:00
parent 261ffcf84c
commit e6089ad9cf
2 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
gerrit-elasticsearch/BUCK
View File

@@ -34,6 +34,7 @@ java_test(
deps = [ deps = [
':elasticsearch', ':elasticsearch',
'//gerrit-extension-api:api', '//gerrit-extension-api:api',
'//gerrit-reviewdb:client',
'//gerrit-reviewdb:server', '//gerrit-reviewdb:server',
'//gerrit-server:server', '//gerrit-server:server',
'//gerrit-server:testutil', '//gerrit-server:testutil',

1
gerrit-elasticsearch/BUILD
View File

@@ -36,6 +36,7 @@ java_library(
deps = [ deps = [
":elasticsearch", ":elasticsearch",
"//gerrit-extension-api:api", "//gerrit-extension-api:api",
"//gerrit-reviewdb:client",
"//gerrit-server:server", "//gerrit-server:server",
"//lib:gson", "//lib:gson",
"//lib:guava", "//lib:guava",