aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/main
Commit message (Collapse)AuthorAge
* [Android] Support aapt-generated main dex specs.Gravatar Googler2016-05-05
| | | | | | | | Add bazel support for using the "aapt -D" command to generate a proguard specification for components which need to be in the main dex. -- MOS_MIGRATED_REVID=121531584
* Rename setHasServices() to the more appropriate allowServices().Gravatar Carmi Grushko2016-05-05
| | | | | -- MOS_MIGRATED_REVID=121524513
* Provide .pcm files as input to LIPO builds.Gravatar Googler2016-05-05
| | | | | | | | | | | | | | | Previously, bazel would be internally inconsistent regarding whether it thought module files were in use for LIPO builds. The merged CppCompilationContext would not list .pcm files as inputs, because its useHeaderModules flag was forced to false, but the CppModel would expand %module_file to a list of .pcm files, because it thought that modules were in use. This results in LIPO build failures due to missing files. RELNOTES: Fix interaction between LIPO builds and C++ header modules. -- MOS_MIGRATED_REVID=121513092
* Fix wrong Skylark documentation for environment_group.Gravatar Greg Estren2016-05-05
| | | | | -- MOS_MIGRATED_REVID=121502983
* Allow modules to inject a custom AttributeContainer.Gravatar Janak Ramakrishnan2016-05-04
| | | | | -- MOS_MIGRATED_REVID=121497233
* cpp: fix documentation wording for data members in GlobalVariablesGravatar Thiago Farina2016-05-04
| | | | | | | | | | | | Some had incorrect words, or were missing a word and the end period. Make it match with the descriptions in CommonCommandOptions.java. Add a short documentation comment to |restart_reason| field. -- Change-Id: Iac802eb3ed4f72dfd402e85d2098a5616b40a9de Reviewed-on: https://bazel-review.googlesource.com/3530 MOS_MIGRATED_REVID=121478703
* Switch to using ../repo-name syntax for runfilesGravatar Kristina Chodorow2016-05-04
| | | | | -- MOS_MIGRATED_REVID=121475668
* Remove the fields of CppCompileAction relating to "extra system include ↵Gravatar Lukacs Berki2016-05-04
| | | | | | | | | prefixes" and compiler plugins. They were dead code. -- MOS_MIGRATED_REVID=121473393
* Remove the apparently unused LibraryToLink#isSolibSymlink() method.Gravatar Lukacs Berki2016-05-04
| | | | | -- MOS_MIGRATED_REVID=121472172
* Make the Blaze server kill the previous instance if the start time file ↵Gravatar Lukacs Berki2016-05-04
| | | | | | | | | doesn't exist. It may be an old version that doesn't know how to write them yet. -- MOS_MIGRATED_REVID=121462577
* Fix manifest_values.applicationId not being picked up by ↵Gravatar Andrew Pellegrini2016-05-04
| | | | | | | ApplicationManifest#getOverridePackage for building split and stub manifests. -- MOS_MIGRATED_REVID=121426669
* Adds the compiler to the key for selecting the cc_toolchain label inGravatar Alex Humesky2016-05-04
| | | | | | | | | | | cc_toolchain_suite. This will allow selecting the correct toolchain based on --compiler / --android_compiler. RELNOTES: The key for the map to cc_toolchain_suite.toolchains is now a string of the form "cpu|compiler" (previously, it was just "cpu"). -- MOS_MIGRATED_REVID=121418076
* Skip building JackAspect when the base rule already provides Jack libraries.Gravatar Michael Staib2016-05-03
| | | | | | | | | | | | | | | | | | | | | | | | | Context: AndroidLibrary provides JavaSourceInfoProvider, which JackAspect wants. But it doesn't advertise it, so it will never have JackAspect applied. This is good, because they both have an :android_sdk attribute and want to provide JackLibraryProvider. Rules in the external repository created with bind() don't have their real advertisements, but instead are considered to advertise everything. So, when an android_library depends on another android_library through a bind() alias... Boom. This avoids the boom by making JackAspect bail out immediately if its current target looks like an Android rule. It wouldn't be doing anything useful anyway if the base rule already provides Jack libraries. Fixes #1222. -- MOS_MIGRATED_REVID=121401981
* List Closure Rules in docs as JavaScriptGravatar Justine Tunney2016-05-03
| | | | | -- MOS_MIGRATED_REVID=121397798
* Add attribute validation of mandatory native providersGravatar Liam Miller-Cushon2016-05-03
| | | | | | | and use it to validate that :java_toolchain has a JavaToolchainProvider. -- MOS_MIGRATED_REVID=121396726
* When creating the exec root symlink forest, take account of the empty ↵Gravatar Janak Ramakrishnan2016-05-03
| | | | | | | | | package's package root when creating top-level directories. Otherwise, if the empty package references files in those top-level directories, they would be hidden. Fixes #1221 -- MOS_MIGRATED_REVID=121392128
* Introduces a registry of native providers to the BuildConfiguration, which ↵Gravatar Cal Peyser2016-05-03
| | | | | | | can be added to by fragments. Uses the registry to enable native access to TranstiveProviderInstances from Skylark. In particular, makes ObjcProvider accessible to skylark rule implementations. -- MOS_MIGRATED_REVID=121390911
* Add a //file target to external jar repositoriesGravatar Colin Marc2016-05-03
| | | | | | | | | | | | | | Fixes https://github.com/bazelbuild/bazel/issues/990. It's sometimes useful to be able to access the file pulled down with maven_jar; for example, scala macro jars don't work with ijar, so it's necessary to pull them into the compile-time classpath unharmed. -- Change-Id: I1d9e1cb30ea6fa56c18cbfcc68ee9b203524dfd4 Reviewed-on: https://bazel-review.googlesource.com/#/c/3550 MOS_MIGRATED_REVID=121379551
* Redesign InfoItem to use classes instead of the InfoKey enum. This way the ↵Gravatar Luis Fernando Pino Duque2016-05-03
| | | | | | | | | InfoItems can now take parameters. -- MOS_MIGRATED_REVID=121379097
* Set the default workspace name to __main__Gravatar Kristina Chodorow2016-05-03
| | | | | | | | | | | | | See https://github.com/bazelbuild/bazel/wiki/Updating-the-runfiles-tree-structure for more details about this change. RELNOTES[INC]: Bazel will now default to using __main__ as the workspace name (instead of "", as previously) if a project does not provide a workspace name in the WORKSPACE file. The repository's runfiles will appear under x.runfiles/__main__/. -- MOS_MIGRATED_REVID=121376094
* --Gravatar Googler2016-05-03
| | | | MOS_MIGRATED_REVID=121373163
* Rollback of commit 140634eccee693b567ba7591df550f7b72db1612.Gravatar Lukacs Berki2016-05-03
| | | | | | | | | | | | | *** Reason for rollback *** Breaks backwards compatibility. *** Original change description *** Collect logic related to extra Proguard outputs in one place. -- MOS_MIGRATED_REVID=121363782
* --Gravatar Luis Fernando Pino Duque2016-05-03
| | | | MOS_MIGRATED_REVID=121353990
* Prioritize kwarg presence error in invokeObjectMethod.Gravatar Googler2016-05-03
| | | | | | | | | | | I believe the original intention of this error check was to always fail if kwargs were present (while Skylark doesn't support kwargs, at least). However, the current code will most likely fail at findJavaMethod because only the args (i.e. not the kwargs) are used to find a matching method. This issue has already been encountered on the bazel-discuss group (g/bazel-discuss/hHyf2txlS70). With the proposed modification, I believe any kwargs provided to Skylark methods will cause the kwarg-specific error. -- MOS_MIGRATED_REVID=121345719
* Set workspace name for http_file, http_jar, and maven_jarGravatar Kristina Chodorow2016-05-03
| | | | | | | | | | | | | | | | Fixes #1166. While I was here, I also: * Renamed the implementations of Decompressor to show that they are Decompressors, not SkyFunctions (they used to be, it's confusing to have them still have the -Function suffix). * Added a unit test and moved the testing target to the rule/repository subdirectory, since it was a good chance to break off a tiny chunk of the monolithic BUILD file in lib/. -- MOS_MIGRATED_REVID=121306165
* Adds a new flag for generation of dSYMs for all apple rules.Gravatar Dmitry Shevchenko2016-05-03
| | | | | | | | * This flag forces -g and enables dSYM bundle actions. * All instances of breakpad generation are now gated by the old --objc_generate_debug_symbols option. -- MOS_MIGRATED_REVID=121304681
* Remove documentation for "abi" and "abi_deps".Gravatar Greg Estren2016-05-03
| | | | | | | | | These attributes are going away imminently. select() is the new abi_deps. -- MOS_MIGRATED_REVID=121294764
* Make compatible_with / restricted_to nonconfigurable.Gravatar Greg Estren2016-05-03
| | | | | | | | | | | | | | | | As of 18c277f, we have special logic for mixing selects and constraints. With that logic, a multiplaform libray declares all environments it supports across all select paths, then each path supports some subset of those environments. With this approach, there's no need for compatible_with = select(). So we'd like to disallow that pattern (and all its downsides) in preference for what 18c277f allows. -- MOS_MIGRATED_REVID=121294433
* Expose platform frameworks dir to Skylark.Gravatar Dmitry Shevchenko2016-05-03
| | | | | | | | * Adds a new method to get Xcode's developer frameworks dir. * Removes platform_dir() as it cannot be used in Skylark as-is - there's no way to get the platform name it requires. -- MOS_MIGRATED_REVID=121275439
* Linux-specific: check if the stray server process we are about to kill -9 is ↵Gravatar Lukacs Berki2016-05-02
| | | | | | | | | | | actually a server process. This should be implemented for other OSes, too, but OS X seems to lack a procfs and it's not clear how to discover anything about a process based on its PID and of course, Windows is a wholly different cup of tea. More work for #930. -- MOS_MIGRATED_REVID=121262673
* Various cleanups and refactorings in the client:Gravatar Lukacs Berki2016-05-02
| | | | | | | | | | | | - Made the control flow much simpler and more understandable - Added some documentation about the interplay of the client and the server - Abstracted out POSIX mechanisms from blaze.cc so that they can be implemented properly on Windows - Added assertions that the methods on BlazeServer are called when they should be Polish for #930. -- MOS_MIGRATED_REVID=121256601
* stub_template.txt: replace A if condition else B statement with if statementGravatar Yun Peng2016-05-02
| | | | | | | Python version priors to 2.5 doesn't support ternary conditional operator. -- MOS_MIGRATED_REVID=121256017
* Delay retrieval of the blacklist patterns file from the graph until we ↵Gravatar Janak Ramakrishnan2016-05-02
| | | | | | | actually need it. -- MOS_MIGRATED_REVID=121160209
* In InvocationPolicyEnforcer#enforce, instead of just checking the direct ↵Gravatar Janak Ramakrishnan2016-05-02
| | | | | | | | | | | command to see if a flag policy applies, check whether the flag applies by seeing if its list of commands matches one of the commands in the hierarchy. This avoids the tedious and brittle specification of all commands that may use a flag, while providing better filtering out of inapplicable flags. RELNOTES: A FlagPolicy specified via the --invocation_policy flag will now match the current command if any of its commands matches any of the commands the current command inherits from, as opposed to just the current command. -- MOS_MIGRATED_REVID=121159131
* When we don't have an OutputService, precompute and cache the ↵Gravatar Nathan Harmata2016-05-02
| | | | | | | | | | | assumed-to-be-immutable filesystem type of output_base. When we do have an OutputService, defer computation of the OutputService's dynamic filesystem type until we actually need it at the start of a "build" or "test" command. This saves ~450us on each invocation, according to my benchmarks. -- MOS_MIGRATED_REVID=121156323
* Memoize the OptionsData per BlazeCommand.Gravatar Nathan Harmata2016-05-02
| | | | | | | | | This saves the cost of (1) collecting all Options classes and (2) getting all their @Option annotations. Note that there is no savings on reflection costs, since that's already memoized internally by OptionsParser. This saves ~250us per Blaze invocation. -- MOS_MIGRATED_REVID=121153156
* Use ConstraintSemantics.getSupportedEnvironments() in ↵Gravatar Liam Miller-Cushon2016-05-02
| | | | | | | supportsJava7TargetEnvironment -- MOS_MIGRATED_REVID=121150741
* In apple_binary, propagate cc_library dependencies for all child ↵Gravatar Chris Parsons2016-05-02
| | | | | | | configurations of the split transition -- MOS_MIGRATED_REVID=121146341
* Clean up J2ObjC proto support.Gravatar Rumou Duan2016-05-02
| | | | | -- MOS_MIGRATED_REVID=121146085
* Move android deploy info from the default files to build to its own output groupGravatar Alex Humesky2016-05-02
| | | | | | | | named "android_deploy_info" so that the .deployinfo.pb file doesn't show up in bazel output. -- MOS_MIGRATED_REVID=121140355
* Clean up and simplify InstrumentedFileManifestAction.Gravatar Ulf Adams2016-04-29
| | | | | | | | | | This fixes an incremental correctness bug in the action, namely that the cache key didn't capture all the data written to the file. Also adds a test (not publicly visible yet). -- MOS_MIGRATED_REVID=121110495
* Clarify plmerge variable substitution behavior.Gravatar Peter Schmitt2016-04-29
| | | | | -- MOS_MIGRATED_REVID=121074077
* Tokenize javacopts in JavaToolchainDataParserGravatar Liam Miller-Cushon2016-04-29
| | | | | -- MOS_MIGRATED_REVID=121059106
* Adds the text output parameter and a partial manifest to the action key for ↵Gravatar Alex Humesky2016-04-29
| | | | | | | ApkManifestAction, so that the key correctly changes when the structure of the manifest changes. -- MOS_MIGRATED_REVID=121058425
* Let select() "unset" values via: select({"//some:condition: None }).Gravatar Greg Estren2016-04-29
| | | | | | | | | | | | | | This not only uses the default value when applicable, but also causes ConfiguredAttributeMapper.isAttributeValueExplicitlySpecified to return false. Note the default value can come from two places: from the rule definition if specified, otherwise from the type default. RELNOTES[NEW]: select({"//some:condition: None }) is now possible (this "unsets" the attribute). -- MOS_MIGRATED_REVID=121029815
* --Gravatar Carmi Grushko2016-04-29
| | | | MOS_MIGRATED_REVID=121029385
* Inject the Constants.TOOLS_REPOSITORY in SkylarkRuleClassFunctions.testBaseRule Gravatar Luis Fernando Pino Duque2016-04-29
| | | | | | | | | | | | | via the Skylark and delete the constant. Also, change the isLoadingPhase in the Skylark environment an enum Phase in order to: - Decide whether testRules are enabled or not and, - Check that the toolsRepository is set when in the LOADING phase. Finally, a few tests that were using ConfiguredRuleClassProvider directly had to be updated to set a tools repository, otherwise createGlobals() fails. -- MOS_MIGRATED_REVID=121022804
* Declare TargetAndConfiguration as final and immutable. Fix documentation.Gravatar Ulf Adams2016-04-28
| | | | | -- MOS_MIGRATED_REVID=121020960
* Acquire the server lock even if the client uses gRPC.Gravatar Lukacs Berki2016-04-28
| | | | | | | | | This is so that only one server instance is started up if two clients are started in a workspace that doesn't have a running server yet. More work towards #930. This may break Windows in case flock() doesn't work there as expected. In anticipation of this, locking is moved to blaze_util_platform.h / blaze_util.cc . -- MOS_MIGRATED_REVID=121013078
* *really* make server mode on Windows work.Gravatar Lukacs Berki2016-04-28
| | | | | | | | | This is still fallout from a bad merge I did yesterday. More progress towards #930. -- MOS_MIGRATED_REVID=121006319