aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* Expose methods for the undeclared outputs and annotations directoriesGravatar Ulf Adams2017-01-10
| | | | | | | | | | This is part of a sequence of changes to update the test strategies; most of the actual code changes in this specific change is Google-internal, but I'm planning to update the external test strategies as well. -- PiperOrigin-RevId: 144064802 MOS_MIGRATED_REVID=144064802
* Changing visibility of junitrunner test sources to be visible from main src ↵Gravatar Irina Iancu2017-01-10
| | | | | | | | package. -- PiperOrigin-RevId: 144064682 MOS_MIGRATED_REVID=144064682
* Encapsulate the required provider logic in RequiredProviders class.Gravatar Dmitry Lomov2017-01-10
| | | | | | | | | For now, only for aspects, but eventually expand to Attribute's mandatory providers as well. -- PiperOrigin-RevId: 144063369 MOS_MIGRATED_REVID=144063369
* Windows: implement and use AsShortWindowsPathGravatar Laszlo Csomor2017-01-10
| | | | | | | | | | | | | | | | | | Because CreateProcessW doesn't support long paths, not even with the "\\?\" prefix [1], we need to convert long paths to short ones to spawn processes. This change implements the corresponding function and uses it in blaze_util_windows. [1] https://github.com/bazelbuild/bazel/issues/2181#issuecomment-270696173 See https://github.com/bazelbuild/bazel/issues/2107 See https://github.com/bazelbuild/bazel/issues/2181 -- PiperOrigin-RevId: 144062404 MOS_MIGRATED_REVID=144062404
* Bazel client: move NormalizePath to Windows codeGravatar Laszlo Csomor2017-01-10
| | | | | | | | | | | | | | | | | | | | | | | | | | This method was only used by the Windows-specific code so move it there and adapt it to Windows paths. Update AsWindowsPath to normalize its output: this is necessary because AsWindowsPath's output is a widechar path string which we often prefix with the UNC prefix, but such paths must be normalized (the kernel won't do it if the path has an UNC prefix). Finally, add an AsWindowsPathWithUncPrefix method which does what the name suggests: converts the path to Windows path, makes it absolute, and adds the UNC prefix if necessary. (This is a very common operation when calling WinAPI functions and we'll use it a lot in subsequent changes.) See https://github.com/bazelbuild/bazel/issues/2107 -- PiperOrigin-RevId: 144060297 MOS_MIGRATED_REVID=144060297
* Gives 3 levels of sandbox error message under different flags.Gravatar Yue Gan2017-01-10
| | | | | | | | | | | | | | | | 1. no flag: only the direct reason is given (command execution termination status), and also the instruction to use "--verbose_failures" 2. flag "--verbose_failures": gives failed execution command and the instruction to use "--sandbox_debug --strategy" 3. flag "--verbose_failures --sandbox_debug": gives failed execution command, debugging message from sandboxing, and the instruction to use "--strategy" Also removes "cd <sandbox_path>" in given failed command, since debugging is only necessary with flag "--verbose_failures --sandbox_debug" and the path is already given in sandboxing debugging message. Addresses #2174. Fixes bazel_sandboxing_test.sh and bazel_test_test.sh for the new error message. -- PiperOrigin-RevId: 144058351 MOS_MIGRATED_REVID=144058351
* Remove the inmemoryfs library from lib:foundationGravatar Ulf Adams2017-01-10
| | | | | | | | No production code should depend on inmemoryfs. -- PiperOrigin-RevId: 144054981 MOS_MIGRATED_REVID=144054981
* Remove the unused class RPCService and its test.Gravatar Lukacs Berki2017-01-10
| | | | | | -- PiperOrigin-RevId: 144054816 MOS_MIGRATED_REVID=144054816
* Open sourcing junitrunner tests.Gravatar Irina Iancu2017-01-10
| | | | | | -- PiperOrigin-RevId: 144053696 MOS_MIGRATED_REVID=144053696
* google-java-format buildjarGravatar Liam Miller-Cushon2017-01-10
| | | | | | -- PiperOrigin-RevId: 144039394 MOS_MIGRATED_REVID=144039394
* Description redacted.Gravatar Rumou Duan2017-01-10
| | | | | | -- PiperOrigin-RevId: 144010091 MOS_MIGRATED_REVID=144010091
* Skylark repository invalidation design doc update Gravatar Damien Martin-Guillerez2017-01-10
| | | | | | | | | | | | | | 1. Set status to implementing 2. Reorder the list of tasks because we need to invalidate correctly Skylark repositories before enabling environ parameter. 3. Fix the step 2 (was step 3) in order to pass a map instead of creating a new function. -- Change-Id: I9387ccedf75d63fef7e2e9546dc1f0421cb12679 Reviewed-on: https://cr.bazel.build/7972 PiperOrigin-RevId: 143999391 MOS_MIGRATED_REVID=143999391
* In the j2objc wrapper script, add the ability to process more than one ↵Gravatar Rumou Duan2017-01-10
| | | | | | | | | | source jars. In the j2objc dead code removal script, quote the object file names when invoking ar. They may contain special shell characeters. -- PiperOrigin-RevId: 143993977 MOS_MIGRATED_REVID=143993977
* Extract a set of advertised providers into a separate class.Gravatar Dmitry Lomov2017-01-10
| | | | | | -- PiperOrigin-RevId: 143991903 MOS_MIGRATED_REVID=143991903
* Expose DEVELOPER_DIR to SkylarkGravatar Dmitry Shevchenko2017-01-10
| | | | | | | | * Also make constants private since they both have getters now. -- PiperOrigin-RevId: 143973536 MOS_MIGRATED_REVID=143973536
* Enable header compilation by defaultGravatar Liam Miller-Cushon2017-01-10
| | | | | | -- PiperOrigin-RevId: 143972504 MOS_MIGRATED_REVID=143972504
* Improve performance and semantics of union of Skylark setsGravatar Jon Brandvein2017-01-10
| | | | | | | | | | | | | | | | | | | | | | == Before this change == Previously, if a and b are sets, then a + b created a new set c whose direct and transitive elements were all those of a, and with b appended as an additional transitive element. If on the other hand b is a list instead of a set, then its contents were appended as additional direct elements of c. In both cases, you can think of c as a copy of a that also knows about b. This copying of a's elements into c can lead to accumulation when you do it repeatedly, e.g. x += y in a loop. Each union can take O(n) time so you get O(n^2) time overall. Nested set union is supposed to be O(1) time per operation and O(n) time overall. It also leads to surprising iteration orders. If you do a + b + c + d (left-associative), where each one is a set, then when you do a post-order traversal you get the elements of b, c, d, and a, in that order. This is because b, c, and d each get appended as transitive children of the copies of a. == After this change == If a and b are sets, then a + b returns a new set c with a and b as its two transitive children. If b is a list, then c has a as its only transitive child and b's elements as its only direct elements. This is straightforward, O(1), and avoids the problem with the confusing order. It is implemented by removing the items/transitiveItems fields and just relying on NestedSetBuilder. RELNOTES[INC]: (Skylark) Set union is now O(1). As a side-effect, the iteration order of sets produced by union has changed. "print(set([1]) + set([2]) + set([3]))" will now give back the order 1, 2, 3 instead of 2, 3, 1. -- PiperOrigin-RevId: 143972165 MOS_MIGRATED_REVID=143972165
* Add missing #include statements for ijar Gravatar Klaus Aehlig2017-01-10
| | | | | | | | | | | We may not assume that those are pulled in by other header files; in fact, they are not on FreeBSD stable/11. -- Change-Id: Ic976cc71968aab0a1c3d6652fe8d8f1c4afa480b Reviewed-on: https://cr.bazel.build/8145 PiperOrigin-RevId: 143970238 MOS_MIGRATED_REVID=143970238
* Making it remove @ in module names Gravatar Mike Lewis2017-01-10
| | | | | | | | | | | Fixes #2311 Closes #2312. -- Reviewed-on: https://github.com/bazelbuild/bazel/pull/2312 PiperOrigin-RevId: 143967880 MOS_MIGRATED_REVID=143967880
* skylark_repository_test: use label in all load statement Gravatar Damien Martin-Guillerez2017-01-10
| | | | | | | | | | | | | | | | | | | This test was still using the old path syntax that is deprecated, use the label syntax instead. Note that there is still several instance of that old syntax in our tests: ``` grep -E -r 'load\(['"'"'"]/[^/]' src/test | wc -l 109 ``` -- Change-Id: Id2d1a9a525f1584136501026de01cf53e2263c3f Reviewed-on: https://cr.bazel.build/8135 PiperOrigin-RevId: 143967775 MOS_MIGRATED_REVID=143967775
* Remove method presence check after a bazel releaseGravatar Dmitry Shevchenko2017-01-10
| | | | | | -- PiperOrigin-RevId: 143966520 MOS_MIGRATED_REVID=143966520
* Jarjar the third-party dependencies of the Jacoco test runner.Gravatar Lukacs Berki2017-01-10
| | | | | | | | | | This is an encore of commit 823091f7516abf7d854021edc765daf1467f1647, which failed because Java temp directory handling in actions is broken at the moment. Fixes #2343. -- PiperOrigin-RevId: 143961605 MOS_MIGRATED_REVID=143961605
* Advertise java Skylark provider from java proto aspects.Gravatar Googler2017-01-10
| | | | | | | | The skylark provider is bound as "proto_java" to avoid collisions with the base, which is called "java". -- PiperOrigin-RevId: 143960605 MOS_MIGRATED_REVID=143960605
* Introduce expand_if_none_available to crosstoolGravatar Marcel Hlopko2017-01-10
| | | | | | | | | | This feature allows us to expand a flag_group when a build variable is not available. This is helpful when migrating crosstools in a backward compatible way (that works with released bazel as well as with bazel at HEAD). -- PiperOrigin-RevId: 143955333 MOS_MIGRATED_REVID=143955333
* Automated [] rollback of commit feee0704b9a2654e1b2a9d99ec9f2fd590f8b23f.Gravatar Marcel Hlopko2017-01-09
| | | | | | | | | | | | | | | | | | | | | | | | *** Reason for rollback *** Breaks //src/test/shell/bazel:bazel_sandboxing_test: http://ci.bazel.io/job/bazel-tests/lastCompletedBuild/BAZEL_VERSION=latest-jdk7,PLATFORM_NAME=linux-x86_64/testReport/(root)/(empty)/test_failing_action_with_ioexception_while_copying_outputs_throws_correct_exception/ *** Original change description *** Gives 3 levels of sandbox error message under different flags. 1. no flag: only the direct reason is given (command execution termination status), and also the instruction to use "--verbose_failures" 2. flag "--verbose_failures": gives failed execution command and the instruction to use "--sandbox_debug --strategy" 3. flag "--verbose_failures --sandbox_debug": gives failed execution command, debugging message from sandboxing, and the instruction to use "--strategy" Also removes "cd <sandbox_... *** ROLLBACK_OF=143937589 -- PiperOrigin-RevId: 143951901 MOS_MIGRATED_REVID=143951901
* Bazel client: add helper methods for Windows pathsGravatar Laszlo Csomor2017-01-09
| | | | | | | | See https://github.com/bazelbuild/bazel/issues/2107 -- PiperOrigin-RevId: 143950681 MOS_MIGRATED_REVID=143950681
* Move ExecutionInfoSpecifier to lib.actionsGravatar Ulf Adams2017-01-09
| | | | | | | | | We were previously jumping through hoops to avoid cyclic dependencies. It's unclear why the class was here in the first place rather than in lib.actions. -- PiperOrigin-RevId: 143948959 MOS_MIGRATED_REVID=143948959
* blaze_util_windows: use ostringstream to build cmdGravatar Laszlo Csomor2017-01-09
| | | | | | | | | | | Use std::ostringstream instead of string concats when building a command line, for more efficiency. See https://github.com/bazelbuild/bazel/issues/2107 -- PiperOrigin-RevId: 143947880 MOS_MIGRATED_REVID=143947880
* Bazel client, Windows: implement GetCwdGravatar Laszlo Csomor2017-01-09
| | | | | | | | | See https://github.com/bazelbuild/bazel/issues/2107 See https://github.com/bazelbuild/bazel/issues/2181 -- PiperOrigin-RevId: 143947877 MOS_MIGRATED_REVID=143947877
* Move ProcMeminfoParser to lib.unix, where it belongsGravatar Ulf Adams2017-01-09
| | | | | | -- PiperOrigin-RevId: 143939410 MOS_MIGRATED_REVID=143939410
* Gives 3 levels of sandbox error message under different flags.Gravatar Yue Gan2017-01-09
| | | | | | | | | | | | | | 1. no flag: only the direct reason is given (command execution termination status), and also the instruction to use "--verbose_failures" 2. flag "--verbose_failures": gives failed execution command and the instruction to use "--sandbox_debug --strategy" 3. flag "--verbose_failures --sandbox_debug": gives failed execution command, debugging message from sandboxing, and the instruction to use "--strategy" Also removes "cd <sandbox_path>" in given failed command, since debugging is only necessary with flag "--verbose_failures --sandbox_debug" and the path is already given in sandboxing debugging message. Addresses #2174. -- PiperOrigin-RevId: 143937589 MOS_MIGRATED_REVID=143937589
* Remove some unused code from FetchCommand that was pulling in JavaOptionsGravatar Ulf Adams2017-01-09
| | | | | | -- PiperOrigin-RevId: 143936734 MOS_MIGRATED_REVID=143936734
* Use ExecutionRequirements.REQUIRES_DARWIN instead of hard-coding the stringGravatar Ulf Adams2017-01-09
| | | | | | -- PiperOrigin-RevId: 143933349 MOS_MIGRATED_REVID=143933349
* Separate bazel/blaze-specific changes in IntelliJ aspect.Gravatar Googler2017-01-09
| | | | | | -- PiperOrigin-RevId: 143905310 MOS_MIGRATED_REVID=143905310
* Create indirections to intellij_info.bzl.Gravatar Googler2017-01-09
| | | | | | -- PiperOrigin-RevId: 143864069 MOS_MIGRATED_REVID=143864069
* Remove filegroups in tools/android/BUILD.tools in favor of aliases.Gravatar Adam Michael2017-01-09
| | | | | | -- PiperOrigin-RevId: 143825037 MOS_MIGRATED_REVID=143825037
* Delete some dead android rules and tools code.Gravatar Adam Michael2017-01-09
| | | | | | -- PiperOrigin-RevId: 143823562 MOS_MIGRATED_REVID=143823562
* Exclude transitive proto deps of deploy_env binaries from the generated ↵Gravatar Googler2017-01-09
| | | | | | | | extension registry for the enclosing binary. -- PiperOrigin-RevId: 143807030 MOS_MIGRATED_REVID=143807030
* RELNOTES: Default --android_dynamic_mode to off.Gravatar Googler2017-01-09
| | | | | | | | The main effect is to cause cc_test rules to build into a self-contained executable rather than relying on external shared libraries. (cc_binary rules do this by default already.) -- PiperOrigin-RevId: 143806513 MOS_MIGRATED_REVID=143806513
* Rollback of commit 27895101ce3d8e88b80e53f98fccf1d72310b54d.Gravatar Googler2017-01-09
| | | | | | | | | | | | | | *** Reason for rollback *** Breaks mac integration test *** Original change description *** Create indirections to intellij_info.bzl. -- PiperOrigin-RevId: 143802424 MOS_MIGRATED_REVID=143802424
* Create indirections to intellij_info.bzl.Gravatar Googler2017-01-09
| | | | | | -- PiperOrigin-RevId: 143793200 MOS_MIGRATED_REVID=143793200
* Updates Android flag documentation.Gravatar Alex Humesky2017-01-09
| | | | | | -- PiperOrigin-RevId: 143792017 MOS_MIGRATED_REVID=143792017
* Unsynchronize InMemoryNodeEntry#isDone. The buildingState variable's state ↵Gravatar Shreya Bhattarai2017-01-09
| | | | | | | | changes are visibile to all other threads by volatility and we don't read the variable at any intermediate or inconsistent state (simply check against null or a constant). reverseDepsToSignal is also made volatile for subclasses that need volatile reads to it. -- PiperOrigin-RevId: 143787032 MOS_MIGRATED_REVID=143787032
* Doc: fix tutorial text to be in sync with the sample. Gravatar Steren2017-01-06
| | | | | | | | | | | https://bazel-review.googlesource.com/#/c/8092/ updated the sample to download App Engine rules, but the text was not updated. Closes #2341. -- Reviewed-on: https://github.com/bazelbuild/bazel/pull/2341 PiperOrigin-RevId: 143776849 MOS_MIGRATED_REVID=143776849
* Add an "integration" test (rather, just a test that exercises the relevant ↵Gravatar Janak Ramakrishnan2017-01-06
| | | | | | | | package-evaluation code) for []. -- PiperOrigin-RevId: 143744672 MOS_MIGRATED_REVID=143744672
* Remove some cases of inferring configuration and platform from ruleContext ↵Gravatar Chris Parsons2017-01-06
| | | | | | | | outside of rule implementation. -- PiperOrigin-RevId: 143720112 MOS_MIGRATED_REVID=143720112
* Use the new turbine implementation with BazelGravatar Liam Miller-Cushon2017-01-06
| | | | | | | | | This improves performance with --java_header_compilation enabled compared to javac-turbine, and lays groundwork for some future optimizations. -- PiperOrigin-RevId: 143719507 MOS_MIGRATED_REVID=143719507
* Update apple_binary documentation to match support for multiple platforms ↵Gravatar Chris Parsons2017-01-06
| | | | | | | | and 3 different binary types. -- PiperOrigin-RevId: 143718198 MOS_MIGRATED_REVID=143718198
* These API levels were removed in NDK r12.Gravatar Adam Michael2017-01-06
| | | | | | | | | | | | | | TESTED= $ ls android-ndk-r12b/platforms android-12 android-15 android-18 android-22 android-9 android-13 android-16 android-19 android-23 android-14 android-17 android-21 android-24 RELNOTES: Removed broken api levels 3, 4 and 5 from Android NDK 12. -- PiperOrigin-RevId: 143710544 MOS_MIGRATED_REVID=143710544
* Revert "blaze_util_windows: use CreateFileW" Gravatar John Cater2017-01-06
| | | | | | | | | | | | This is for #2344. This reverts commit 116d039ccb49ec71c2ea9626d8a0a0aee5cbabc0. -- Change-Id: I4a8bc697c608c0f61af5bef8ff41faea96424ac3 Reviewed-on: https://cr.bazel.build/8095 PiperOrigin-RevId: 143708707 MOS_MIGRATED_REVID=143708707