aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/main/java/com/google/devtools/build
Commit message (Collapse)AuthorAge
* Sort the results returned by the HybridGlobber if we are using results from ↵Gravatar Janak Ramakrishnan2016-07-19
| | | | | | | | | Skyframe globbing. This adds a log(n) factor to uses of globs, but getting globs to be returned in a reasonable order that can be emulated by legacy globbing is hard and bug-prone right now, and we must sort anyway if we are merging legacy and Skyframe globs. Note that this log(n) factor is already present on clean builds with legacy globbing. If we end up seeing performance issues on incremental loading, we can investigate making GlobFunction efficiently return elements in sorted order. (We would still need to sort if merging legacy and Skyframe globs, but that should be a relatively rare occurrence, and can be dealt with by a more efficient merge sort if necessary.) -- MOS_MIGRATED_REVID=127752554
* Make the proto bundling behavior the default when using the new library.Gravatar Sergio Campama2016-07-19
| | | | | -- MOS_MIGRATED_REVID=127747661
* Global cleanup change.Gravatar Googler2016-07-19
| | | | | -- MOS_MIGRATED_REVID=127735980
* Check maven_jar sha1s for validityGravatar Kristina Chodorow2016-07-18
| | | | | | | Fixes #1125. -- MOS_MIGRATED_REVID=127719941
* Rollback of commit 5ea55cbab969820da346d16c4998e957b8c3f60e.Gravatar Carmi Grushko2016-07-18
| | | | | | | | | | | | | *** Reason for rollback *** Let's launch open-source java_proto_library with strict-deps checking turned on by default. *** Original change description *** Unveil Bazel's proto_library and java_proto_library. -- MOS_MIGRATED_REVID=127716437
* Don't store workspace name on ruleGravatar Michajlo Matijkiw2016-07-18
| | | | | | | Instead have callers get it via package. -- MOS_MIGRATED_REVID=127715494
* Actually restart server timeout on Ping() and Cancel() RPCs.Gravatar Lukacs Berki2016-07-18
| | | | | | | In theory, now we can cancel Ping() and Cancel() RPCs, too, but since we don't tell their UUID anyone, we are fine. -- MOS_MIGRATED_REVID=127703598
* Make sure that aspects are only added once to Skylark attribute defintion.Gravatar Dmitry Lomov2016-07-18
| | | | | -- MOS_MIGRATED_REVID=127701838
* Use System#nanoTime() in the server timeout thread because it's monotonous, ↵Gravatar Lukacs Berki2016-07-18
| | | | | | | unlike System#currentTimeMillis(). -- MOS_MIGRATED_REVID=127697254
* Remove GlobList formatting logicGravatar Michajlo Matijkiw2016-07-18
| | | | | | | Isn't strictly necessary, remove it before anyone decides they want to use it. -- MOS_MIGRATED_REVID=127585617
* Model the TopLevelArtifactContext as an argument to the CompletionFunction ↵Gravatar Googler2016-07-18
| | | | | | | | | rather than a PRECOMPUTED value. Having a stale TopLevelArtifactContext leads to invalidation of all the top level target nodes, causing time wasted due to a lot of cache hits for a null build. -- MOS_MIGRATED_REVID=127585059
* Stop input and output of cc_library from clobbering each other.Gravatar Michael Staib2016-07-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before this change: Any given cc_library can only contribute one library with a given name to targets which depend on it. If an input library has the same name as the cc_library which it is an input to, the decision of which to use is based on the link mode. e.g., cc_library( name = "foo", srcs = ["foo.c", "libfoo.so"], ) will only contribute libfoo.a (a static library containing foo.o) in static mode, while it will only contribute libfoo.so (the precompiled shared library) in dynamic mode. This change alters cc_library's behavior in this case: * If libfoo.a would be empty (i.e., there are no linkable sources), then this is allowed. The libfoo.so from srcs is simply passed through. (Previously, the empty libfoo.a would be forwarded.) * Otherwise, this is an error. In the case where there are multiple libraries in the srcs with the same library identifier (lib[name].[a|so|lo]), cc_library will still choose one based on the link mode. This behavior has not changed. Similarly, cc_library will still choose one of its own outputs based on the link mode. That behavior has not changed either. RELNOTES[INC]: It is now an error to include a precompiled library (.a, .lo, .so) in a cc_library which would generate a library with the same name (e.g., libfoo.so in cc_library foo) if that library also contains other linkable sources. -- MOS_MIGRATED_REVID=127569615
* Unveil Bazel's proto_library and java_proto_library.Gravatar Carmi Grushko2016-07-18
| | | | | -- MOS_MIGRATED_REVID=127543896
* Rename namespace-sandbox/-runner to linux-sandbox.Gravatar Philipp Wollermann2016-07-18
| | | | | -- MOS_MIGRATED_REVID=127538990
* Add support for Hazelcast configuration fileGravatar Dan Fabulich2016-07-18
| | | | | | | -- Change-Id: I3255a14a60b7ae7749c49d5a885d92f4f19ec84f Reviewed-on: https://bazel-review.git.corp.google.com/#/c/3980/ MOS_MIGRATED_REVID=127537367
* 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
* Renamed module java_annotation-processing to java_annotation_processing.Gravatar Florian Weikert2016-07-15
| | | | | -- MOS_MIGRATED_REVID=127531480
* java_proto_library: add support for avoiding generating code for protos that ↵Gravatar Carmi Grushko2016-07-15
| | | | | | | are already compiled into a proto runtime. -- MOS_MIGRATED_REVID=127489419
* Remove not-quite necessary serialization bitsGravatar Michajlo Matijkiw2016-07-15
| | | | | | | What we really are doing here is formatting. -- MOS_MIGRATED_REVID=127481183
* Fix embarassing typo in VariableContext#with.Gravatar Nathan Harmata2016-07-15
| | | | | | | | | Turns out that our existing test coverage was insufficient for catching this (I misread the tests during development of unknown commit and thus incorrectly thought we had test coverage for this). -- MOS_MIGRATED_REVID=127461716
* Create another pseudo-label for the JDK launcherGravatar Liam Miller-Cushon2016-07-15
| | | | | -- MOS_MIGRATED_REVID=127458867
* Expose some additional ObjcProvider fields to Skylark.Gravatar Googler2016-07-14
| | | | | | | | | | The new packaging rules need these in order to access resources included in objc_library dependencies of an application. RELNOTES: -- MOS_MIGRATED_REVID=127445152
* Build encyclopedia: mark attributes that don't work withGravatar Greg Estren2016-07-14
| | | | | | | | select() as "nonconfigurable", polish up general configurable attributes docs. -- MOS_MIGRATED_REVID=127440164
* Exposes ios_minimum_os setting to Skylark.Gravatar Erik Abair2016-07-14
| | | | | -- MOS_MIGRATED_REVID=127435666
* StandaloneTestStrategy: Use relative paths for environmental variables ↵Gravatar Philipp Wollermann2016-07-14
| | | | | | | | | | | related to tests. This allows for better caching and also makes it easy to run tests in a different directory than the global execroot. The paths are rewritten to absolute paths in test-setup.sh, because Test Encyclopedia says that we pass absolute paths to tests in these variables. -- MOS_MIGRATED_REVID=127432675
* Silence Error Prone warnings in BuildConfiguration.java .Gravatar Lukacs Berki2016-07-14
| | | | | -- MOS_MIGRATED_REVID=127425448
* Delete ConfigurationsCreatedEvent. This is a resubmission of unknown commit ↵Gravatar Lukacs Berki2016-07-14
| | | | | | | | | and commit 6388680b44893c2b3ccddd73f6c4abb693a18cd4 now that the incompatibilities have been resolved. RELNOTES[INC]: The host_platform and target_platform entries are not written to the master log anymore. -- MOS_MIGRATED_REVID=127417109
* Rollback of commit 270f59b3630ef009f4a3e973581b3fc65a24ecc9.Gravatar Michael Staib2016-07-14
| | | | | | | | | | | | | *** Reason for rollback *** Broke Android targets with native code when cpu is explicitly set. *** Original change description *** RELNOTES: Improve Android split transition handling. -- MOS_MIGRATED_REVID=127377943
* 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
* Memoize TestSize.name().toLowerCase() -> TestSize mappingsGravatar Michajlo Matijkiw2016-07-14
| | | | | | | | | | Saves some needless garbage generation. We expect TestSize#getTestSize(String) to always pass in a lowercase value, which we promptly change to uppercase for the enum lookup. Instead memoize the lowercase -> TestSize mapping to avoid this call. -- MOS_MIGRATED_REVID=127362499
* Allow SpawnActions to always use a params fileGravatar Liam Miller-Cushon2016-07-14
| | | | | | | | | | | and use it to work around an analysis performance regression caused by header compilation. SpawnAction expands the full argument list and compares the length to --min_param_file_size when deciding whether to use a params file, and header compilation actions often have very long JoinExecPathsArg fragemnts that are expensive to expand. -- MOS_MIGRATED_REVID=127354241
* Add host/target feature support for ThinLTOGravatar Googler2016-07-14
| | | | | | | | | | | ThinLTO isn't fully supported for host builds, in order to handle this add support for enabling either a 'host' or 'target' feature in the configuration. The thin_lto feature will require the new 'target' configuration. Companion crosstool change: unknown commit. -- MOS_MIGRATED_REVID=127350506
* Discourage passing NestedSets to SpawnAction.Builder#addInputsGravatar Liam Miller-Cushon2016-07-14
| | | | | | | This is equivalent to the deprecation of NestedSetBuilder#addAll(NestedSet). -- MOS_MIGRATED_REVID=127336417
* Omit redundant string formattingGravatar Liam Miller-Cushon2016-07-14
| | | | | -- MOS_MIGRATED_REVID=127334955
* Replace calls to NestedSetBuilder#addAll(NestedSet) with addTransitiveGravatar Liam Miller-Cushon2016-07-14
| | | | | -- MOS_MIGRATED_REVID=127334869
* Don't treat empty files specially with respect to mtime/digest.Gravatar Janak Ramakrishnan2016-07-14
| | | | | | | RELNOTES: Bazel no longer regards an empty file as changed if its mtime has changed. -- MOS_MIGRATED_REVID=127328552
* Only include name for UserDefinedFunctionProfilingGravatar Michajlo Matijkiw2016-07-14
| | | | | | | | | | | These are called rather frequently I would imagine, but we don't profile often. Avoid useless object allocation. The SKYLARK_USER_FN qualifier and function name _should_ be enough to identify what's what, though I understand there could be duplicates. If this turns out to be an issue we can add more detail by memoizing location + name, but for now I'm inclined to treat it as YAGNI. -- MOS_MIGRATED_REVID=127328480
* Move more things from TestConstants to AnalysisMock and LoadingMock.Gravatar Ulf Adams2016-07-14
| | | | | | | Also remove other uses of TestConstants. -- MOS_MIGRATED_REVID=127327339
* Re-implement variables in the blaze query language. Instead of using a ↵Gravatar Nathan Harmata2016-07-14
| | | | | | | | | mutable global context of variable bindings, pass around immutable local contexts. The motivation is so we can safely evaluate all blaze query expressions concurrently under the hood. A global context is hostile to this goal. -- MOS_MIGRATED_REVID=127324600
* Use BuiltinFunction name for profilingGravatar Michajlo Matijkiw2016-07-14
| | | | | | | | | BuiltInFunction#call is pretty popular, avoid constructing a new string, or any object for that matter, every time we call it. FWIW the class name probably won't be particularly useful anyway since most built ins are anonymous. -- MOS_MIGRATED_REVID=127318185
* Properly support interaction of pre-grepped source files and LIPO even in theGravatar Googler2016-07-14
| | | | | | | | | presence of interface dependencies. The problem is that pre-grepped source files can be pulled in through CppCompileActions only available via lipoScannables. In this case, we need to make those pre-grepped source files inputs of the action or else we cannot be sure that the corresponding action is actually executed. -- MOS_MIGRATED_REVID=127317557
* Add all available toolchains to the FileProvider of cc_toolchain_suite.Gravatar Lukacs Berki2016-07-14
| | | | | | | This is necessary because we don't have a replacement for saying "this genrule needs the C++ toolchain" and I'd prefer this to do it in a backward-compatible way; burned myself here by trying to be principled, so let's now do the pragmatic thing. -- MOS_MIGRATED_REVID=127317024
* Upgrade to Hazelcast 3.6.Gravatar Dan Fabulich2016-07-14
| | | | | | | | | Fixes issue #1457. -- Change-Id: I913be48c924a48495f4da25a6710971c5c9d20b5 Reviewed-on: https://bazel-review.git.corp.google.com/#/c/3981/ MOS_MIGRATED_REVID=127314440
* Re-enable JDK logs. Fixes issue #1483.Gravatar Dan Fabulich2016-07-14
| | | | | | | -- Change-Id: I1c61b47ac1a6feac889ed8a02136edfd1e72b9e6 Reviewed-on: https://bazel-review.googlesource.com/#/c/3990/ MOS_MIGRATED_REVID=127313910
* Fixes Xcodegen when using --experimental_auto_top_level_union_objc_protos.Gravatar Sergio Campama2016-07-14
| | | | | -- MOS_MIGRATED_REVID=127313628
* Extract CppLinkAction.Builder to its own top level class.Gravatar Cal Peyser2016-07-13
| | | | | -- MOS_MIGRATED_REVID=127221256
* Fix native.existing_rule() when argument is an existing fileGravatar Laurent Le Brun2016-07-13
| | | | | -- MOS_MIGRATED_REVID=127218963
* Use canonical Java 7 javacopts in JavaProtoAspectGravatar Liam Miller-Cushon2016-07-13
| | | | | -- MOS_MIGRATED_REVID=127203325
* Rollback of commit feb1321044e3b7be34f9c1d7dacf9881d99a7327.Gravatar Klaus Aehlig2016-07-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | *** Reason for rollback *** Underlying issue fixed in commit 5b12f6e759c1f3137a7149d3026ff96686d07696 *** Original change description *** Automated [] rollback of commit a3381b6ac136a0cab3ba86020c739fe16b42cee9. *** Reason for rollback *** Broke bazel_rules_test See https://github.com/bazelbuild/bazel/issues/1501. *** Original change description *** Fix default for temporary directories to honor TMPDIR ...and only use the hard-coded "/tmp" as default for the default. Note that is unchanged that blaze.rpcserver.tmpdir still overrides. -- MOS_MIGRATED_REVID=127201018
* Glob arguments 'exclude' and 'exclude_directories' must be namedGravatar Laurent Le Brun2016-07-13
| | | | | | | | | | | Unamed arguments are confusing, e.g. glob(["*.java"], ["testing/*.java"]) The second list is actually excluded. RELNOTES: Glob arguments 'exclude' and 'exclude_directories' must be named -- MOS_MIGRATED_REVID=127190991