| Commit message (Collapse) | Author | Age |
|
|
|
|
|
|
| |
Native dependencies should be collected from both deps and exports.
--
MOS_MIGRATED_REVID=110279453
|
|
|
|
|
|
|
| |
targets.
--
MOS_MIGRATED_REVID=110275427
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
documentation passed to templates.
* This simplifies the logic in BuildEncyclopediaProcessor, reducing the number
of various different data structures we use for generating different pages.
* This also makes RuleFamily more similar to the BuildLanguage proto, which we
we may consider using in the future for the native rule docgen since the
Skylark docgen will be using BuildLanguage.
--
MOS_MIGRATED_REVID=110265727
|
|
|
|
|
|
|
| |
Type "map" doesn't exist. "struct" is not allowed.
--
MOS_MIGRATED_REVID=110265638
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=110261986
|
|
|
|
|
|
|
|
|
|
|
| |
to aapt.
Turns out, it does the wrong thing when using --nodistinct_host_configuration. Not that option should ever be used, but still.
Fixes #523. Probably.
--
MOS_MIGRATED_REVID=110232316
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=110174447
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=110155243
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=110153136
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=110141376
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=110135504
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
directory that I should populate". The directory itself is not created because local_repository actually puts a symlink in its place.
As a side effect, make HTTP downloading, git cloning and archive decompressing not be SkyFunctions. This is necessary because it needs to be the RepositoryFunction and not a
dependent SkyFunction that populates the output directory, because it that's the case what happens is:
1. RepositoryDelegatorFunction cleans up the directory and prepares it for RepositoryFunction
2. RepositoryFunction calls env.getValue(<function that populates the directory>)
3. That value hasn't been computed yet, thus RepositoryDelegatorFunction returns early
4. The function that populates the directory is called
5. RepositoryDelegatorFunction is restarted
6. RepositoryDelegatorFunction cleans up the directory
7. RepositoryFunction calls env.getValue(), and nothing is done because the value has already been computed
8. RepositoryDelegatorFunction proudly returns, even though the directory is actually empty
Another way to solve this problem would be to make RepositoryFunction not clean the directory up on Skyframe restarts, but that means that we'd need to keep state somewhere, which doesn't strike me as a particularly great idea because let's keep state outside of Skyframe only when absolutely necessary (e.g. the marker files for cross-server instance persistence of downloaded repositories). That "somewhere" could either be a member variable of RepositoryDelegatorFunction or the file system.
Note that this change causes external communication to be re-done in a few more cases than before (see that changes to the test cases), but I'd rather we be correct and simple than fast. We can optimize things later if needed and there is enough complexity going around, thank you very much.
--
MOS_MIGRATED_REVID=110134397
|
|
|
|
|
|
|
|
|
|
|
| |
always point to the main repository.
They now all point to @bazel_tools anyway, so there is no need.
Fixes #700 (what a delightfully round bug number!)
--
MOS_MIGRATED_REVID=110132857
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=110025690
|
|
|
|
|
|
|
| |
Prevents overhead of extra Labels in builds with lots of selectors
--
MOS_MIGRATED_REVID=110011063
|
|
|
|
|
|
|
| |
null but the SkyFunction that called it returns non-null.
--
MOS_MIGRATED_REVID=110009359
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=110008191
|
|
|
|
|
|
|
|
|
| |
- If we have a class object available, we can use .cast(Object)
- Only store the required options objects in CppConfigurationParameters
rather than keeping the entire BuildOptions
--
MOS_MIGRATED_REVID=109981236
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=109942021
|
|
|
|
|
|
|
| |
RELNOTES[NEW]: Evaluate the query expression in a file by passing --query_file=<file> to query
--
MOS_MIGRATED_REVID=109932769
|
|
|
|
|
|
|
| |
Fixes #650, #551, and the stack trace parts of #625 and #330.
--
MOS_MIGRATED_REVID=109925156
|
|
|
|
|
|
|
|
|
|
|
| |
Make the python binary stub compatible with both python2 and python3.
Fixes #638
--
Change-Id: Ic66d942deb43e131c8c442850acebe40adda9739
Reviewed-on: https://bazel-review.googlesource.com/#/c/2370/
MOS_MIGRATED_REVID=109923063
|
|
|
|
|
|
|
| |
Reduces garbage.
--
MOS_MIGRATED_REVID=109914243
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=109909215
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=109908202
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=109900770
|
|
|
|
|
|
|
| |
Reduces garbage.
--
MOS_MIGRATED_REVID=109898521
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=109897602
|
|
|
|
|
|
|
| |
execroot of another workspace.
--
MOS_MIGRATED_REVID=109885272
|
|
|
|
|
|
|
|
|
| |
package and the prefix of external packages in the execroot.
This is in preparation for renaming that path segment from "external" to "bazel-external" so that we can provide a symlink to it from the source tree so that the source tree looks like the execroot.
--
MOS_MIGRATED_REVID=109882753
|
|
|
|
|
|
|
| |
"overlaid BUILD files" hack in RepositoryValue.
--
MOS_MIGRATED_REVID=109877252
|
|
|
|
|
|
|
|
|
| |
transitively transient". Some followup changes will use this method.
Previously, ErrorInfo#isTransient was only used internally in ParallelEvaluator; I think this method was originally added to ErrorInfo solely for the sake of convenience.
--
MOS_MIGRATED_REVID=109840031
|
|
|
|
|
|
|
|
|
| |
Presize GroupedListHelper sets when initialized from collections. Use
CompactHashSets throughout. Avoid primitive autoboxing in Precondition
statements.
--
MOS_MIGRATED_REVID=109835986
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=109835697
|
|
|
|
|
|
|
| |
Such providers have a create() method which returns an object whose runtime class differs from its compile-time class, which makes the shortcut methods behave unexpectedly.
--
MOS_MIGRATED_REVID=109815042
|
|
|
|
|
|
|
|
|
| |
Previously we created this collection for each AggregatingAttributeMapper,
which we create at least every attribute encountered. Calculate the collection
up front to avoid wasting time and memory.
--
MOS_MIGRATED_REVID=109805907
|
|
|
|
|
|
|
|
|
| |
Also, in GroupedList, short-circuit expensive group equality check
with a reference check, saving time and garbage when groups are the
same object.
--
MOS_MIGRATED_REVID=109795332
|
|
|
|
|
|
|
|
|
| |
$OUTPUT_BASE/execroot/<workspace name> instead of $OUTPUT_BASE/<workspace name>.
This makes it possible to run Bazel in directories whose name clashes with one of the pre-existing files in the output base. Note that this option is turned off by default for now.
--
MOS_MIGRATED_REVID=109791053
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In particular:
- Separate the implementation of maven_server into a RepositoryFunction and one that creates the MavenServerValue (ideally, maven_server wouldn't exist but we'll have to make to for the time being)
- Refactor the logic of determining whether an external repository needs to be re-fetched to RepositoryDelegatorFunction
- Make RepositoryFunctions not be SkyFunctions anymore (they are called from RepositoryDelegatorFunction, though, who *is* a SkyFunction)
- Add a Skyframe dirtiness checker that makes --nofetch RepositoryValues not be cached
- Add a bunch of test cases and javadoc
There is only one wart that I know of that remains: changes to BUILD files of new_* repository rules that weren't refetched when their RepositoryValue was initiall created on server restart won't take effect. This is because we don't add those BUILD files to the created RepositoryValue. This will fix itself once the ExternalFilesHelper refactoring is submitted.
--
MOS_MIGRATED_REVID=109768345
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=109722614
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=109715685
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Blaze discovers inputs for some actions when running with some
strategies. Those actions should not discover additional inputs
when they run, regardless of the strategy they end up using.
There are now no known legitimate cases of such additional input
discovery, so we should reinstate this check and find new ones :)
We also change the failure mode to be a normal error rather than
a crash. This error does indicate a tooling issue, and a small
chance of incorrect builds, but it doesn't create such an
inconsistent state that a crash is warranted.
--
Change-Id: I5d498d2fc1c5e23bfb5d77971f866c2027cbf03a
Reviewed-on: https://bazel-review.googlesource.com/#/c/2500/3
MOS_MIGRATED_REVID=109703508
|
|
|
|
|
|
|
| |
short-circuit check when all package paths have good diff information and no external files have been seen.
--
MOS_MIGRATED_REVID=109703164
|
|
|
|
|
|
|
| |
to crash or exit abruptly. It just confuses users and could potentially cause other problems.
--
MOS_MIGRATED_REVID=109702171
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=109690378
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
*** Reason for rollback ***
Broke some alipo builds. I had to leave in the change to TransitiveTargetFunctionTest, which is relied upon by follow-up CLs.
*** Original change description ***
Pre-trim build options for all remaining calls to ConfigurationFragment.key. Move
the trimming logic into key() itself to:
a) eliminate code redundancy
b) guarantee all future calls to key() also do this.
--
MOS_MIGRATED_REVID=109683849
|
|
|
|
|
|
|
|
|
| |
Fixes #352.
RELNOTES: Files in external repositories are now treated as mutable, which will make the correctness guarantees of using external repositories stronger (existent), but may cause performance penalties.
--
MOS_MIGRATED_REVID=109676408
|
|
|
|
|
|
|
|
|
| |
They accumulate indefinitely and take up space.
--
Change-Id: I74c7ffd795b21793f8654e0e30785e1b8ba99ca4
Reviewed-on: https://bazel-review.googlesource.com/#/c/2510/
MOS_MIGRATED_REVID=109656651
|
|
|
|
|
| |
--
MOS_MIGRATED_REVID=109649366
|
|
|
|
|
|
|
| |
auto-complete flags
--
MOS_MIGRATED_REVID=109595089
|