aboutsummaryrefslogtreecommitdiffhomepage
Commit message (Collapse)AuthorAge
* 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
* Fix broken build under Debian Wheezy.Gravatar Brian2015-09-28
| | | | | | | | | | | With the toolchain under Wheezy (GCC 4.7.2 and binutils 2.22), using both -fPIE and -fPIC does not create pic object files, which the build of Bazel seems to expect as of ee8fcd312eef51d2558c940f00d6381af0d52dff. -- Change-Id: I0a33d1bc6dc0e3cdcffb473d4e78dac1a6b8ab41 Reviewed-on: https://bazel-review.googlesource.com/#/c/1690/ MOS_MIGRATED_REVID=104093416
* Remove duplicate of me in AUTHORSGravatar Brian Silverman2015-09-28
| | | | | | | | | | It was caused by bad git configuration on my end for one of my commits... -- Change-Id: I2692085f996c9217216a7de9230cbff79b83ae14 Reviewed-on: https://bazel-review.googlesource.com/#/c/2050/ MOS_MIGRATED_REVID=104088950
* Simplify PackageFactoryApparatus.Gravatar Han-Wen Nienhuys2015-09-28
| | | | | -- MOS_MIGRATED_REVID=104086390
* 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
* Support for a black list of documented rulesGravatar Damien Martin-Guillerez2015-09-28
| | | | | | | | | | | Any rule in that black list will be skipped during build encyclopedia generation. This is for supporting outside documentation of rules that have a different documentation internally. -- MOS_MIGRATED_REVID=103958542
* 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
* Quick and dirty fix for #456.Gravatar John Sullivan2015-09-28
| | | | | | -- Reviewed-on: https://github.com/bazelbuild/bazel/pull/458 MOS_MIGRATED_REVID=103943941
* 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
* Port LabelValidatorTest.java to JUnit4.Gravatar Han-Wen Nienhuys2015-09-28
| | | | | -- MOS_MIGRATED_REVID=103943187
* 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
* Add PetitParser to usersGravatar Kristina Chodorow2015-09-25
| | | | | | | Also moved the corporate users up. -- MOS_MIGRATED_REVID=103940710
* Fix broken links to lib/Globals.html.Gravatar Googler2015-09-25
| | | | | -- MOS_MIGRATED_REVID=103940678
* Use JUnit4 for skylark tests.Gravatar Han-Wen Nienhuys2015-09-25
| | | | | | | -- Change-Id: I3cf7a8127da60d93005594d73abbdb8f7bda7cf2 Reviewed-on: https://bazel-review.googlesource.com/#/c/2041 MOS_MIGRATED_REVID=103940423
* 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
* Mention chroot in --batch description.Gravatar Han-Wen Nienhuys2015-09-25
| | | | | | | Fixes #134. -- MOS_MIGRATED_REVID=103935102
* 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
* [sass] Fixing typo in README.mdGravatar Derek Perez2015-09-25
| | | | | | -- Reviewed-on: https://github.com/bazelbuild/bazel/pull/482 MOS_MIGRATED_REVID=103931458
* Skylark rules can no longer overwrite built-in attributes.Gravatar Florian Weikert2015-09-25
| | | | | -- MOS_MIGRATED_REVID=103931317
* Minor usability fixes for the incremental deployment stub application:Gravatar Lukacs Berki2015-09-25
| | | | | | | | - Pass a File around for cacheDir instead of its String name. It was needlessly converted to String, which made the stub application fail if cacheDir was null (odd, but seems to happen) - Enhance a log message so that it contains more than zero information -- MOS_MIGRATED_REVID=103928940
* 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
* Removed new_file_suffix().Gravatar Florian Weikert2015-09-25
| | | | | -- MOS_MIGRATED_REVID=103926575
* Add a serialization proxy to RepositoryName instead of PackageIdentifier so ↵Gravatar Lukacs Berki2015-09-25
| | | | | | | | | that naked RepositoryName objects can also be serialized properly. Also store the initial offset a target pattern was parsed with. -- MOS_MIGRATED_REVID=103921930
* Clarify contract of EvaluationResult -- the same key can never be present in ↵Gravatar Janak Ramakrishnan2015-09-25
| | | | | | | both an "error" and "success" state. -- MOS_MIGRATED_REVID=103897656
* Prefetch and assert done on direct deps. Also print a more informative error ↵Gravatar Janak Ramakrishnan2015-09-25
| | | | | | | message if a previously known direct dep isn't done. -- MOS_MIGRATED_REVID=103880990
* Change Attribute.skipConstraintsCheck() (and similar methods)Gravatar Greg Estren2015-09-25
| | | | | | | | | | | | | to Attribute.skipPrereqValidatorCheck. This is to disambiguate the concept of "constraints" and keep the word consciously focused on Bazel's *new* constraint system. The changed methods refer to checks done by PrerequisiteValidator, which is basically an adhoc version of the "old" system (e.g. checking visibility) -- MOS_MIGRATED_REVID=103872412
* Make sure runtime shuts down on exception in uncaught exception handlerGravatar Michajlo Matijkiw2015-09-25
| | | | | | | RELNOTES: -- MOS_MIGRATED_REVID=103869828
* [Docker] Correctly append ./ prefix to dotted files in archivesGravatar Damien Martin-Guillerez2015-09-25
| | | | | | | Previously, dotted files were though of having the './' prefix. -- MOS_MIGRATED_REVID=103864015