aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/java_tools
Commit message (Collapse)AuthorAge
* Fix cycle in dependency graph during bootstrapGravatar Damien Martin-Guillerez2016-12-20
| | | | | | | | | | | | | | | | This cycle was due to JacocoCoverage_deploy.jar being an implicit dependency of java_binary. We do not need the binary during bootstrap because we are not running with coverage but we need the target. Simply move it to exists as a java_import in tools/jdk. Fixes #2269. To be cherry-picked for #2246. -- Change-Id: I63aee0e76b305833ac0c0c2cae5abe1cbad5e6d1 Reviewed-on: https://cr.bazel.build/8036 PiperOrigin-RevId: 142537244 MOS_MIGRATED_REVID=142537244
* Fix genclass' handling of class names that contain '$'Gravatar Liam Miller-Cushon2016-12-19
| | | | | | | | | | Previously it was assuming '$' was only present for inner classes, but that isn't the case. This was causing problems specifically for AutoValue extensions, which generate class names that start with '$'. -- PiperOrigin-RevId: 142468150 MOS_MIGRATED_REVID=142468150
* Initial code for Persistent Java Test Runner.Gravatar Kush Chakraborty2016-12-09
| | | | | | | | | | | | At this point this does nothing more than re-run the exact same test without having to re-start the test runner. In future iterations the aim is to be able to re-run tests with modified code, without having to re-start the test runner. To test out the WorkerTestStrategy simply use --test_strategy=experimental_worker for a test with bazel. -- PiperOrigin-RevId: 141465929 MOS_MIGRATED_REVID=141465929
* Update to javac9-r3297Gravatar Liam Miller-Cushon2016-12-08
| | | | | | -- PiperOrigin-RevId: 141396618 MOS_MIGRATED_REVID=141396618
* Description redacted.Gravatar Kristina Chodorow2016-12-07
| | | | | | -- PiperOrigin-RevId: 141325412 MOS_MIGRATED_REVID=141325412
* Add JacocoCoverageRunner to junitrunner.Gravatar Yue Gan2016-12-05
| | | | | | | | (series 3/4 of open-sourcing coverage command for java test) -- PiperOrigin-RevId: 141046146 MOS_MIGRATED_REVID=141046146
* Add Jacoco related code to buildjar.Gravatar Yue Gan2016-12-01
| | | | | | | (series 1/4 of open-sourcing coverage command for java test) -- MOS_MIGRATED_REVID=140723068
* Fix a typoGravatar Liam Miller-Cushon2016-12-01
| | | | | | | follow-up to commit 1f4ee384922f63b3132cb98a6bc950b3fe65e8ca -- MOS_MIGRATED_REVID=140621947
* Hard-code `-parameters` and `-source 8 -target 8` in javac-turbineGravatar Liam Miller-Cushon2016-11-30
| | | | | | | | This makes its behaviour consistent with regular turbine, avoiding skew when switching between implementations. -- MOS_MIGRATED_REVID=140547517
* Removing Dagger dependency from junitrunner.Gravatar Irina Iancu2016-11-24
| | | | | -- MOS_MIGRATED_REVID=140132183
* Have Bazel use the the prebuilt guava jars from the 21.0-20161101 daily ↵Gravatar Nathan Harmata2016-11-21
| | | | | | | build. See https://bazel-review.googlesource.com/#/c/7270/ for details. -- MOS_MIGRATED_REVID=139580987
* Roll forward commit 4e8e67bcf25ea4e06c16104fc7f4db528a16508fGravatar Liam Miller-Cushon2016-11-17
| | | | | -- MOS_MIGRATED_REVID=139399869
* Fix a javac-turbine tree pruning bugGravatar Liam Miller-Cushon2016-11-17
| | | | | | | | | | | | When rewriting blocks to `{ throws new AssertionError(); }`, use a fully-qualified name for `java.lang.AssertionError` in case we're inside the declaration of a type with the same simple name. Also, only add the throws expression statement when rewriting lambda body blocks, since that's the only place it matters. -- MOS_MIGRATED_REVID=139397324
* Rollback of commit 06ded1e91c1a3754e21d2782e8bfd1c5cd4f5929.Gravatar Kristina Chodorow2016-11-16
| | | | | | | | | | | | | *** Reason for rollback *** Lambda expression doesn't work with JDK7, which Bazel still supports *** Original change description *** Fix strict deps handling of the default package. -- MOS_MIGRATED_REVID=139327789
* Enable strict deps enforcement for importsGravatar Liam Miller-Cushon2016-11-16
| | | | | -- MOS_MIGRATED_REVID=139243894
* Fix strict deps handling of the default package.Gravatar Liam Miller-Cushon2016-11-15
| | | | | -- MOS_MIGRATED_REVID=139159834
* Remove cargo-culted workaround for JDK-6440786Gravatar Liam Miller-Cushon2016-11-15
| | | | | | | | ZipOutputStream has been able to create empty zip files since JDK7: https://bugs.openjdk.java.net/browse/JDK-6440786 -- MOS_MIGRATED_REVID=139089919
* Don't break lambda bodies during tree pruningGravatar Liam Miller-Cushon2016-11-14
| | | | | | | | Rewriting block-bodied lambda bodies into empty blocks breaks breaks overload resolution. -- MOS_MIGRATED_REVID=138936301
* Rollback of commit 33ad37612ab7edc32d5e82c3912acebacaef42dc.Gravatar Irina Iancu2016-11-11
| | | | | | | | | | | | | *** Reason for rollback *** junitrunner/java/com/google/testing/junit/runner/sharding/weighted has an internal reference that is not easy to open-source. For now it makes more sense to roll this back and keep this package just internally. *** Original change description *** Open sourcing junitrunner/java/com/google/testing/junit/runner/sharding/weighted. -- MOS_MIGRATED_REVID=138852305
* Fix tree pruning inside enum constantsGravatar Liam Miller-Cushon2016-11-11
| | | | | -- MOS_MIGRATED_REVID=138831285
* Remove the dependency on AutoValue from the Java test runner so that it ↵Gravatar Lukacs Berki2016-11-09
| | | | | | | | | doesn't conflict with whatever AutoValue version the code under test has. Fixes #2044. -- MOS_MIGRATED_REVID=138618191
* Fall back to the transitive classpath for -Xdoclint:reference errorsGravatar Liam Miller-Cushon2016-11-09
| | | | | | | | -Xlintdoc:reference causes javac to load symbols referenced in javadoc, which may fail with the reduced classpath optimization enabled. -- MOS_MIGRATED_REVID=138529301
* Check for strict deps exemptions at the top levelGravatar Liam Miller-Cushon2016-11-07
| | | | | -- MOS_MIGRATED_REVID=138223170
* Description redacted.Gravatar Googler2016-10-31
| | | | | -- MOS_MIGRATED_REVID=137669164
* Change our URL to bazel.buildGravatar Damien Martin-Guillerez2016-10-28
| | | | | -- MOS_MIGRATED_REVID=137500893
* Global cleanup change.Gravatar Damien Martin-Guillerez2016-10-28
| | | | | -- MOS_MIGRATED_REVID=137495530
* Delete obsolete API in ErrorProneAnalyzerGravatar Liam Miller-Cushon2016-10-28
| | | | | -- MOS_MIGRATED_REVID=137464177
* Move junitrunner fake/stub classes from javatests/c/g/testing/junit/runner ↵Gravatar Googler2016-10-24
| | | | | | | | | | to java/c/g/testing/junit/runner/sharding/testing. Also: - Remove Guava dependencies and fix minor style issues in FakeShardingFilters - Fix lint issues in TestSuite[]ModelTest -- MOS_MIGRATED_REVID=136840327
* Rollback of commit e6191f7209ebf3f67480bc7aa9db5ac58beb2156.Gravatar Damien Martin-Guillerez2016-10-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | *** Reason for rollback *** Roll-forward with fix. Tested with `bazel build src:srcs //src/test/...` using a bootstrapped bazel. Fixes #1923. RELNOTES[INC]: Non test-only targets can no longer depends on test-only targets. *** Original change description *** Automated [] rollback of commit a9f20b0d6459d395444c45cf5e94a899f3443633. *** Reason for rollback *** Broke Bazel CI: ERROR: /home/ci/workspace/Bazel/JAVA_VERSION/1.8/PLATFORM_NAME/linux-x86_64/src/BUILD:284:1: in filegroup rule //src:srcs: non-test target '//src:srcs' depends on testonly target '//src/java_tools/junitrunner/java/com/google/testing/junit/junit4:srcs' and doesn't have testonly attribute set. ERROR: Analysis of target '//scripts/packages:packages' failed; build aborted. http://ci.bazel.io/view/Bazel%20b... *** -- MOS_MIGRATED_REVID=136589937
* Global cleanup change.Gravatar Liam Miller-Cushon2016-10-13
| | | | | -- MOS_MIGRATED_REVID=135942005
* Need to make methods accessible, as currently they are only available to ↵Gravatar Googler2016-10-12
| | | | | | | package. -- MOS_MIGRATED_REVID=135869348
* Remove flag control of turbine bridge pruningGravatar Liam Miller-Cushon2016-10-12
| | | | | -- MOS_MIGRATED_REVID=135810752
* Rollback of commit 51417759a37e5b11003dd773d5c90ddd4c7a6aa7.Gravatar Googler2016-10-11
| | | | | | | | | | | | | | | | | | | | | | | | *** Reason for rollback *** This CL is causing failures in one of our integration tests: //javatests/com/google/security/authorization/sphinx/integration:SimpleUIEditorPageE2eTest_firefox-linux [] link: [] Note: Specifically, it looks like BinStackingShardingFilterFactory.java pointing to the branched RuntimeCost annotation is causing the failures. I tried to change our test to point to the branched RuntimeCost annotation but Target '//third_party/bazel/src/java_tools/junitrunner/java/com/google/testing/junit/runner/sharding/weighted:weighted' is not visible. *** Original change description *** Branched internal RuntimeCost in junitrunner. RuntimeCost class is needed by the package com.google.testing.junit.runner.sharding.weighted. I branched it from the internal code because the weighted package is now open sourced and does not have access to the previous import. This didn't break Bazel before because there was no dependency to the weighted target when building BazelTestRunner. I catched this trying to open-source junitrunner tests. -- MOS_MIGRATED_REVID=135734237
* Move MemoizingRequest from c.g.testing.junit.junit4.runner toGravatar Googler2016-10-11
| | | | | | | | | | | | c.g.testing.junit.runner.internal.junit4 Other minor changes: - reduce visibility of MemoizingRequest.createRunner() - Fix bug in CancellableRequestFactory where createRequest() can be called multiple times -- MOS_MIGRATED_REVID=135722005
* Expunge reference to c.g.common.base.FlagGravatar Liam Miller-Cushon2016-10-07
| | | | | -- MOS_MIGRATED_REVID=135388964
* Put turbine bridge pruning behind a flagGravatar Liam Miller-Cushon2016-10-05
| | | | | -- MOS_MIGRATED_REVID=135171940
* Move some sharable classes from c.g.testing.junit.runner.junit4 toGravatar Googler2016-10-04
| | | | | | | | c.g.testing.junit.junit4.runner.internal or c.g.testing.junit.junit4.runner.internal.junit4 -- MOS_MIGRATED_REVID=135003685
* When add_dep should wrap a missing dependency in a rule (e.g., ↵Gravatar Carmi Grushko2016-09-30
| | | | | | | java_proto_library), pass it the kind of the rule. -- MOS_MIGRATED_REVID=134694958
* Move some RegExTestCaseFilter from c.g.testing.junit.runner.junit4 toGravatar Googler2016-09-30
| | | | | | | c.g.testing.junit.junit4.runner. -- MOS_MIGRATED_REVID=134685188
* Branched internal RuntimeCost in junitrunner.Gravatar Irina Iancu2016-09-29
| | | | | | | | | RuntimeCost class is needed by the package com.google.testing.junit.runner.sharding.weighted. I branched it from the internal code because the weighted package is now open sourced and does not have access to the previous import. This didn't break Bazel before because there was no dependency to the weighted target when building BazelTestRunner. I catched this trying to open-source junitrunner tests. -- MOS_MIGRATED_REVID=134634122
* Remove some Dagger dependencies from the Bazel JUnit test runner.Gravatar Googler2016-09-29
| | | | | -- MOS_MIGRATED_REVID=134546010
* Fix import ordering for files under com/google/testing/junit/runner/junit4Gravatar Googler2016-09-28
| | | | | -- MOS_MIGRATED_REVID=134486526
* Open sourcing ↵Gravatar Irina Iancu2016-09-23
| | | | | | | junitrunner/java/com/google/testing/junit/runner/sharding/weighted. -- MOS_MIGRATED_REVID=134046554
* Bazel to support external links integration. This is so that the tests can ↵Gravatar Googler2016-09-22
| | | | | | | inject custom extra data (link, icons, etc) so that a test method can be associated with an external tool. -- MOS_MIGRATED_REVID=133912980
* Drop bridge methods in turbineGravatar Liam Miller-Cushon2016-09-22
| | | | | -- MOS_MIGRATED_REVID=133899664
* Use -source 8 -target 8 in turbine integration testsGravatar Liam Miller-Cushon2016-09-22
| | | | | -- MOS_MIGRATED_REVID=133894022
* Mark unresolved classes in jdeps as Kind.INCOMPLETE.Gravatar Googler2016-09-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, all classes from the symbol table are added to jdeps, even if some of them haven't been resolved and as such aren't actually used. This causes siblings from other jars (such as R.java) to be added to jdeps, even though they are not actually referenced. Mark these with Kind.UNRESOLVED to allow code to discard these. All non-test code that references Kind.IMPLICIT has been modified to also check for Kind.INCOMPLETE to keep the current behavior unchanged. Eg: jar liba.jar: package a; class A { void test() { // Use R.id } } jar liba_resources.jar: package a; class R { } package b; import a; class MyClass { A a = new A(); } ---> jdeps will contain both liba.jar, liba_resources.jar the latter will now have Kind.INCOMPLETE. -- MOS_MIGRATED_REVID=133791687
* Drop class initializers in turbineGravatar Liam Miller-Cushon2016-09-21
| | | | | -- MOS_MIGRATED_REVID=133756960
* Rollback of commit 82d43279f93d95e4c41b4bc598a3cc05ddd1ae1a.Gravatar Laszlo Csomor2016-09-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | *** Reason for rollback *** Breaks TensorFlow and other Bazel jobs on ci.bazel.io *** Original change description *** Change execution root for external repositories to be ../repo Some of the important aspect of this change: * Remote repos in the execution root are under output_base/execroot/repo_name, so the prefix is ../repo_name (to escape the local workspace name). * Package roots for external repos were previously "output_base/", they are now output_base/external/repo_name (which means source artifacts always have a relative path from their repository). * Outputs are under bazel-bin/external/repo_name/ (or similarly under genfiles). Note that this is a bit of a change from how this was implemented in the previous cl. Fixes #1262. RELNOTES[INC]: Previously, an external repository would be symlinked into the execution root at execroot/local_repo/external/remote_repo. This changes it to be at execroot/remote_repo. This may break genrules/Skylark actions that hardcode execution root paths. If this causes breakages for you, ensure that genrules are using $(location :target) to access files and Skylark rules are using http://bazel.io/docs/skylark/lib/File.html's path, dirname, etc. functions. Roll forward of bdfd58a. -- MOS_MIGRATED_REVID=133709658
* Change execution root for external repositories to be ../repoGravatar Kristina Chodorow2016-09-20
| | | | | | | | | | | | | | | | | | | | | | | Some of the important aspect of this change: * Remote repos in the execution root are under output_base/execroot/repo_name, so the prefix is ../repo_name (to escape the local workspace name). * Package roots for external repos were previously "output_base/", they are now output_base/external/repo_name (which means source artifacts always have a relative path from their repository). * Outputs are under bazel-bin/external/repo_name/ (or similarly under genfiles). Note that this is a bit of a change from how this was implemented in the previous cl. Fixes #1262. RELNOTES[INC]: Previously, an external repository would be symlinked into the execution root at execroot/local_repo/external/remote_repo. This changes it to be at execroot/remote_repo. This may break genrules/Skylark actions that hardcode execution root paths. If this causes breakages for you, ensure that genrules are using $(location :target) to access files and Skylark rules are using http://bazel.io/docs/skylark/lib/File.html's path, dirname, etc. functions. Roll forward of bdfd58a. -- MOS_MIGRATED_REVID=133606309
generated by cgit v1.2.3 (git 2.39.1) at 2024-07-05 03:02:42 +0000