aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/java_tools/buildjar
Commit message (Collapse)AuthorAge
* Record dependencies for exempt annotation processorsGravatar Liam Miller-Cushon2016-03-23
| | | | | | | | | | | | | | | | | | | | | | | Certain annotation processors that generate references to transitive dependencies are exempt from Strict Java Deps. Dagger ensures that all references to transitive types are reachable via a chain of direct dependencies specified in @Component interfaces. We don't record dependencies inside dagger-generated code, since Dagger is already enforcing strict dependencies. By not recording the deps we allow dependency management tools to remove depednencies that are only needed transitively by dagger. This approach doesn't work in general, since processors may generate references to transitive types that are not reachable via a chain of direct deps, and which may be subject to pruning by the reduced classpath optimization. This change causes strict deps to record dependencies (but not emit strict deps errors) inside code generated by exempt annotation processors other than Dagger. -- MOS_MIGRATED_REVID=117882599
* Stop using //tools/defaults:java_langtoolsGravatar Liam Miller-Cushon2016-03-18
| | | | | | | | | | | | | | | | | | The //tools/defaults package exports the values of blaze flags, including --java_langtools. That flag is being deprecated and the configuration moved into the java_toolchain rule. This change migrates from the //tools/defaults targets for Java tools to the canonical versions of those tools. Unlike the previous solution, the tools in the package are no longer affected by the current build configuration (e.g. the value of --java_toolchain). If that configurability becomes necessary again, we can achieve it using Skylark (via the java.compilation_info.boot_classpath provider), or by expanding the blaze query output for java_toolchain (as used in java_toolchain_content). -- MOS_MIGRATED_REVID=117485737
* Mask turbine classes from the processor classpath to avoid version skewGravatar Liam Miller-Cushon2016-03-17
| | | | | | | | This prevents processors from seeing turbine's version of e.g. guava. javac is still available becuase Blaze puts it on the bootstrap classpath. -- MOS_MIGRATED_REVID=117363448
* Add Python protobuf to Bazel third_party and move all protobuf targets intoGravatar David Chen2016-03-17
| | | | | | | //third_party/protobuf. -- MOS_MIGRATED_REVID=117336377
* Description redacted.Gravatar Liam Miller-Cushon2016-03-15
| | | | | -- MOS_MIGRATED_REVID=117151379
* Global cleanup change.Gravatar Googler2016-03-08
| | | | | -- MOS_MIGRATED_REVID=116498132
* Canonicalize the name of the target itself for java indirect dep errors.Gravatar Brian Silverman2016-03-03
| | | | | | | | | | | | | | | Roll-forward of I3762e6be911b79a133a2959fce3a8bf34d183232 with fixes. Otherwise it prints the wrong message when the target which is missing a dependency is in a repository other than the default one. Second try with the new test disabled for jdk7 because that uses an old, fixed version of the code. -- Change-Id: I7e16fa874066e1b63064662df42a98784fcae233 Reviewed-on: https://bazel-review.googlesource.com/#/c/3032/ MOS_MIGRATED_REVID=116248259
* Rollback of commit 37059e4ffd23fbc10828f61ab198b98c2ca7a8a9.Gravatar Kristina Chodorow2016-03-02
| | | | | | | | | | | | | | | | | | | *** Reason for rollback *** Breaks on Java 1.7 See http://ci.bazel.io/job/Bazel/385/. *** Original change description *** Canonicalize the name of the target itself for java indirect dep errors. Otherwise it prints the wrong message when the target which is missing a dependency is in a repository other than the default one. -- MOS_MIGRATED_REVID=116164887
* Canonicalize the name of the target itself for java indirect dep errors.Gravatar Brian Silverman2016-03-02
| | | | | | | | | | Otherwise it prints the wrong message when the target which is missing a dependency is in a repository other than the default one. -- Change-Id: I3762e6be911b79a133a2959fce3a8bf34d183232 Reviewed-on: https://bazel-review.googlesource.com/#/c/2782/ MOS_MIGRATED_REVID=116128663
* Fix pruning of implicitly final fieldsGravatar Liam Miller-Cushon2016-03-01
| | | | | -- MOS_MIGRATED_REVID=115902893
* Add a bazel-srcs target to the top-level package containing all Bazel sourcesGravatar Damien Martin-Guillerez2016-02-29
| | | | | | | | This target include all non tests targets of Bazel to do integration tests of bootstrapping. -- MOS_MIGRATED_REVID=115830741
* Rename proto dependencies to include a _java component in them.Gravatar Julio Merino2016-02-23
| | | | | | | | While doing this, homogenize the two dependencies on build.proto (via build_proto and build_proto_v2) into a single build_java_proto. -- MOS_MIGRATED_REVID=115328219
* Remove an unused field from AbstractPostProcessorGravatar Liam Miller-Cushon2016-02-19
| | | | | -- MOS_MIGRATED_REVID=114985098
* Make effectively final fields finalGravatar Liam Miller-Cushon2016-02-19
| | | | | -- MOS_MIGRATED_REVID=114984938
* Make effectively final fields finalGravatar Liam Miller-Cushon2016-02-19
| | | | | -- MOS_MIGRATED_REVID=114984900
* Turn //tools/build_rules into a package.Gravatar Julio Merino2016-02-17
| | | | | | | | | This is in preparation for another change that will add a new helper script to the tools/build_rules directory, and such script requires a BUILD rule of its own. -- MOS_MIGRATED_REVID=114898083
* Fix pruning of annotation declaration fieldsGravatar Liam Miller-Cushon2016-02-17
| | | | | | | | Fields in annotation declarations are implicitly final, so are candidates for constant variables. -- MOS_MIGRATED_REVID=114878247
* Fix tree pruning bug with constructor delegationGravatar Liam Miller-Cushon2016-02-17
| | | | | | | | | | -relax disables checks in Resolve to catch constructors that do not explicitly delegate to a superclass constructor when no nullary superclass constructor exists. Now that we're not using -relax, those constructor calls have to be preserved. -- MOS_MIGRATED_REVID=114875860
* Marking more jdk8 tests.Gravatar Damien Martin-Guillerez2016-02-16
| | | | | | | Fixes #881. -- MOS_MIGRATED_REVID=114731500
* Mark turbine test as JDK 8Gravatar Damien Martin-Guillerez2016-02-15
| | | | | | | | | This should unbreak our JDK 7 builds. Fixes #881. -- MOS_MIGRATED_REVID=114537482
* Turbine, a tool for improving Java build performanceGravatar Liam Miller-Cushon2016-02-10
| | | | | | | | | Turbine compiles ijars from source, ignoring method bodies and relaxing error checks for performance. We can then do normal compilation against those ijars and move javac off the build's critical path. -- MOS_MIGRATED_REVID=114247125
* Refactor buildjar BUILD filesGravatar Liam Miller-Cushon2016-02-10
| | | | | -- MOS_MIGRATED_REVID=114223734
* Turbine tree prunerGravatar Liam Miller-Cushon2016-02-04
| | | | | | | | | Prunes AST nodes that are not required for header compilation: method bodies, class and instance initializers, and definitely non-constant field initializers. -- MOS_MIGRATED_REVID=113807661
* Turbine options parserGravatar Liam Miller-Cushon2016-02-04
| | | | | -- MOS_MIGRATED_REVID=113806383
* Use labels in load statementsGravatar Damien Martin-Guillerez2016-02-02
| | | | | | | | | | | | This will fix some issue with loading skylark rules from @bazel_tools. Tested with ./compile.sh all and also a project that use pkg_tar from @bazel_tools (which is broken at HEAD). -- Change-Id: Iffbb7134ef5fee497890c4d01b85084973d45ad0 Reviewed-on: https://bazel-review.googlesource.com/2800 MOS_MIGRATED_REVID=113508661
* Report source position information for Error Prone crashesGravatar Liam Miller-Cushon2016-02-01
| | | | | -- MOS_MIGRATED_REVID=113492679
* Use correct file path separator.Gravatar Dmitry Lomov2016-01-29
| | | | | -- MOS_MIGRATED_REVID=113349046
* Add a strict dependencies exception for the GenerateWrapperModuleProcessor,Gravatar Googler2016-01-27
| | | | | | | | which is doing approximately the same thing dagger is doing, so is safe for the same reasons. -- MOS_MIGRATED_REVID=113054388
* Refactor DependencyModule#computeStrictClasspathGravatar Liam Miller-Cushon2016-01-21
| | | | | | | | | | so it can operate on lists of classpath entries, instead of pre-joined classpath strings. Also improve error for missing .jdeps inputs. -- MOS_MIGRATED_REVID=112634009
* Stop explicitly disabling useOptimizedZipGravatar Liam Miller-Cushon2016-01-21
| | | | | | | | javac's bespoke zip implementation is now off by default: https://bugs.openjdk.java.net/browse/JDK-7018859 -- MOS_MIGRATED_REVID=112600878
* Remove javac/testing from the public tree.Gravatar Lukacs Berki2016-01-19
| | | | | | | It is currently unused and messes up IDEs because it requires a few dependencies that are not in the source tree (most notably, jimfs) -- MOS_MIGRATED_REVID=112461226
* Set test_class for java_test test target where it is neededGravatar Damien Martin-Guillerez2016-01-19
| | | | | | | | | This make those test target compatible with --nolegacy_bazel_java_test. -- Change-Id: I2316c9aa53327b417ecce5fd5dab95ec726da11d Reviewed-on: https://bazel-review.googlesource.com/#/c/2690 MOS_MIGRATED_REVID=112446514
* Don't append the class output dir to the classpathGravatar Liam Miller-Cushon2016-01-15
| | | | | -- MOS_MIGRATED_REVID=112280234
* Remove explicit GC calls in javac persistent worker.Gravatar Googler2016-01-14
| | | | | | | | | Testing seems to have shown that the original reason for this being needed (running out of file handles due to unclosed file streams) has been fixed, meaning we don't need the GC workaround anymore. -- MOS_MIGRATED_REVID=112088923
* Open-source Java Builder test helpersGravatar Damien Martin-Guillerez2015-12-16
| | | | | -- MOS_MIGRATED_REVID=110280330
* Make buildjar plugins tests depends on BazelJavaBuilder instead of ↵Gravatar Damien Martin-Guillerez2015-12-16
| | | | | | | BlazeJavaBuilder. -- MOS_MIGRATED_REVID=110278416
* Split off more BUILD files.Gravatar Han-Wen Nienhuys2015-10-28
| | | | | -- MOS_MIGRATED_REVID=106493557
* Replace "Google Inc" with "The Bazel Authors" in every copyright notice.Gravatar Lukacs Berki2015-10-21
| | | | | -- MOS_MIGRATED_REVID=105949508
* Rename protobuf targets to xxx_proto.Gravatar Han-Wen Nienhuys2015-10-09
| | | | | -- MOS_MIGRATED_REVID=105054615
* Add IdlClass to the embedded default android tools repository and rearrangeGravatar Alex Humesky2015-10-05
| | | | | | | BuildJar's JarHelper so that it too can be embedded. -- MOS_MIGRATED_REVID=104539260
* Fix Java 7 test suiteGravatar Damien Martin-Guillerez2015-10-02
| | | | | | | | | | | | | - Prevent test from JavaBuilder to be run (mark them as JDK8 only). - Hack to replace the JavaBuilder target so that test that needs JavaBuilder will depends on the pre-built binary version. Tested: bash -c 'export JAVA_VERSION=1.7; source scripts/ci/build.sh; bazel_build' (and verified that there is no build failure). We might still have failing tests (worker and runfiles) but the failure seems like flakiness. -- MOS_MIGRATED_REVID=104504123
* Update to javac r2973Gravatar Liam Miller-Cushon2015-10-01
| | | | | -- MOS_MIGRATED_REVID=104410793
* Opt out of ClientCodeWrappingGravatar Liam Miller-Cushon2015-10-01
| | | | | -- MOS_MIGRATED_REVID=104407868
* Use an isolated classloader for JavaBuilder pluginsGravatar Liam Miller-Cushon2015-09-30
| | | | | -- MOS_MIGRATED_REVID=104131197
* Fix JavaBuilder bug when the source files are empty.Gravatar Eddie Aftandilian2015-09-30
| | | | | -- MOS_MIGRATED_REVID=104112686
* Add Android IDL jar outputs.Gravatar Googler2015-09-28
| | | | | | | | | | These outputs are a jar and source jar for the results of aidl processing. This is used to add aidl output to IDEs separate from the source code, similar to annotation output (gen jars). -- MOS_MIGRATED_REVID=104024453
* Rationalize copyright headersGravatar Damien Martin-Guillerez2015-09-25
| | | | | | | | | | | The headers were modified with `find . -type f -exec 'sed' '-Ei' 's|Copyright 201([45]) Google|Copyright 201\1 The Bazel Authors|' '{}' ';'` And manual edit for not Google owned copyright. Because of the nature of ijar, I did not modified the header of file owned by Alan Donovan. The list of authors were extracted from the git log. It is missing older Google contributors that can be added on-demand. -- MOS_MIGRATED_REVID=103938715
* Remove the callback that allowed excluding files from deletion.Gravatar Ulf Adams2015-08-26
| | | | | -- MOS_MIGRATED_REVID=101555261
* Fix -proc:only after e927532Gravatar Liam Miller-Cushon2015-08-24
| | | | | | | | | | Don't expect FLOW events if the compile policy is set to stop before FLOW (e.g. with -proc:only). Fixes google/bazel#379 -- MOS_MIGRATED_REVID=101242866
* Fall back to transitive classpath for no.such.pkg errors.Gravatar Liam Miller-Cushon2015-08-21
| | | | | | | | | | | | | | | | With -XDshouldStopPolicyIfError=INIT (the default) and -XDcompilePolicy=byfile (not the default), javac will stop compiling after emitting no.such.pkg and before any missing symbol errors are reported. The reduced classpath builder needs to fall back to the transitive classpath in this case. This currently works because with -XDcompilePolicy=todo (the default), javac attributes compilation units even if they have with syntax errors to avoid "surprising existing users". [1] [1] http://hg.openjdk.java.net/jdk9/dev/langtools/file/7fd155b7041c/test/tools/javac/policy/test3/Test.java -- MOS_MIGRATED_REVID=101175706