0b774c478e
This version fixed a major issue: [1] that was a reason of frustration
of many plugin developers: Not cache sources files under symbolic link.
Now for all such source files, the warning is issued:
"
Disabling caching for target //plugins/wip:wip__plugin, because one or
more input files are under a symbolic link
({plugins/wip=/home/davido/projects/wip}). This will severely impact
performance! To resolve this, use separate rules and declare
dependencies instead of using symbolic links.
"
To suppress this warning we add project.allow_symlink option. This
doesn't have any impact for gerrit core but silences the warning above
when plugins are built in gerrit tree mode.
As pointed out in this issue: [2], we are using some artifacts as source
to the java_library() rule as well as binary_jar for prebuilt_ja rule.
To avoid the warning, we rename sources to have "-sources.jar" suffix
and we rename *.zip to end with .jar in other places.
"
Assuming edit.src.zip is a JAR and renaming to edit.src.jar in
//gerrit-patch-jgit:edit_src. Change the extension of the binary_jar to
'.jar' to remove this warning.
"
source_under_test attribute was removed from java_test() rule.
Replication and cookbook-plugin are updated as well.
local.properties support was removed, but we use it only for download
process customization in our own python script, so that we can keep it
usage and not need to move it to .buckconfig.local.
[1] https://github.com/facebook/buck/issues/341
[2] https://github.com/facebook/buck/issues/855
Change-Id: Idf76cc71c21df43e808179b645f9175767b322a8
40 lines
832 B
Python
40 lines
832 B
Python
java_library(
|
|
name = 'http',
|
|
srcs = glob(['src/main/java/**/*.java']),
|
|
provided_deps = ['//lib:servlet-api-3_1'],
|
|
visibility = ['PUBLIC'],
|
|
)
|
|
|
|
TESTUTIL_SRCS = glob(['src/test/**/testutil/**/*.java'])
|
|
|
|
java_library(
|
|
name = 'testutil',
|
|
srcs = TESTUTIL_SRCS,
|
|
deps = [
|
|
'//gerrit-extension-api:api',
|
|
'//lib:guava',
|
|
'//lib:servlet-api-3_1',
|
|
'//lib/httpcomponents:httpclient',
|
|
'//lib/jgit/org.eclipse.jgit:jgit',
|
|
],
|
|
visibility = ['PUBLIC'],
|
|
)
|
|
|
|
java_test(
|
|
name = 'http_tests',
|
|
srcs = glob(
|
|
['src/test/java/**/*.java'],
|
|
excludes = TESTUTIL_SRCS,
|
|
),
|
|
deps = [
|
|
':http',
|
|
':testutil',
|
|
'//lib:junit',
|
|
'//lib:servlet-api-3_1',
|
|
'//lib:truth',
|
|
'//lib/easymock:easymock',
|
|
],
|
|
# TODO(sop) Remove after Buck supports Eclipse
|
|
visibility = ['//tools/eclipse:classpath'],
|
|
)
|