| Commit message (Collapse) | Author | Age |
... | |
|
|
|
|
|
|
|
|
|
|
|
| |
Release 0.0.2 is too old and doesn't work any more.
I'll make the same change in the tutorial repo.
Closes #1962.
--
Reviewed-on: https://github.com/bazelbuild/bazel/pull/1962
MOS_MIGRATED_REVID=136825134
|
|
|
|
|
|
|
|
|
| |
Linaro compiler filegroup on macOS. This fixes the broken bazel_toolchain_test on macOS.
I filed a bug to track the real issue (HardlinkedExecRoot sometimes seems to fail when inputs contain directories): https://github.com/bazelbuild/bazel/issues/1974
--
MOS_MIGRATED_REVID=136823559
|
|
|
|
|
|
|
|
|
|
|
| |
and backends that don't. If a CppCompileAction doesn't do ahead of time include
scanning with its current executor, it might still have inherited discovered
inputs from an earlier execution with a different executor. In particular,
module files might have been pruned out and we need to adjust the command line
accordingly.
--
MOS_MIGRATED_REVID=136822752
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=136818366
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=136666421
|
|
|
|
|
|
|
| |
now handle failed TreeArtifact output checks gracefully.
--
MOS_MIGRATED_REVID=136627086
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=136626076
|
|
|
|
|
|
|
|
| |
* This change enables most of LLDB functionality on binaries built by swift_library.
It makes the linker record references to the .swiftmodule outputs, which then can be read by LLDB to deserialize Swift modules and provide debug info.
--
MOS_MIGRATED_REVID=136620475
|
|
|
|
|
|
|
|
| |
Closes #1938.
--
Reviewed-on: https://github.com/bazelbuild/bazel/pull/1938
MOS_MIGRATED_REVID=136603317
|
|
|
|
|
|
|
| |
Will be re-added once it is fixed. Tracking bug is #1965.
--
MOS_MIGRATED_REVID=136595269
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=136594531
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
To set and use a RepositoryCache instance in HttpDownloader while parsing the command line options, we can pass an AtomicReference<HttpDownloader> instance from BazelRepositoryModule to the HttpArchiveFunctions. However, we'll need to change HttpDownloader download() calls to be non-static in order to initialize an instance of HttpDownloader in BazelRepositoryModule.
Remaining TODOs:
- RepositoryCache implementation and unit testing
- RepositoryCache lockfiles
- RepositoryCache integration testing
GITHUB: #1752
--
MOS_MIGRATED_REVID=136593517
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=136590448
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
*** Reason for rollback ***
Roll-forward with fix.
Tested with `bazel build src:srcs //src/test/...` using a bootstrapped bazel.
Fixes #1923.
RELNOTES[INC]: Non test-only targets can no longer depends on test-only targets.
*** Original change description ***
Automated [] rollback of commit a9f20b0d6459d395444c45cf5e94a899f3443633.
*** Reason for rollback ***
Broke Bazel CI:
ERROR: /home/ci/workspace/Bazel/JAVA_VERSION/1.8/PLATFORM_NAME/linux-x86_64/src/BUILD:284:1: in filegroup rule //src:srcs: non-test target '//src:srcs' depends on testonly target '//src/java_tools/junitrunner/java/com/google/testing/junit/junit4:srcs' and doesn't have testonly attribute set.
ERROR: Analysis of target '//scripts/packages:packages' failed; build aborted.
http://ci.bazel.io/view/Bazel%20b...
***
--
MOS_MIGRATED_REVID=136589937
|
|
|
|
|
|
|
| |
of the old test-setup.
--
MOS_MIGRATED_REVID=136589691
|
|
|
|
|
|
|
|
|
|
| |
src/test/java/com/google/devtools/build/lib/skyframe/TreeArtifactBuildTest.java:154: error: local variable normalOutput is accessed from within inner class; needs to be declared final
touchFile(normalOutput);
http://ci.bazel.io/job/bazel-tests/BAZEL_VERSION=HEAD-jdk7,PLATFORM_NAME=linux-x86_64/290/console
--
MOS_MIGRATED_REVID=136588819
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
*** Reason for rollback ***
Suspected root cause for Windows bootstrap on Bazel CI breakage:
java.lang.NullPointerException
at com.google.devtools.build.lib.vfs.Path$1.run(Path.java:123)
http://ci.bazel.io/view/Bazel%20bootstrap%20and%20maintenance/job/Bazel/922/JAVA_VERSION=1.8,PLATFORM_NAME=windows-x86_64/console
*** Original change description ***
VFS: implement a Windows-specific Path subclass
The new subclass WindowsFileSystem.WindowsPath is
aware of Windows drives.
This change:
- introduces a new factory for Path objects so
FileSystems can return a custom implementation
that instantiates filesystem-specific Paths
- implements the WindowsPath subclass of Path that
is aware of Windows drives
- introduces the bazel.windows_unix_root JVM
argument that defines the MSYS root, which
defines the absolute Windows path that is the...
***
--
MOS_MIGRATED_REVID=136583352
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
*** Reason for rollback ***
Suspected root cause for Windows bootstrap on Bazel CI breakage:
java.lang.NullPointerException
at com.google.devtools.build.lib.vfs.Path$1.run(Path.java:123)
http://ci.bazel.io/view/Bazel%20bootstrap%20and%20maintenance/job/Bazel/922/JAVA_VERSION=1.8,PLATFORM_NAME=windows-x86_64/console
*** Original change description ***
VFS, WindowsFileSystem: fix UNIX_ROOT retrieval
Executing bash.exe directly instead of through
cmd.exe doesn't seem to work. This change fixes
that problem.
Fixes https://github.com/bazelbuild/bazel/issues/1463 (again)
--
MOS_MIGRATED_REVID=136581532
|
|
|
|
|
|
|
| |
are not lost.
--
MOS_MIGRATED_REVID=136579898
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=136575806
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
*** Reason for rollback ***
Broke Bazel CI:
ERROR: /home/ci/workspace/Bazel/JAVA_VERSION/1.8/PLATFORM_NAME/linux-x86_64/src/BUILD:284:1: in filegroup rule //src:srcs: non-test target '//src:srcs' depends on testonly target '//src/java_tools/junitrunner/java/com/google/testing/junit/junit4:srcs' and doesn't have testonly attribute set.
ERROR: Analysis of target '//scripts/packages:packages' failed; build aborted.
http://ci.bazel.io/view/Bazel%20bootstrap%20and%20maintenance/job/Bazel/JAVA_VERSION=1.8,PLATFORM_NAME=linux-x86_64/923/console
*** Original change description ***
Enforce that non test-only targets cannot depends on test-only targets
This behavior was not enforced since we open-sourced Bazel even though this was
documented and the attributed existed.
Also removed the incorrect part from the documentations: we do not enforce
restrictions on javatests package.
Fixes #1923.
RELNOTES[INC]: Non test-only targets can no longer depends on test-only targets.
--
MOS_MIGRATED_REVID=136574806
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=136574107
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=136523173
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We were including the linkstamp object file in the linker inputs on
the command line for the LTO indexing step, despite not adding the
linkstamp compile command or the files to the inputs and outputs,
causing an error. The linkstamp file is not compiled in ThinLTO mode.
The linkstamp is correctly added to the final native link.
The fix is to not pass the linkstampMap to the CppLinkVariablesExtension
for the LTO indexing step. Passing in the map causes the linkstamp
outputs to be added to the buildVariables, resulting in the outputs
being added to the command line via the featureConfiguration.
Added linkstamps to several of the ThinLTO tests.
--
MOS_MIGRATED_REVID=136480400
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=136475556
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Currently a call to "bazel" in an integration test means calling a (quite
hidden) function in test-setup.sh which actually calls "$bazel" defined
in "shell/bazel/testenv.sh" which is equal to "$(rlocation io_bazel/src/bazel)".
This is extremely confusing and error prone.
The new mechanism is to add a wrapper script to shell/bin called bazel
and export this directory to the PATH.
Moreover, not every test loads the same test environment, for instance consider
how bazel_query_test loads the test environment:
- Load shell/integration/testenv.sh which loads,
- shell/bazel/test-setup.sh which loads,
- shell/bazel/testenv.sh which loads,
- shell/unittest.bash which loads,
- shell/testenv.sh
Again this is error prone and specially hard to understand, in fact
each test writer needs to decide which of these testenv to load.
This change fixes all of this by having only one testenv.sh
and summarizing the test setup in integration_test_setup.sh.
Namely, for any new integration test, the developer
needs to load integration_test_setup to get the environment set up including
the unittest framework (also it helps to attract contributions).
This change also allows to open sourcing client_sigint_test: Since bazel was a
function client_sigint_test was using a wrong process id to interrupt
the build. The problem is that $! returns
bash's id instead of the id of the process running in the background
when using a function instead of an executable.
A few tests needed to be adapted to the new infrastructure.
--
MOS_MIGRATED_REVID=136470360
|
|
|
|
|
|
|
|
|
| |
*** Reason for rollback ***
Causes huge memory blow-up after the analysis phase in CppLinkAction (LinkCommandLine).
--
MOS_MIGRATED_REVID=136467836
|
|
|
|
|
|
|
|
|
|
|
| |
The "protobuf_python" target references a "protobuf_python_srcs" target in its deps, which does not exist, causing a build error when I tried to run "bazel build //src/main/...":
ERROR: /Users/philwo/src/bazel/third_party/protobuf/3.0.0/BUILD:70:1: in deps attribute of py_library rule //third_party/protobuf/3.0.0:protobuf_python: rule '//third_party/protobuf/3.0.0:protobuf_python_srcs' does not exist. Since this rule was created by the macro 'py_proto_library', the error might have been caused by the macro implementation in /Users/philwo/src/bazel/third_party/protobuf/protobuf.bzl:256:12.
I think the "python_srcs" target should actually be called "protobuf_python_srcs". With that change, the build works.
--
MOS_MIGRATED_REVID=136467485
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=136463385
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This behavior was not enforced since we open-sourced Bazel even though this was
documented and the attributed existed.
Also removed the incorrect part from the documentations: we do not enforce
restrictions on javatests package.
Fixes #1923.
RELNOTES[INC]: Non test-only targets can no longer depends on test-only targets.
--
MOS_MIGRATED_REVID=136459421
|
|
|
|
|
|
|
|
|
|
|
|
| |
This fixes #1888: on my laptop (Mac Book Air), the test is highly
flaky (6-7 tests fails out of 10) but with this change, I observe
no failures. I cannot explains it but it seems to be some kind
of race in the stream handling.
--
Change-Id: I09eb9d5a1c8468c7a0299966a5ea20e7a5fc86f0
Reviewed-on: https://bazel-review.googlesource.com/c/6696/
MOS_MIGRATED_REVID=136459264
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Before - failing to find completions in the relevant variable would always
force getting them fresh, regardless of whether they are present in the cache
because the `||` would always be true.
- in _bazel_get_options use a short circuit return when the variable is not
empty
- in _bazel_command, _bazel_help_topic, and _bazel_info_key nest the cache
conditions inside the if block checking for an empty varible
cleanup:
- Use the same 2 space indent for continued conditions in _bazel_get_options as
was already used in _bazel_command and the rest
Closes #1951.
--
Reviewed-on: https://github.com/bazelbuild/bazel/pull/1951
MOS_MIGRATED_REVID=136458728
|
|
|
|
|
|
|
| |
advertise and providers.
--
MOS_MIGRATED_REVID=136452596
|
|
|
|
|
|
|
| |
*** Reason for rollback ***
--
MOS_MIGRATED_REVID=136448178
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=136447767
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=136446429
|
|
|
|
|
|
|
|
|
| |
Move the Bazel-specific setup into a separate RuleSet and don't explicitly
depend on that - instead, implicitly call it's init() whenever we need a
valid RCP.
--
MOS_MIGRATED_REVID=136444074
|
|
|
|
|
|
|
|
|
|
|
|
| |
- add a convenience method to make adding options + fragment factories a bit
simpler
- sort the objc rules alphabetically
- split the j2objc rules from the objc rules
- unfortunately, the objc rules depend on the j2objc configuration, so that
has to stay
--
MOS_MIGRATED_REVID=136442577
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=136431274
|
|
|
|
|
|
|
| |
RELNOTES[NEW]: Files now have an "extension" property in Skylark.
--
MOS_MIGRATED_REVID=136425934
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=136416405
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=136402597
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=136401346
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=136400800
|
|
|
|
|
|
|
|
|
|
|
|
| |
Those BUILD files were broken in the default bazel set-up, preventing test on //src/test/...
The bazel_toolchain_test now rename those file.
Fixes #1957.
--
Change-Id: Ie0a1dbde349a307998fd4d42f73fffabc67c113c
Reviewed-on: https://bazel-review.googlesource.com/6693
MOS_MIGRATED_REVID=136379678
|
|
|
|
|
|
|
|
|
|
|
| |
Executing bash.exe directly instead of through
cmd.exe doesn't seem to work. This change fixes
that problem.
Fixes https://github.com/bazelbuild/bazel/issues/1463 (again)
--
MOS_MIGRATED_REVID=136364606
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In our bazel-srcs tar ball, files are packed with permissions 550.
Copying into a temporary directory may preserve permissions. So,
when overwriting one of those files in the temporary directory,
grant the user write permissions before writing to that file. Otherwise,
bootstrapping from //:bazel-srcs via ./compile.sh as normal user
will fail on some systems.
--
Change-Id: I70b4645c9bc3c25b9fd356bacdb959205c64b931
Reviewed-on: https://bazel-review.googlesource.com/#/c/6695
MOS_MIGRATED_REVID=136360945
|
|
|
|
|
|
|
|
|
|
|
| |
* remove "using std::" declarations from header files
* add missing "std::" to some string declarations at some header files
* add "using std::string;" to some source files where necessary
--
Change-Id: Ib64f62b5add499d6171fa922227194ac992fa542
Reviewed-on: https://bazel-review.googlesource.com/#/c/6630/
MOS_MIGRATED_REVID=136355671
|
|
|
|
|
|
|
|
|
| |
doesn't exist.
In addition to saving a filesystem operation, this removes a source of a potential filesystem inconsistency.
--
MOS_MIGRATED_REVID=136355008
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The new subclass WindowsFileSystem.WindowsPath is
aware of Windows drives.
This change:
- introduces a new factory for Path objects so
FileSystems can return a custom implementation
that instantiates filesystem-specific Paths
- implements the WindowsPath subclass of Path that
is aware of Windows drives
- introduces the bazel.windows_unix_root JVM
argument that defines the MSYS root, which
defines the absolute Windows path that is the
root of all Unix paths that Bazel creates (e.g.
"/usr/lib" -> "C:/tools/msys64/usr/lib") except
if the path is of the form "/c/foo" which is
treated as "C:/foo"
- removes all Windows-specific logic from Path
PathFragment is still aware of drive letters and
it has to remain so because it is unaware of file
systems.
WindowsPath restricts the allowed path strings to
absolute Unix paths where the first segment, if
any, is a volume specifier. From now on if Bazel
attempts to create a WindowsPath from an absolute
Unix path, Bazel will make it relative to
WindowsPath.UNIX_ROOT, unless the first component
is a single-letter name (e.g. "/c/foo" which is
"C:/foo").
Subclassing Path is necessary because a Unix-style
absolute path doesn't sufficiently define a full
Windows path, as it may be relative to any drive.
Fixes https://github.com/bazelbuild/bazel/issues/1463
--
MOS_MIGRATED_REVID=136350304
|