aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/main/java/com/google
Commit message (Collapse)AuthorAge
* BEP: Provide more useful data in TestResult events Gravatar Klaus Aehlig2017-02-10
| | | | | | | | | | | | | | | | | | | | | For individual test actions, more useful data can be provided than just the success status. Do so. Also, to keep each log reported only once, send a new TestAttempt event for the final test attempt, but don't have the TestResult (which contains to union of all test files) an instance of the BuildEvent. As we now report test results in a separate object, drop the test data and directly pass in the files created. In this way, we also get all the moved files reported correctly in the individual attempts. -- Change-Id: Ic04b7bad4f92a381bd4d1b4ec91f743b89f81f84 Reviewed-on: https://cr.bazel.build/8694 PiperOrigin-RevId: 147149025 MOS_MIGRATED_REVID=147149025
* Standalone Skylark binary: now accepts a file as argumentGravatar Laurent Le Brun2017-02-10
| | | | | | | | | Usage: Skylark file.sky -- PiperOrigin-RevId: 147148097 MOS_MIGRATED_REVID=147148097
* Rollback of commit 49d97f9bb4dd04ef62198ded7b53b2a8f8c603db.Gravatar Luis Fernando Pino Duque2017-02-10
| | | | | | | | *** Reason for rollback *** -- PiperOrigin-RevId: 147146321 MOS_MIGRATED_REVID=147146321
* Fix java_stub_template.txt on Windows Gravatar Yun Peng2017-02-10
| | | | | | | | | | | | | To define a associative arrary in bash, we need `declare -A` instead of `declare -a`, but `declare -A` is not supported on macOS, so fall back to the old implementation of rlocation on macOS. Related issue: #2426 -- Change-Id: Ifd42eea0faa5cdbe73b26c9b52437ef6c771f71c Reviewed-on: https://cr.bazel.build/8790 PiperOrigin-RevId: 147144803 MOS_MIGRATED_REVID=147144803
* Update docs about built-in attributes of rules.Gravatar Googler2017-02-10
| | | | | | | | Historically built-in attributes were possible to override, but it's not the case anymore. This feature caused issues. -- PiperOrigin-RevId: 147138115 MOS_MIGRATED_REVID=147138115
* Fix building python zip when middleman runfile is used Gravatar Yun Peng2017-02-10
| | | | | | | | | | | | | | When adding a binary A into data attribute of a py_bianry B, a middleman runfile is used to ensure B's runfiles includes A's runfiles. But this middlman runfile is not really created, so it should not be zipped. Fixed: https://github.com/bazelbuild/bazel/issues/2088 -- Change-Id: Id1a99c66cd8027e945883cf4367f003bccd73473 Reviewed-on: https://cr.bazel.build/8772 PiperOrigin-RevId: 147137055 MOS_MIGRATED_REVID=147137055
* Tighten a precondition check during action execution.Gravatar Lukacs Berki2017-02-10
| | | | | | | | Input discovery is handled somewhere else, so the same Action instance should never be put twice into buildActionMap . -- PiperOrigin-RevId: 147132869 MOS_MIGRATED_REVID=147132869
* Change CppCompileAction to store execution info as a mapGravatar Ulf Adams2017-02-10
| | | | | | | | Update the builder to allow setting arbitrary execution info. -- PiperOrigin-RevId: 147130533 MOS_MIGRATED_REVID=147130533
* Open source java 8 desugarer.Gravatar Adam Michael2017-02-10
| | | | | | | | | | Fixes https://github.com/bazelbuild/bazel/issues/2222. RELNOTES: Support for Java 8 lambdas, method references, type annotations and repeated annotations in Android builds with --experimental_desugar_for_android. -- PiperOrigin-RevId: 147109786 MOS_MIGRATED_REVID=147109786
* --Gravatar Carmi Grushko2017-02-10
| | | | | PiperOrigin-RevId: 147105048 MOS_MIGRATED_REVID=147105048
* Emphasize that java_xxx_proto_library.strict_deps should not be used for new ↵Gravatar Carmi Grushko2017-02-10
| | | | | | | | code. -- PiperOrigin-RevId: 147096200 MOS_MIGRATED_REVID=147096200
* Use AliasProvider for DexArchiveAspect to "see through" alias targets until ↵Gravatar Googler2017-02-10
| | | | | | | | [] can be fixed. -- PiperOrigin-RevId: 147094979 MOS_MIGRATED_REVID=147094979
* Make aar_import fail fast if no android_sdk is set up.Gravatar Adam Michael2017-02-10
| | | | | | | | https://github.com/bazelbuild/bazel/issues/2500 -- PiperOrigin-RevId: 147089163 MOS_MIGRATED_REVID=147089163
* Use //third_party/protobuf:protobuf_java instead of ↵Gravatar Carmi Grushko2017-02-09
| | | | | | | | | | | | //third_party/protobuf:protobuf to refer to the Java proto runtime. (second attempt) This is the name in the upstream protobuf repo. -- PiperOrigin-RevId: 147057949 MOS_MIGRATED_REVID=147057949
* Using os.pathsep in python stub template.txtGravatar Yun Peng2017-02-09
| | | | | | | | See https://github.com/bazelbuild/bazel/commit/7093b97ec3f332016822ed154e81bd86bc7ca645#commitcomment-20814099 -- PiperOrigin-RevId: 147043323 MOS_MIGRATED_REVID=147043323
* Generated files directory is passed to dependers as a user header search ↵Gravatar Cal Peyser2017-02-09
| | | | | | | | path. In the process, rename and replace the incorrectly named ObjcCommon.Builder#addUserHeaderSearchPaths -- PiperOrigin-RevId: 147038434 MOS_MIGRATED_REVID=147038434
* Test output directory presence for skylark repositories without a FileValue Gravatar Damien Martin-Guillerez2017-02-09
| | | | | | | | | | | | Using a FileValue depenendency was not needed and create over-invalidation of the repository. On a cold server, this duplicate the call to the repository function, including potential slow operation like downloads. -- Change-Id: Ie09fa45b2d77f8986f4a4e95d06e8133481aff50 Reviewed-on: https://cr.bazel.build/8138 PiperOrigin-RevId: 147032408 MOS_MIGRATED_REVID=147032408
* Java template windows performanceGravatar Yun Peng2017-02-09
| | | | | | | | | | Address issue #2426 Closes #2427. -- PiperOrigin-RevId: 147031246 MOS_MIGRATED_REVID=147031246
* Remove the first argument of TestStrategy.getArgs.Gravatar Ulf Adams2017-02-09
| | | | | | | | | All callers were performing identical steps before calling getArgs - move the functionality into getArgs instead. -- PiperOrigin-RevId: 147028753 MOS_MIGRATED_REVID=147028753
* Replace PackageProvider with TargetProvider where possibleGravatar Ulf Adams2017-02-09
| | | | | | | | | | I was trying to simplify GenQuery, and found this during an audit of the code; it's not sufficient to make any larger changes, but it may help eventuall. -- PiperOrigin-RevId: 147023626 MOS_MIGRATED_REVID=147023626
* Remove dead code - unused method in PackageManager interfaceGravatar Ulf Adams2017-02-09
| | | | | | -- PiperOrigin-RevId: 147018240 MOS_MIGRATED_REVID=147018240
* Fix int*list binary operator.Gravatar Laurent Le Brun2017-02-09
| | | | | | | | [2] * 3 should return [2, 2, 2] and not [2, 2, 2, 2]. -- PiperOrigin-RevId: 147017581 MOS_MIGRATED_REVID=147017581
* Add "CLEAN_UP" as a reason for retrieving a node.Gravatar Shreya Bhattarai2017-02-09
| | | | | | -- PiperOrigin-RevId: 146982656 MOS_MIGRATED_REVID=146982656
* make DexArchiveAspect propagate through alias targetsGravatar Googler2017-02-09
| | | | | | | | RELNOTES: n/a -- PiperOrigin-RevId: 146968274 MOS_MIGRATED_REVID=146968274
* experimental_objc_library excludes non-.h headers, as does objc_library.Gravatar Cal Peyser2017-02-09
| | | | | | -- PiperOrigin-RevId: 146958087 MOS_MIGRATED_REVID=146958087
* Use //third_party/protobuf:protobuf_java instead of ↵Gravatar Carmi Grushko2017-02-09
| | | | | | | | | | //third_party/protobuf:protobuf to refer to the Java proto runtime. This is the name in the upstream protobuf repo. -- PiperOrigin-RevId: 146949832 MOS_MIGRATED_REVID=146949832
* Fix spelling mistake.Gravatar Nathan Harmata2017-02-09
| | | | | | -- PiperOrigin-RevId: 146949259 MOS_MIGRATED_REVID=146949259
* Rollback of commit 602a98cbebeff2ba1c2d8605a9552b2c8f5622ae.Gravatar Googler2017-02-09
| | | | | | | | | | | | | | | | *** Reason for rollback *** Rolling forward with fixes for the incremental tool. *** Original change description *** Automated [] rollback of commit d11d510c571b10787856395709f9ad945ca70bb2. *** Reason for rollback *** -- PiperOrigin-RevId: 146940409 MOS_MIGRATED_REVID=146940409
* --Gravatar Carmi Grushko2017-02-09
| | | | | PiperOrigin-RevId: 146926312 MOS_MIGRATED_REVID=146926312
* Introduce a new SkyValue that merely contains the workspace name. The ↵Gravatar Nathan Harmata2017-02-09
| | | | | | | | | | | | workspace name is needed for package loading, and so splitting out this computation into a separate skyframe node that can be change-pruned gives us better incrementality; previously we'd need to reload all packages on a WORKSPACE file change. N.B. (i) This CL doesn't solve all the other performance issues with //external in Bazel/Blaze since it's still inefficiently used for resolving labels like @foo//bar:baz. (ii) This CL doesn't address the wasteful invalidation + change pruning of all the packages. -- PiperOrigin-RevId: 146925369 MOS_MIGRATED_REVID=146925369
* Rollback of commit 6521cb22788adc2a74f6c26221dbe302dc9ccc14.Gravatar Damien Martin-Guillerez2017-02-08
| | | | | | | | | | | | | | | | | | *** Reason for rollback *** Break JDK 7 build on ci.bazel.io Found out by downloading bazel 0.4.4 for JDK7 and `bazel test //src/test/shell/integration:stub_finds_runfiles_test --define JAVA_VERSION=1.7` This is due to JDK 7 using JavaBuilder that comes with version 0.1 of Bazel, which is not fixed for that issue. *** Original change description *** Don't add the class output dir to the classpath -- PiperOrigin-RevId: 146920718 MOS_MIGRATED_REVID=146920718
* Stop retrieving CcToolChainProvider statically from hard-coded attribute ↵Gravatar Rumou Duan2017-02-08
| | | | | | | | | | ":cc_toolchain" in RuleContext, instead take the provider from users and pass it around to where it is used. This gives J2ObjcAspect the ability to specify the C++ toolchain attribute under a different name to avoid attribute conflicts with attached rules that have already declared attribute ":cc_toolchain". -- PiperOrigin-RevId: 146920294 MOS_MIGRATED_REVID=146920294
* Introduce a Skyframe function: ActionEnvironmentFunction Gravatar Damien Martin-Guillerez2017-02-08
| | | | | | | | | | | | | | | | | | ActionEnvironmentFunction returns the list of environment variable with the one overwritten by --action_env being replaced. This let other Skyframe function declares dependency to any value of the environment and being influenced by the --action_env flag. This will be used to declare dependency of remote repositories on environment variables (step 3 of https://bazel.build/designs/2016/10/18/repository-invalidation.html) -- Change-Id: I1ed3fb6f48e8e17d4d64c903fccecb6ed7596350 Reviewed-on: https://cr.bazel.build/7974 PiperOrigin-RevId: 146918603 MOS_MIGRATED_REVID=146918603
* Add a boolean flag to a Feature to specify whether it is enabled by default.Gravatar Googler2017-02-08
| | | | | | | | | This enables us to configure default features for each toolchain without having to hard-code anything in class such as CcCommon. -- PiperOrigin-RevId: 146904287 MOS_MIGRATED_REVID=146904287
* Apply --symlink_prefix to the output and exec root symlinks.Gravatar Luis Fernando Pino Duque2017-02-08
| | | | | | | | | | | | | | | | Turns out --symlink_prefix is not applied to the bazel-{out,workspace} unlike bazel-{bin,genfiles,testlogs} thus breaking the semantics of --symlink_prefix. This CL fixes the problem by using the symlink prefix instead of the product name when building the output and exec root symlinks. RELNOTES[INC]: Using --symlink_prefix is now applied to the output symlink (e.g. bazel-out) and the exec root symlink (e.g. bazel-workspace). -- PiperOrigin-RevId: 146902534 MOS_MIGRATED_REVID=146902534
* Just symlink instead of running 'lipo' on a single-architecture binaryGravatar Chris Parsons2017-02-08
| | | | | | -- PiperOrigin-RevId: 146847283 MOS_MIGRATED_REVID=146847283
* Treat configuration errors as failures in the master log.Gravatar Peter Schmitt2017-02-08
| | | | | | -- PiperOrigin-RevId: 146820813 MOS_MIGRATED_REVID=146820813
* Rollback of commit d11d510c571b10787856395709f9ad945ca70bb2.Gravatar Alex Humesky2017-02-08
| | | | | | | | *** Reason for rollback *** -- PiperOrigin-RevId: 146820790 MOS_MIGRATED_REVID=146820790
* Directory headers are filtered out of header inputs to objc compilation.Gravatar Cal Peyser2017-02-08
| | | | | | -- PiperOrigin-RevId: 146813606 MOS_MIGRATED_REVID=146813606
* Move all the resource processing tools into a "busybox".Gravatar Googler2017-02-08
| | | | | | | | | This makes the code simpler as well as reducing the number of targets to build. It also makes testing and profiling different action strategies vastly easier. -- PiperOrigin-RevId: 146812659 MOS_MIGRATED_REVID=146812659
* AppleCrosstoolTransition sets cpu based on platform.Gravatar Cal Peyser2017-02-08
| | | | | | -- PiperOrigin-RevId: 146812370 MOS_MIGRATED_REVID=146812370
* Allow aspects to adveritise providers they provide.Gravatar Dmitry Lomov2017-02-07
| | | | | | -- PiperOrigin-RevId: 146794883 MOS_MIGRATED_REVID=146794883
* Move android_sdk_for_testing bind into Android{S,N}dkRepositoryRule so that ↵Gravatar Adam Michael2017-02-07
| | | | | | | | | | eventually we can remove it from Bazel's WORKSPACE file. We'll need to wait for the next Bazel release to remove it from the WORKSPACE file. This should hopefully make the process of the Bazel sheriff running android_integration_test.sh a little easier. -- PiperOrigin-RevId: 146789106 MOS_MIGRATED_REVID=146789106
* Remove dead code in SkyframeLabelVisitorGravatar Ulf Adams2017-02-07
| | | | | | | | | | Two of the functions are never used, several of the parameter are not used or can be simplified (require callers to convert from Target -> Label). -- PiperOrigin-RevId: 146787380 MOS_MIGRATED_REVID=146787380
* Windows: fix off-by-one errors with MAX_PATHGravatar Laszlo Csomor2017-02-07
| | | | | | | | | | | | | | | | | | In almost every place we compared paths against MAX_PATH, we had it wrong. MAX_PATH is the null-terminated maximum length, so paths exactly MAX_PATH long (not counting the null-terminator) were incorrectly considered short. Also fix the error message in the MSVC python wrapper, because it reported an incorrect path length limit in the warning message. See https://github.com/bazelbuild/bazel/issues/2107 -- PiperOrigin-RevId: 146762382 MOS_MIGRATED_REVID=146762382
* WindowsFileSystem.java: Fix createSymbolicLink when target is a relative path Gravatar Yun Peng2017-02-07
| | | | | | | | | | Fixed https://github.com/bazelbuild/bazel/issues/2494 -- Change-Id: I2ca335fa5b3a7759f57085717128912f09363789 Reviewed-on: https://cr.bazel.build/8650 PiperOrigin-RevId: 146761747 MOS_MIGRATED_REVID=146761747
* Reinstate IdleServerTasks that accidentally got disabled with the switch ↵Gravatar Lukacs Berki2017-02-07
| | | | | | | | | | from AF_UNIX to gRPC. Do not reinstate the checks whether the server should terminate itself, though, because we apparently get along pretty well without those. -- PiperOrigin-RevId: 146760481 MOS_MIGRATED_REVID=146760481
* Do not store two profile artifacts in FdoSupportProvider.Gravatar Lukacs Berki2017-02-07
| | | | | | -- PiperOrigin-RevId: 146757663 MOS_MIGRATED_REVID=146757663
* Adds abstractions for include processing to CppSemanticsGravatar Googler2017-02-07
| | | | | | -- PiperOrigin-RevId: 146694092 MOS_MIGRATED_REVID=146694092
* Fix documentation for java_runtime_suite.defaultGravatar Liam Miller-Cushon2017-02-06
| | | | | | -- PiperOrigin-RevId: 146687524 MOS_MIGRATED_REVID=146687524