aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/java_tools
Commit message (Collapse)AuthorAge
...
* Roll forward commit ff8ea95fb2e4c77104263bebbf21c7e4e7a00d00Gravatar Liam Miller-Cushon2016-07-21
| | | | | -- MOS_MIGRATED_REVID=128080654
* Prepare JavaBuilder to accept an Aspect's name, in addition to a label, of ↵Gravatar Carmi Grushko2016-07-21
| | | | | | | | | | | the owner of a jar. The motivation is strict-deps checking for Aspected rules such as java_proto_library. With this additional information, JavaBuilder will be able to better tell the user which rule is missing in the 'deps' attribute of the target being built. Note that the AspectParameters aren't passed right now, because (1) serializing them adds complexity and (2) they're unneeded for JavaBuilder right now. -- MOS_MIGRATED_REVID=127993845
* Description redacted.Gravatar Googler2016-07-20
| | | | | -- MOS_MIGRATED_REVID=127962492
* Consolidate Bazel-specific javacopt handlingGravatar Liam Miller-Cushon2016-07-20
| | | | | -- MOS_MIGRATED_REVID=127957458
* Remove support for -extra_checksGravatar Liam Miller-Cushon2016-07-19
| | | | | -- MOS_MIGRATED_REVID=127745971
* Drop Error Prone-specific javacoptsGravatar Liam Miller-Cushon2016-07-18
| | | | | -- MOS_MIGRATED_REVID=127576227
* When running Javac and related tools, use -XX:+TieredCompilation ↵Gravatar Philipp Wollermann2016-07-18
| | | | | | | | | -XX:TieredStopAtLevel=1 instead of -client, as the latter is ignored on 64-bit JVMs since at least JDK 6. The new flags have been shown to cut down CPU time by 50% due to reduced time spent in JIT and improving overall run time for short running actions, some times also cutting them in half. -- MOS_MIGRATED_REVID=127536578
* Don't join header compilation classpath argumentsGravatar Liam Miller-Cushon2016-07-14
| | | | | | | The classpaths get split during flag parsing, so joining them is unnecessary. -- MOS_MIGRATED_REVID=127377219
* Fix incorrect merged of https://bazel-review.googlesource.com/#/c/3921/Gravatar Damien Martin-Guillerez2016-07-11
| | | | | | | -- Change-Id: I4c9d749d2b4acb1254cd65efbabf4e6515924414 Reviewed-on: https://bazel-review.googlesource.com/#/c/3921/ MOS_MIGRATED_REVID=127074886
* Global cleanup change.Gravatar Chris Povirk2016-07-11
| | | | | -- MOS_MIGRATED_REVID=126939078
* Add all the sources to //:srcs filegroup and add a check to detectGravatar Damien Martin-Guillerez2016-07-01
| | | | | | | | | | | missing file to it. We need to activate this check on presubmits -- Change-Id: Ia95e92d3816ce92bb69bc0e2cf56e9c60b68d970 Reviewed-on: https://bazel-review.googlesource.com/#/c/3949/ MOS_MIGRATED_REVID=126404792
* Factor out generation of Java source files to separate shells so that we ↵Gravatar Lukacs Berki2016-06-28
| | | | | | | don't have to work around the stupidity of CreateProcess(). -- MOS_MIGRATED_REVID=126049022
* Description redacted.Gravatar Carmi Grushko2016-06-23
| | | | | -- MOS_MIGRATED_REVID=125589653
* Global cleanup change.Gravatar Googler2016-06-21
| | | | | -- MOS_MIGRATED_REVID=125449526
* When running Java-based tools, use -XX:+TieredCompilation ↵Gravatar Philipp Wollermann2016-06-21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | -XX:TieredStopAtLevel=1 instead of -client, as the latter is ignored on 64-bit JVMs since at least JDK 6. The new flags have been shown to cut down CPU time by 50% due to reduced time spent in JIT and improving overall run time for short running actions, some times also cutting them in half. Here's a benchmark of Bazel building itself, giving a nice 2-4x speedup: ./output/bazel clean && ./output/bazel build //src:bazel Bazel before this change: INFO: Elapsed time: 197.783s, Critical Path: 194.10s INFO: Elapsed time: 198.928s, Critical Path: 195.65s INFO: Elapsed time: 194.913s, Critical Path: 191.56s Bazel before this change with --strategy=Javac=worker: INFO: Elapsed time: 130.436s, Critical Path: 124.27s INFO: Elapsed time: 116.114s, Critical Path: 112.30s INFO: Elapsed time: 116.852s, Critical Path: 84.45s Bazel with this change: INFO: Elapsed time: 121.625s, Critical Path: 119.08s INFO: Elapsed time: 119.401s, Critical Path: 115.80s INFO: Elapsed time: 121.198s, Critical Path: 119.03s Bazel with this change and --strategy=Javac=worker: INFO: Elapsed time: 54.456s, Critical Path: 51.28s INFO: Elapsed time: 53.272s, Critical Path: 51.09s INFO: Elapsed time: 52.966s, Critical Path: 49.86s Bazel with this change and --strategy=Javac=worker and stripping -XX:TieredStopAtLevel=1 from JVM flags when using workers: INFO: Elapsed time: 61.599s, Critical Path: 58.93s INFO: Elapsed time: 48.695s, Critical Path: 45.02s INFO: Elapsed time: 46.874s, Critical Path: 43.91s INFO: Elapsed time: 46.174s, Critical Path: 43.32s (The latter is a possible optimization for workers and it's also nice to know that the limited tiered compilation does not hurt their performance.) -- MOS_MIGRATED_REVID=125448089
* Rollback of commit bdfd58a8ca2ed5735d6aaa5b238fb0f689515724.Gravatar Laurent Le Brun2016-06-20
| | | | | -- MOS_MIGRATED_REVID=125160288
* Clean up the test runner code a bit.Gravatar Ulf Adams2016-06-20
| | | | | | | Small steps towards #1150. -- MOS_MIGRATED_REVID=125158135
* Fix deps of the Java test runner.Gravatar Ulf Adams2016-06-20
| | | | | | | This should help with #1203. -- MOS_MIGRATED_REVID=125154544
* Make the execution root match the runfiles tree structure for external ↵Gravatar Kristina Chodorow2016-06-17
| | | | | | | | | | | | | | | | | | | | | repositories One interesting side effect of how this is implemented is that for external repositories, bin/ and genfiles/ are combined. External repo output is under bazel-out/local-fastbuild/repo_name for each repo. 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. -- MOS_MIGRATED_REVID=125095799
* Convert the Bazel JUnit4 test runner from Guice to Dagger.Gravatar Googler2016-06-06
| | | | | -- MOS_MIGRATED_REVID=124008772
* For open-source Error Prone, disable checks that are more stylistic thanGravatar Eddie Aftandilian2016-05-30
| | | | | | | | | | | functional. This will improve signal-to-noise ratio for Android and our external users. Specifically, disables ClassName, DepAnn, LongLiteralLowerCaseSuffix, MultipleTopLevelClasses, and StaticAccessedFromInstance. -- MOS_MIGRATED_REVID=123459006
* Description redacted.Gravatar Googler2016-05-30
| | | | | -- MOS_MIGRATED_REVID=123431237
* Convert the Bazel JUnit4 test runner from Guice to Dagger.Gravatar Googler2016-05-27
| | | | | -- MOS_MIGRATED_REVID=123342439
* Write a -gensrc.jar even if the compilation failedGravatar Liam Miller-Cushon2016-05-19
| | | | | | | The generated sources are useful for debugging annotation processors. -- MOS_MIGRATED_REVID=122683781
* Add turbine to embedded toolsGravatar Liam Miller-Cushon2016-05-06
| | | | | | | | This allows --experimental_java_header_compilation=true to be used with Bazel. It is still off by default. -- MOS_MIGRATED_REVID=121623213
* Make processRequest public.Gravatar Googler2016-05-05
| | | | | -- MOS_MIGRATED_REVID=121520005
* Set Bazel's workspace nameGravatar Kristina Chodorow2016-04-27
| | | | | | | | | | RELNOTES: Bazel's workspace name is now io_bazel. If you are using Bazel's source as an external repository, then you may want to update the name you're referring to it as or you'll begin seeing warnings about name mismatches in your code. -- MOS_MIGRATED_REVID=120926179
* Do not compress small manifest files.Gravatar Googler2016-04-27
| | | | | -- MOS_MIGRATED_REVID=120846643
* Refactor JavaBuilder output handlingGravatar Liam Miller-Cushon2016-04-26
| | | | | | | | | Prefer Writer to OutputStream to reduce the number of char/byte conversions, and pass in a Writer for the persistent worker entry point instead of redirecting stdout and stderr. -- MOS_MIGRATED_REVID=120771072
* Clean up assertionsGravatar Liam Miller-Cushon2016-04-26
| | | | | -- MOS_MIGRATED_REVID=120742339
* Make --target_label and --rule_kind optionalGravatar Liam Miller-Cushon2016-04-25
| | | | | | | | This is consistent with JavaBuilder. Some actions (e.g. resource compilations) don't set these flags. -- MOS_MIGRATED_REVID=120590777
* Automated [] rollback of commit 857cda2c45a5cc68c3fa398311c48c571a64915d and ↵Gravatar Damien Martin-Guillerez2016-04-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | commit 790d2f6009d47fe92cf0cd92a1473bbf0141f32e. *** Reason for rollback *** Broke non-Bazel projects on ci.bazel.io Fixes #1168 *** Original change description *** Move the runfiles for external repositories to under the x.runfiles/ directory This also sets the Bazel workspace name to io_bazel_source. Fixes #848. Relevant to #1116, #1124, RELNOTES[INC]: All repositories are now directly under the x.runfiles directory in the runfiles tree (previously, external repositories were at x.runfiles/main-repo/external/other-repo. This simplifies handling remote repository runfiles considerably, but will break existing references to external repository runfiles.... *** -- MOS_MIGRATED_REVID=120535721
* Flush output before exitingGravatar Liam Miller-Cushon2016-04-22
| | | | | | | | Explicitly close the PrintWriter before exiting, calling write(String) on an auto-flushing PrintWriter doesn't flush the output. -- MOS_MIGRATED_REVID=120473105
* Switch from relying on the Guice-specific annotations to the JSR-330 ↵Gravatar Googler2016-04-21
| | | | | | | standard annotations. -- MOS_MIGRATED_REVID=120351013
* Refactor JDK 7 build to use the vendored version of itGravatar Damien Martin-Guillerez2016-04-20
| | | | | | | | | | | | | Along the path, fix the build for JDK 7 and get rid of most ugliness in the JDK 7 build. Now simply setting JAVA_VERSION to 1.7 will build a JDK 7 compatible version. Fixes #1159. -- Change-Id: I9599283844a57d9e053f12d37445907f22a9232e Reviewed-on: https://bazel-review.googlesource.com/#/c/3452 MOS_MIGRATED_REVID=120332747
* Move the runfiles for external repositories to under the x.runfiles/ directoryGravatar Kristina Chodorow2016-04-20
| | | | | | | | | | | | | | | This also sets the Bazel workspace name to io_bazel_source. Fixes #848. Relevant to #1116, #1124, RELNOTES[INC]: All repositories are now directly under the x.runfiles directory in the runfiles tree (previously, external repositories were at x.runfiles/main-repo/external/other-repo. This simplifies handling remote repository runfiles considerably, but will break existing references to external repository runfiles. --- Furthermore, if a Bazel project does not provide a workspace name in the WORKSPACE file, Bazel will now default to using __main__ as the workspace name (instead of "", as previously). The repository's runfiles will appear under x.runfiles/__main__/. -- MOS_MIGRATED_REVID=120224534
* Deprecate top-level Java toolchain flags in favour of java_toolchainGravatar Liam Miller-Cushon2016-04-19
| | | | | | | | | | | | | This change disables --java_langtools, --javabuilder_top, --singlejar_top, --genclass_top, and --ijar_top, and finishes replacing them with java_toolchain.{javac,javabuilder,singlejar,genclass,ijar}. RELNOTES: Replace --java_langtools, --javabuilder_top, --singlejar_top, --genclass_top, and --ijar_top with java_toolchain.{javac,javabuilder,singlejar,genclass,ijar} -- MOS_MIGRATED_REVID=120154954
* Fixes GenClass exception when the temp directory does not exists.Gravatar Damien Martin-Guillerez2016-04-19
| | | | | | | -- Change-Id: Ie4236fbadc63ef38f08783acc0162cf0dd4fe872 Reviewed-on: https://bazel-review.googlesource.com/3422 MOS_MIGRATED_REVID=120146252
* GenClass: delete the temporary directory before usageGravatar Damien Martin-Guillerez2016-04-18
| | | | | | | Fixes #1104. -- MOS_MIGRATED_REVID=120098776
* Don't print extra newlinesGravatar Liam Miller-Cushon2016-04-11
| | | | | -- MOS_MIGRATED_REVID=119421719
* Relax strict deps checking in Turbine, leave enforcement to JavaBuilderGravatar Liam Miller-Cushon2016-04-11
| | | | | -- MOS_MIGRATED_REVID=119394125
* Don't emit native headers, only Java headersGravatar Liam Miller-Cushon2016-04-08
| | | | | | | If NATIVE_HEADER_OUTPUT is set (which it always was for ZipOutputFileManager), then javac will emit jni headers for native methods. Emitting the the headers requires doing unnecessary additional work and completing additional symbols, which may not be present on the classpath. -- MOS_MIGRATED_REVID=119286406
* Remove runWithLargeStack and remove some single-use abstractionsGravatar Liam Miller-Cushon2016-04-07
| | | | | -- MOS_MIGRATED_REVID=119188524
* Disable tree pruning for blacklisted annotation processorsGravatar Liam Miller-Cushon2016-04-07
| | | | | | | | | | Some annotation processors use non-standard APIs to examine the AST, and are broken by tree pruning. This change adds configuration to Turbine to allow a blacklist of processors to be provided, and disables tree pruning if any blacklisted processors are found. -- MOS_MIGRATED_REVID=119079114
* Remove shell quotes from javacoptsGravatar Liam Miller-Cushon2016-04-05
| | | | | -- MOS_MIGRATED_REVID=118967358
* Allow compilations to have an empty source listGravatar Liam Miller-Cushon2016-04-04
| | | | | | | for compatibility with JavaBuilder. -- MOS_MIGRATED_REVID=118802288
* Deduplicate annotation processor classesGravatar Liam Miller-Cushon2016-04-01
| | | | | | | Also rename the build method from add* to set*. -- MOS_MIGRATED_REVID=118746866
* Track which output files are writtenGravatar Liam Miller-Cushon2016-04-01
| | | | | | | | | instead of initializing all file objects to empty. Some annotation processors speculatively open files and handle FileNotFoundException, and we were always providing empty files. -- MOS_MIGRATED_REVID=118730162
* Handle more javacoptsGravatar Liam Miller-Cushon2016-04-01
| | | | | | | | | | | | | | Previously turbine whitelisted -source and -target flags and ignored all other configured javacopts, including custom annotation processor options and javac debug flags. The change switches to allowing all javacopts, and blackisting options that are known to be incompatible. Also ensure that invalid flags result in an error. -- MOS_MIGRATED_REVID=118718745
* Prune private fields and methods from the outputGravatar Liam Miller-Cushon2016-04-01
| | | | | -- MOS_MIGRATED_REVID=118713377