aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/main/java/com/google/devtools/build/lib
Commit message (Collapse)AuthorAge
* Extract configuration fragment access logic into ConfigurationFragmentPolicy.Gravatar Michael Staib2015-09-30
| | | | | | | | | | | This is the first step toward giving aspects the ability to define their own required configuration fragments, extracting the required configuration metadata into a common place. This should be a no-op refactoring. -- MOS_MIGRATED_REVID=104249500
* Reintroduce an inconsistency check (albeit, in a weaker form) removed by a ↵Gravatar Nathan Harmata2015-09-30
| | | | | | | previous change that was trying to optimize away a filesystem call. -- MOS_MIGRATED_REVID=104246368
* Move actoolzip, momczip and swiftstdlibtoolzip to tools/xcode and convert ↵Gravatar Dave MacLachlan2015-09-30
| | | | | | | | | them to scripts instead of java apps. RELNOTES: actoolzip, momczip and swiftstdlibtoolzip have all been made into bash scripts and have been renamed to actoolwrapper, momcwrapper and swiftstdlibtoolwrapper respectively. The old versions will be deleted in a later change. -- MOS_MIGRATED_REVID=104225062
* Optimize how we mark files read-only upon action completion.Gravatar Eric Fellheimer2015-09-30
| | | | | -- MOS_MIGRATED_REVID=104219083
* Change uses of Lists#newArrayListWithExpectedSize to ↵Gravatar Nathan Harmata2015-09-30
| | | | | | | ArrayList#ArrayList(int) when it's clear the author intended (the now deprecated) Lists#newArrayListWithCapacity. Also change uses of Lists#newArrayListWithCapacity in the affected files to ArrayList#ArrayList(int). -- MOS_MIGRATED_REVID=104217692
* Limit the length of the output of Printer.printList().Gravatar Florian Weikert2015-09-30
| | | | | -- MOS_MIGRATED_REVID=104197461
* Replace 'Google Inc' by the 'Bazel Authors'Gravatar Damien Martin-Guillerez2015-09-30
| | | | | | | | | | This is the new conventions for Copyrights line and some were missed in the previous change. Generated by: find -type f -exec sed -i -E 's/Copyright 201([54]) Google Inc/Copyright 201\1 The Bazel Authors/' '{}' ';' -- MOS_MIGRATED_REVID=104189975
* Rename PackageManager.getTargetPatternEvaluator to newTargetPatternEval.Gravatar Ulf Adams2015-09-30
| | | | | | | | Also change the semantics to create a new one on every call; inject the offset where we were previously relying on hidden state in the SkyframeExecutor. -- MOS_MIGRATED_REVID=104188580
* Move loadingPhaseRunner from BlazeRuntime to CommandEnvironment.Gravatar Ulf Adams2015-09-30
| | | | | | | | | | The loadingPhaseRunner is stateless, and moving it prevents accidentally adding state to it. The TargetPatternEvaluator is not stateless, but is currently retained by the SkyframePackageManager, so we still keep that state across invocations (see follow-up CL). -- MOS_MIGRATED_REVID=104187659
* Adds a mechanism for invocation policy. The policy is taken through the ↵Gravatar Alex Humesky2015-09-30
| | | | | | | --invocation_policy startup flag and allows an application invoking Bazel to set or override flag values (whether from the command line or a bazelrc). -- MOS_MIGRATED_REVID=104160290
* Rollback of "Propagates cc_library linkopts attribute to dependent ↵Gravatar Chris Parsons2015-09-30
| | | | | | | objc_libraries.": breaks certain objc_binary build targets. -- MOS_MIGRATED_REVID=104146276
* Don't crash when building environment groups directlyGravatar Greg Estren2015-09-30
| | | | | | | | | | (following PackageGroup's precedent). Also cleanup: generalize the pattern by which we determine non-configurable target types. -- MOS_MIGRATED_REVID=104125803
* Make test setup name public.Gravatar Han-Wen Nienhuys2015-09-30
| | | | | -- MOS_MIGRATED_REVID=104119503
* Move test setup script up in the hierarchy.Gravatar Han-Wen Nienhuys2015-09-30
| | | | | -- MOS_MIGRATED_REVID=104107730
* BlazeRuntime.reporter is now null in production.Gravatar Ulf Adams2015-09-30
| | | | | -- MOS_MIGRATED_REVID=104102106
* Adds the textual_hdrs attribute, which are treated as regular headers but ↵Gravatar Googler2015-09-30
| | | | | | | | | not currently included in the module map. On Xcode 7, we should add them to the module map as additionalExportedHeaders, since it supports the textual keyword. RELNOTES: Add support for objc textual headers, which will not be compiled when modules are enabled. -- MOS_MIGRATED_REVID=104100551
* Set the SkyframeActionExecutor.reporter in prepareForExecution.Gravatar Ulf Adams2015-09-30
| | | | | | | Remove a lot of now-unused Reporter parameters. -- MOS_MIGRATED_REVID=104100061
* Remove BuildView.packageManager, which was only used for testing.Gravatar Ulf Adams2015-09-30
| | | | | | | | Now that we have an EventHandler everywhere, we can just use the bridge code to call into the usual PackageManager. -- MOS_MIGRATED_REVID=104098660
* BuildView; move the methods around.Gravatar Ulf Adams2015-09-28
| | | | | | | | Production API at the top, then ide_build_info, and testing at the bottom. This is separate from the refactoring to make both easier to review. -- MOS_MIGRATED_REVID=104095498
* Refactor BuildView; clearly identify the ide_build_info and the testing API.Gravatar Ulf Adams2015-09-28
| | | | | | | Also inject the EventHandler all the way through to the SkyframeExecutor. -- MOS_MIGRATED_REVID=104094731
* RELNOTES: remove webstatusserver (--use_webstatusserver). Gravatar Han-Wen Nienhuys2015-09-28
| | | | | | | It seems unused, and the Bazel dashboard is more useful, as can persist results. -- MOS_MIGRATED_REVID=104085798
* Add Android IDL jar outputs.Gravatar Googler2015-09-28
| | | | | | | | | | These outputs are a jar and source jar for the results of aidl processing. This is used to add aidl output to IDEs separate from the source code, similar to annotation output (gen jars). -- MOS_MIGRATED_REVID=104024453
* Move the BuildView over from BlazeRuntime to CommandEnvironment.Gravatar Ulf Adams2015-09-28
| | | | | | | | | It doesn't contain any state anymore, so creating a new instance per command is safe. On the other hand, it would be just as safe to merge the remaining code into SkyframeBuildView, which we might do in the future. -- MOS_MIGRATED_REVID=104010694
* Move the remaining state from BuildView to SkyframeBuildView.Gravatar Ulf Adams2015-09-28
| | | | | -- MOS_MIGRATED_REVID=104009600
* Remove BuildView.skyframeCacheWasInvalidated.Gravatar Ulf Adams2015-09-28
| | | | | | | | | | | | It was mainly used for testing during the transition phase to Skyframe. The only reason I can see to keep it would be to have test coverage that changing the configuration drops the configured targets. However, do we even want that? I tried to find out whether it's safe to remove that check (the corresponding comment is outdated), but couldn't find where (if anywhere) we're doing garbage collection of old configured targets. -- MOS_MIGRATED_REVID=104009210
* Remove BuildView.binTools, which was only used for tests.Gravatar Ulf Adams2015-09-28
| | | | | -- MOS_MIGRATED_REVID=104008237
* Give a nice error message on double included files in objc rules.Gravatar Peter Schmitt2015-09-28
| | | | | -- MOS_MIGRATED_REVID=103986912
* Inject an event handler into the getErrorFreeVisitedPackages call.Gravatar Ulf Adams2015-09-28
| | | | | | | This means we no longer need an event handler on the object itself. -- MOS_MIGRATED_REVID=103982299
* Refactor the LoadingPhaseRunner: create a new TransitivePackageLoader per run.Gravatar Ulf Adams2015-09-28
| | | | | | | | | | | | | | The only implementation of TransitivePackageLoader at this point is in SkyframeLabelVisitor, which by itself does not keep any state; it relies on Skyframe to do all the caching. Creating a new one per run should ensure that we don't accidentally keep state there, and also reduces memory consumption (a little), because it can now be garbage collected after the loading phase. As a side effect, we no longer need to create an instance if we're using the simplified loading phase. -- MOS_MIGRATED_REVID=103979794
* Clean up Aspect checks in query tests.Gravatar Eric Fellheimer2015-09-28
| | | | | -- MOS_MIGRATED_REVID=103977080
* Remove unnecessary env.valuesMissing() call.Gravatar Nathan Harmata2015-09-28
| | | | | -- MOS_MIGRATED_REVID=103975426
* Make explicit that the SkyframeTransitivePackageLoader needs an EventHandler.Gravatar Ulf Adams2015-09-28
| | | | | -- MOS_MIGRATED_REVID=103975222
* Use the same method as LoadedPackageProvider.Bridge to look up targets.Gravatar Ulf Adams2015-09-28
| | | | | | | | | | This unfortunately requires injecting an EventHandler into the getTargetForLabel call, which is not super nice. On the other hand, it's not clear how this can be much better - looking up targets in the cycle reporter doesn't lend itself to prettiness. -- MOS_MIGRATED_REVID=103965373
* Inject an EventHandler rather than using the one on SkyframeExecutor.Gravatar Ulf Adams2015-09-28
| | | | | | | In preparation for removing SkyframeExecutor.errorEventListener. -- MOS_MIGRATED_REVID=103960561
* Fix attribute's rule nameGravatar Kristina Chodorow2015-09-28
| | | | | | | Gah. -- MOS_MIGRATED_REVID=103959841
* Expose proto sources in the Skylark providerGravatar Laurent Le Brun2015-09-28
| | | | | -- MOS_MIGRATED_REVID=103947930
* Move BuildView.getActionGraph to SkyframeExecutor.Gravatar Ulf Adams2015-09-28
| | | | | | | | This is another case of uninterruptible evaluation, which directly accesses SkyframeExecutor.errorEventListener. -- MOS_MIGRATED_REVID=103946310
* Resolve labels in linkopts against either 'deps' or 'srcs'Gravatar Googler2015-09-28
| | | | | | | RELNOTES[NEW]: labels in "linkopts" may match any label in either "deps" or "srcs" to be considered valid. -- MOS_MIGRATED_REVID=103945104
* Disable include scanning for assembler-without-preprocessor source.Gravatar Googler2015-09-28
| | | | | | | | The scanner only looks for C preprocessor directives, but most assemblers allow '.include' assembly directives, and those aren't found by the scanner. So skip the include scanner for assembly files that don't want C preprocessing, because correctly declared inclusions are to be preferred anyway. -- MOS_MIGRATED_REVID=103944189
* Find the test setup script based on its basename and not its precise exec path.Gravatar Lukacs Berki2015-09-28
| | | | | | | This is necessary so that the existence of a script //tools/test:test-setup.sh is not hardcoded into Bazel, which is in turn necessary so that the Bazel tools (//tools/...) are not required to be in the main repository. -- MOS_MIGRATED_REVID=103943618
* Split the PackageManager type hierarchy; no longer inherit LoadedPackageProv.Gravatar Ulf Adams2015-09-28
| | | | | | | | | | | | | This limits the exposure of LoadedPackageProvider, such that there will be no regressions in the use of getLoadedTarget. Unfortunately, fully removing LoadedPackageProvider is more work than I'm willing to take on right now, and this is the cleanest intermediate solution I could come up with. This unblocks my other work (removing SkyframeExecutor.errorEventHandler). Someone else will have to shave this yak. -- MOS_MIGRATED_REVID=103943375
* Also ship the linker for LTO backend actions.Gravatar Han-Wen Nienhuys2015-09-28
| | | | | -- MOS_MIGRATED_REVID=103942611
* Make rules construction more flexible.Gravatar Miguel Alcon Pinto2015-09-28
| | | | | -- MOS_MIGRATED_REVID=103942367
* Rationalize copyright headersGravatar Damien Martin-Guillerez2015-09-25
| | | | | | | | | | | The headers were modified with `find . -type f -exec 'sed' '-Ei' 's|Copyright 201([45]) Google|Copyright 201\1 The Bazel Authors|' '{}' ';'` And manual edit for not Google owned copyright. Because of the nature of ijar, I did not modified the header of file owned by Alan Donovan. The list of authors were extracted from the git log. It is missing older Google contributors that can be added on-demand. -- MOS_MIGRATED_REVID=103938715
* Use Label.parseAbsolute() to determine whether command line options that can ↵Gravatar Lukacs Berki2015-09-25
| | | | | | | | | take absolute paths and labels are labels. This makes them work with labels not in the main repository (i.e. @repo//pkg:target). The alternative would have been to simply add .startsWith("@"), but why try to parse labels in a lame and limited way if we can do the right thing. -- MOS_MIGRATED_REVID=103934380
* Improved error reporting in RuleContext:Gravatar Florian Weikert2015-09-25
| | | | | | | | - Unified duplicate code from RuleContext and RuleContext.Builder in a new class, RuleContext.ErrorReporter - Added the BUILD file location to error/warning messages if the offending rule was created by a macro -- MOS_MIGRATED_REVID=103934375
* Do not hardcode the path to genrule-setup.sh in the implementation of genrule.Gravatar Lukacs Berki2015-09-25
| | | | | | | I don't know why it was implemented like that in the first place. -- MOS_MIGRATED_REVID=103934038
* Implemented copy constructor for Skylark dictionaries: new_dict = dict(old_dict)Gravatar Florian Weikert2015-09-25
| | | | | -- MOS_MIGRATED_REVID=103932279
* Skylark rules can no longer overwrite built-in attributes.Gravatar Florian Weikert2015-09-25
| | | | | -- MOS_MIGRATED_REVID=103931317
* Tweak the installation order of split APKs so that there is less logcat spam ↵Gravatar Lukacs Berki2015-09-25
| | | | | | | during "mobile install --split_apks". -- MOS_MIGRATED_REVID=103927507