| Commit message (Collapse) | Author | Age |
|
|
|
|
|
|
|
|
| |
text around the version string. Fixes #1678.
--
Change-Id: I765699c43119e28d5f0ef8245b91e263d6af6dc0
Reviewed-on: https://bazel-review.googlesource.com/c/5570
MOS_MIGRATED_REVID=131080854
|
|
|
|
|
|
|
|
|
| |
If the user had an alias grep='grep --color=always' it would break the
bash completion, outputting the formatting character. Bypass the user
alias to avoid the effect.
--
MOS_MIGRATED_REVID=131040601
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=130761732
|
|
|
|
|
|
|
|
|
|
|
| |
bfaff29b1bafd7892a2197b7cf826e1be0ff3927.
Without the newline, our bazel/scripts/ci/build.sh will append an android_sdk_repository() rule into the file in such a way, that it produces a syntax error.
Also fix the build.sh script to be more resilient and not actually require the newline.
--
MOS_MIGRATED_REVID=130737331
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=130406085
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
*** Reason for rollback ***
The underlying issue (wrong operator '==' instead of '=' for test (a.k.a '[')) was fixed by commit 31b059fc07831f86be951609128b7cd51613df98 which removed the offending code completely.
*** Original change description ***
Revert build-runfiles to use /bin/bash
If /bin/sh is dash (default on Ubuntu), this output a lot of garbage error, which make thing that the dummy build-runfiles is incorrect.
See http://ci.bazel.io/view/Bazel%20bootstrap%20and%20maintenance/job/Bazel/JAVA_VERSION=1.8,PLATFORM_NAME=ubuntu_15.10-x86_64/731/console
--
MOS_MIGRATED_REVID=129432905
|
|
|
|
|
|
|
|
|
| |
If /bin/sh is dash (default on Ubuntu), this output a lot of garbage error, which make thing that the dummy build-runfiles is incorrect.
See http://ci.bazel.io/view/Bazel%20bootstrap%20and%20maintenance/job/Bazel/JAVA_VERSION=1.8,PLATFORM_NAME=ubuntu_15.10-x86_64/731/console
--
MOS_MIGRATED_REVID=129430198
|
|
|
|
|
|
|
|
|
| |
bootstrap.
Previously, we deleted the temp dir and after that, we tried to print errfile in it, and were surprised that it didn't exist.
--
MOS_MIGRATED_REVID=129320375
|
|
|
|
|
|
|
| |
on Windows by default)
--
MOS_MIGRATED_REVID=129319018
|
|
|
|
|
|
|
|
|
| |
bootstrapping.
It apparently happened not to make a difference, but still.
--
MOS_MIGRATED_REVID=129314926
|
|
|
|
|
|
|
|
|
|
|
| |
Without that, we write some data in the cache that should not be cached.
This fix the remaining issue about bazel writing out of the cache
directory described in Homebrew/homebrew-core#3427.
--
Change-Id: I8794693233675124fbc7ee3691297e7a84d999b4
Reviewed-on: https://bazel-review.googlesource.com/#/c/4221/
MOS_MIGRATED_REVID=128980718
|
|
|
|
|
|
|
| |
--
Change-Id: I70c5707db557637894054c0ccec98999d043c7bf
Reviewed-on: https://bazel-review.googlesource.com/#/c/4220
MOS_MIGRATED_REVID=128979317
|
|
|
|
|
|
|
|
|
| |
Fixes #1577
--
Change-Id: I81343d611490263bc7e16dedfbbc56f9f988f63d
Reviewed-on: https://bazel-review.googlesource.com/#/c/4170
MOS_MIGRATED_REVID=128576630
|
|
|
|
|
|
|
| |
--
Change-Id: I4926b820dbab3c218ad6ead8bc6eb7b781dedd00
Reviewed-on: https://bazel-review.googlesource.com/#/c/4162
MOS_MIGRATED_REVID=128465857
|
|
|
|
|
|
|
| |
--
Change-Id: I8c0b6adf08a4ca64ad41e0454cb30842c133fa22
Reviewed-on: https://bazel-review.googlesource.com/#/c/4161
MOS_MIGRATED_REVID=128465441
|
|
|
|
|
|
|
| |
--
Change-Id: I2da9049019b3965975fab9b7f606d099d6eab2ff
Reviewed-on: https://bazel-review.googlesource.com/#/c/4040/
MOS_MIGRATED_REVID=128208129
|
|
|
|
|
|
|
|
|
| |
The label pattern was incorrectly passed around (the command was not
translated into a target pattern) which resulted in the target not
being completed.
--
MOS_MIGRATED_REVID=128177563
|
|
|
|
|
|
|
| |
--
Change-Id: I3a6f5a0bce24a4f6c42ef1811f0b1587fed1c8f5
Reviewed-on: https://bazel-review.googlesource.com/#/c/4130
MOS_MIGRATED_REVID=128154259
|
|
|
|
|
|
|
| |
reference by future repository rules. Removes the xcode-locator binary file under tools/objc. Originally, the precompiled binary was going to be referenced, but it's easier to build from source in the repository rule.
--
MOS_MIGRATED_REVID=128063694
|
|
|
|
| |
MOS_MIGRATED_REVID=128060563
|
|
|
|
|
|
|
| |
of tests run on Jenkins on Windows.
--
MOS_MIGRATED_REVID=128053672
|
|
|
|
|
|
|
|
|
| |
Fixes #1540.
--
Change-Id: I0f16bde6496f2abef8ba3d3562e62a1534c177ca
Reviewed-on: https://bazel-review.googlesource.com/#/c/4081/
MOS_MIGRATED_REVID=127960571
|
|
|
|
|
|
|
|
|
| |
I reused some functions defined in test-setup.sh and some test cases in bazel_example_test.sh
--
Change-Id: Ic9a20a819bfcc52f2ff72db74d017e6cdc062fc3
Reviewed-on: https://bazel-review.googlesource.com/#/c/4090
MOS_MIGRATED_REVID=127926337
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=127923810
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=127847570
|
|
|
|
|
|
|
|
|
| |
-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
|
|
|
|
|
|
|
| |
This helps a lot when trying to debug Bazel from an IDE.
--
MOS_MIGRATED_REVID=127529499
|
|
|
|
|
|
|
| |
--
Change-Id: I7b20894240611e52b6ba9bfd10cc37e2edb24e1f
Reviewed-on: https://bazel-review.googlesource.com/#/c/4032/
MOS_MIGRATED_REVID=127326078
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=127198779
|
|
|
|
|
|
|
|
|
| |
This should let the users wrap around ci.sh if they want to use the instructions from
the comments (those related to Gerrit and Travis-CI), as opposed to having to edit the
script.
--
MOS_MIGRATED_REVID=127093626
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=127086608
|
|
|
|
|
|
|
| |
--
Change-Id: I12ae7bdf6b7fe5340843bc1750313556970f4c90
Reviewed-on: https://bazel-review.googlesource.com/c/4020
MOS_MIGRATED_REVID=127076309
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Change the bootstrapping process so that setting the BAZEL_WRKDIR
environment variable will tell the bootstrap process to mostly write
to that directory---apart from adding the symlinks next to the WORKSPACE
file. So setting this variable will avoid the usual writes to random
places in the file system (like /tmp and the user's home directory).
--
Change-Id: I9d1af747e75cc2a7bb1af08308acc9dfd927e920
Reviewed-on: https://bazel-review.googlesource.com/#/c/3963
MOS_MIGRATED_REVID=127075535
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
*** Reason for rollback ***
Apparently we now try to open output files for the process twice: once when we are constructing the output streams, and the second time when we tell the process to redirect its outputs. This causes the outputs to be empty on Windows
*** Original change description ***
Do redirection of stdout / stderr in Java instead of reimplementing it in every process wrapper again.
--
MOS_MIGRATED_REVID=126801016
|
|
|
|
|
|
|
|
|
| |
So people don't redistribute the binaries without the sources.
--
Change-Id: Ia3f91285d40669f8aa367633bb54214e9fdf1454
Reviewed-on: https://bazel-review.googlesource.com/#/c/3954
MOS_MIGRATED_REVID=126681605
|
|
|
|
|
|
| |
This makes the regeneration instruction in much simpler (since now they do publish 64-bit Linux binaries), and more importantly, fixes a race condition between a client request and the gRPC server timeout thread.
Change-Id: I9c8b628530fa0192804f8e67b285963ac26110c4
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
types.
Fixes #921
Certain parameters, such as the `executable` and `command` parameters of
ctx.action, allow multiple types. However, the allowed types are not enumerated
in the SkylarkSignature annotation, causing the generated Skylark Library
document to not display any type information for those params.
This change adds a new field, `allowedTypes`, to `SkylarkSignature` that is a
list of `ParamType` objects. If the param can accept multiple types, then
`type` is set to `Object.class`, and `allowedTypes` is set to the list of types
that can be accepted.
--
MOS_MIGRATED_REVID=126617047
|
|
|
|
|
|
|
|
|
| |
Also remove some bashism to fix #1456.
--
Change-Id: I374f1290a3eb9e9e5bcc48e19a1c0ab1c5bdaa99
Reviewed-on: https://bazel-review.googlesource.com/#/c/3952/
MOS_MIGRATED_REVID=126572632
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
...instead of cleaning up the last one once for each tmpdir.
Commit 63e8d6321 changed the command registered atexit from
an explict "rm -rf '${DIR}'" to a call to a cleanup_tempdir
function, with the function containing the remove command.
As, however, the function name is not unique (in fact, it is
a constant), that function will get overridden at each invocation
of the tempdir function. The fact that the function is registered
several times with atexit doesn't help, as it will always remove
the last created temporary directory. Fix this, by creating function
names that encode the directory to be removed in the name. Fixes #1466.
--
Change-Id: I833aef8ee5423412f058e74c8c9e2f4bb53a0cba
Reviewed-on: https://bazel-review.googlesource.com/#/c/3955
MOS_MIGRATED_REVID=126406196
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
| |
Declare openjdk-9-jdk as Conflicts
Progress towards #1456
--
Change-Id: I67c17b78f90f81dbcc9c39bf6f3915789662fa94
Reviewed-on: https://bazel-review.googlesource.com/#/c/3951
MOS_MIGRATED_REVID=126402113
|
|
|
|
|
|
|
|
|
| |
We have the source so build from sources instead. This was
causing strange behavior were the input file was conflicting
with the output file of the corresponding java_library but
was matched by the glob.
Change-Id: Ie0b70215d1862627747bd7b83f075743c904c967
|
|
|
|
|
|
|
|
|
| |
that's what it does.
In particular, it affects Windows.
--
MOS_MIGRATED_REVID=126293339
|
|
|
|
|
|
|
| |
every process wrapper again.
--
MOS_MIGRATED_REVID=126279021
|
|
|
|
|
|
|
| |
--
Change-Id: Ic75d8f1d05b0ccba2faaad6527ab2719612325d8
Reviewed-on: https://bazel-review.googlesource.com/#/c/3892/
MOS_MIGRATED_REVID=125779082
|
|
|
|
|
|
|
|
|
| |
Fixes 1444.
--
Change-Id: If9afa2bebcd35b0c0882685588d8e1f3b18f5344
Reviewed-on: https://bazel-review.googlesource.com/#/c/3890/
MOS_MIGRATED_REVID=125771832
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
When comparing two strings with test(1) we need to
make sure that the strings are passed each as precisely
one argument to test (a.k.a. [). Shell variables, like $1,
however may expand to zero or more arguments; hence quote
appropriately.
--
Change-Id: I7fd1cee970ed24e6c332b0da361bc916aafcb006
Reviewed-on: https://bazel-review.googlesource.com/#/c/3881
MOS_MIGRATED_REVID=125671405
|
|
|
|
|
|
|
| |
jekyll by pressing "r".
--
MOS_MIGRATED_REVID=125593474
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=125589653
|
|
|
|
|
|
|
|
|
| |
is not picked up when you run the scripts manually using "bash script.sh".
This is also in our shell style guide: "Executables must start with #!/bin/bash and a minimum number of flags. Use set to set shell options so that calling your script as bash <script_name> does not break its functionality."
--
MOS_MIGRATED_REVID=125450962
|