aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/main
Commit message (Collapse)AuthorAge
* Skylark docs: fix ctx.file and ctx.files.Gravatar Laszlo Csomor2016-04-07
| | | | | | | | | | | | | | | They were using "<ATTR>" which was parsed as a HTML tag. ctx.file appeared as: It is a shortcut for: list(ctx.attr..files)[0] ctx.files appeared as: It is a shortcut for: [f for t in ctx.attr. for f in t.files] -- MOS_MIGRATED_REVID=119149347
* Remove Aspect field from AspectKey.Gravatar Dmitry Lomov2016-04-07
| | | | | -- MOS_MIGRATED_REVID=119141878
* Mount whole directories into the sandbox when possibleGravatar Brian Silverman2016-04-07
| | | | | | | | | | | | | | | | | | | | This halves the overhead with sandboxing enabled vs disabled for a test that basically only mounts a bunch of files out of a directory, and slows that same test with a single extra file added to the directory (but not mounted) by only ~4%. The test is <https://gist.github.com/bsilver8192/10527a862ce16bb7f79a> with 30000 inputs moved to a subdirectory and only 10 genrules. This change means symlinks will be mounted directly as their target rather than as a symlink, but this solves some weird behavior with multi-level symlinks and will only break things which don't declare all of their dependencies. -- Change-Id: I1aa39dccb2e5fca2893bdab9065ee043d34019b2 Reviewed-on: https://bazel-review.googlesource.com/#/c/3220/ MOS_MIGRATED_REVID=119138157
* Use the correct Aspect in AspectFunction for Skylark aspects.Gravatar Dmitry Lomov2016-04-07
| | | | | | | | | | | | | Previously AspectFunction was using an Aspect from the SkyKey, which might have been stale. This CL fixes the bug as uncovered in the test (see SkylarkAspectsTest), but further refactoring is needed since SkylarkAspectClass equals() is incorrect, and in fact obtaining the Skylark aspect definition should always introduce Skyframe dependency. -- MOS_MIGRATED_REVID=119137636
* cpp: fix the name of the function where |extracted_binaries| is populatedGravatar Thiago Farina2016-04-07
| | | | | | | | | | | There is no GetInstallDir() function that populates |extracted_binaries| vector. The one that exists and populates it, is called GetInstallBase() in src/main/cpp/blaze.cc. -- Change-Id: I09f951a132800e133197f5ec3206f436f90555f9 Reviewed-on: https://bazel-review.googlesource.com/c/3250/ MOS_MIGRATED_REVID=119107145
* Fix link to rust documentation.Gravatar Tom Prince2016-04-07
| | | | | | -- Reviewed-on: https://github.com/bazelbuild/bazel/pull/1119 MOS_MIGRATED_REVID=119104941
* Fix incremental dexing with bazel testGravatar Googler2016-04-07
| | | | | -- MOS_MIGRATED_REVID=119104560
* Override any existing watch stubs and watch application bundle artifacts ↵Gravatar Googler2016-04-07
| | | | | | | when copying. -- MOS_MIGRATED_REVID=119094214
* Make non-empty attribute checks happen during analysis of the target in ↵Gravatar Nathan Harmata2016-04-07
| | | | | | | question, rather than during loading of the target's package. This way a target's package won't be in error if e.g. an unrelated target has empty 'srcs'. -- MOS_MIGRATED_REVID=119079777
* Removing left over debug warning.Gravatar Googler2016-04-07
| | | | | -- MOS_MIGRATED_REVID=119055012
* Description redacted.Gravatar Googler2016-04-07
| | | | | -- MOS_MIGRATED_REVID=119053259
* Clean up FancyTerminalEventHandlerGravatar Googler2016-04-07
| | | | | -- MOS_MIGRATED_REVID=119051781
* cpp: remove unnecessary visibility attribute from blaze_util targetGravatar Thiago Farina2016-04-05
| | | | | | | | | | | | | | | | Commit 5d737d642623: ("Some minor updates to the BUILD files.") made the default visibility for the whole package public, and thus setting it again in blaze_util target is rendundat. While at it remove blaze_util_platform.h from srcs attribute of client target, as it is already in hdrs list of blaze_util target and client already depends on it, and thus it is not necessary to list it again here. -- Change-Id: I995c3cd6e2f63384acef4b24b9a082e7cb59331a Reviewed-on: https://bazel-review.googlesource.com/#/c/3150/1 MOS_MIGRATED_REVID=119041634
* Add SMT detection for mac. Fixes #963.Gravatar Yue Gan2016-04-05
| | | | | | | -- Change-Id: Ib83af0d0a04dc6b173bef1df28d17abc7a3c824d Reviewed-on: https://bazel-review.googlesource.com/#/c/3120/ MOS_MIGRATED_REVID=119027507
* strip remote workspace path from java resourcesGravatar Steven Parkes2016-04-05
| | | | | | | | Resources in `java_library`s get the external repo path added to them, e.g., `foo` becomes `external/git_remote_repo/foo`. It needs to be removed, though I'm not sure this is the right way to strip it ... -- Reviewed-on: https://github.com/bazelbuild/bazel/pull/1085 MOS_MIGRATED_REVID=119024770
* Add the preinclude files to the mandatory input list of C++ compile actions.Gravatar Lukacs Berki2016-04-05
| | | | | | | Then less special-casing is needed and these files are always read by the compiler anyway, so we don't lose any incrementality. -- MOS_MIGRATED_REVID=119023956
* Add 'apple_watch1_extension' and 'apple_watch_extension_binary' to support ↵Gravatar Googler2016-04-05
| | | | | | | | | building watch OS 1 apps. RELNOTES: Support apple_watch1_extension and apple_watch_extension_binary rules for creating watch OS 1 extensions. -- MOS_MIGRATED_REVID=119000703
* transition to two blaze flags for incremental dexingGravatar Googler2016-04-05
| | | | | -- MOS_MIGRATED_REVID=118969066
* Adds the output of -printconfiguration from proguard to the implicit outputs ofGravatar Alex Humesky2016-04-05
| | | | | | | | | | | android_binary (and java_binary). RELNOTES[NEW]: The output of -printconfiguration of proguard is available through an implicit "[target_name]_proguard.confg" output of android_binary (and java_binary). -- MOS_MIGRATED_REVID=118958246
* Introduce a seperate name to distinguish the objc provider passed from ↵Gravatar Cal Peyser2016-04-05
| | | | | | | native rules to skylark rules and that returned from skylark rules to depending native rules. This fixes a bug where a skylark rule that did not touch the objc provider would automatically pass up the objc provider it received from dependencies to dependant native rules, leading to duplicate symbol linking errors when the same objc provider was exported twice. -- MOS_MIGRATED_REVID=118947782
* Correctly generates the expected file names of the generated protos when the ↵Gravatar Googler2016-04-05
| | | | | | | names contain url, http or https. -- MOS_MIGRATED_REVID=118946808
* experimental UI: update progress bar on progress eventsGravatar Klaus Aehlig2016-04-05
| | | | | | | | | | | In the experimental UI, use progress events as an occasion to update the progress bar. However, to avoid unnecessary noise, only do so, if the progress bar potentially changes with time. -- Change-Id: I9bee093c29e0eb50e7fcce2fa6e941a02d615def Reviewed-on: https://bazel-review.googlesource.com/#/c/3206 MOS_MIGRATED_REVID=118944208
* Introduces action_config. Does this by:Gravatar Cal Peyser2016-04-05
| | | | | | | | | 1) Introducing the action_config message in the crosstool protobuf definition. The only part of that definition that are implemented in this CL is the "tool" section, other parts will be implemented in future CLs. The proto fields are here now to avoid being delayed by release cycles at each step of the implementation. 2) Refactoring the implementation of the "feature algebra" that computes the enabled features for a given toolchain. An interface called "CrosstoolActivatable" is used to represent any participant int the feature algebra, and can be either a feature or an action_config. -- MOS_MIGRATED_REVID=118943663
* Expose AppleToolchain to skylark.Gravatar Cal Peyser2016-04-05
| | | | | -- MOS_MIGRATED_REVID=118943054
* Add workspace name to module_space in python stub_template.txt fileGravatar Yun Peng2016-04-05
| | | | | | | | | Fixed #1040 RELNOTES:none -- MOS_MIGRATED_REVID=118941667
* Replace an exception message check with a class check. It's saner.Gravatar Lukacs Berki2016-04-05
| | | | | -- MOS_MIGRATED_REVID=118940625
* experimental UI: show run-time for long-running actions in the progress barGravatar Klaus Aehlig2016-04-04
| | | | | | | | | | | | Currently, the progress bar shows the 3 earliest started still running actions. While this might give some indication about which actions delay the build process, it lacks quantitative information about how long the delay is. By adding the current run time to the progress bar, at least some quantitative information is provided. -- Change-Id: Ib90eda67d87384a4372cda7779a09a44cea2b8dd Reviewed-on: https://bazel-review.googlesource.com/#/c/3205 MOS_MIGRATED_REVID=118933973
* Pass through the clock to the ExperimentalStateTrackerGravatar Klaus Aehlig2016-04-04
| | | | | | | | | | In this way, the ExperimentalStateTracker has access to all the information needed to provide information about the run-time of currently running actions. -- Change-Id: I0f4e48f39e9ebe63555e4bb1d70df2a6dbb65430 Reviewed-on: https://bazel-review.googlesource.com/#/c/3204 MOS_MIGRATED_REVID=118929758
* Dig out the builtin include file from the filegroup of libc_top instead of ↵Gravatar Lukacs Berki2016-04-04
| | | | | | | | | | | special-casing it in CppConfiguration. This seems to be the most reasonable solution. I was toying with the idea of adding a field to CROSSTOOL but that would fail if you set libc_top to something other than what was specified in that file. If I had a infinite amount of time, I'd create a custom rule called cc_libc where libc_top would point so that this file can be referenced by an attribute, but since I don't, this seems to be workable compromise. Also note that contrary to what you'd glean from the code, we don't actually have "compile" and "link" filegroups for libc. -- MOS_MIGRATED_REVID=118921101
* Point links for Skylark rule sets to the appropriate repositories.Gravatar David Chen2016-04-04
| | | | | | | Fixes #1087 -- MOS_MIGRATED_REVID=118919990
* Simplify some of the code around invoking proguard.Gravatar Alex Humesky2016-04-04
| | | | | -- MOS_MIGRATED_REVID=118819625
* Clarify that indirect deps are not necessarily transitiveGravatar Liam Miller-Cushon2016-04-04
| | | | | | | | | The compiler may implicitly load types that are direct dependencies but not referred to in source, e.g. in the case of package-info files where the package symbol is implicitly loaded to resolve annotations. -- MOS_MIGRATED_REVID=118800795
* *_test rules now return an .xctest.dSYM bundle.Gravatar Googler2016-04-01
| | | | | -- MOS_MIGRATED_REVID=118786316
* Remove BuildConfiguration#supportsIncrementalBuild , which is apparently unused.Gravatar Lukacs Berki2016-04-01
| | | | | | | BuildConfiguration.Fragment is much more complicated than it should be. Every little helps. -- MOS_MIGRATED_REVID=118783105
* Enable all Label instantiations to use a single intern pool to optimize ↵Gravatar Googler2016-04-01
| | | | | | | | | memory usage Label instances do not share a single intern pool today, as some other objects, such as SkyKey and PackageIdentifier. This change allows all Label instantiations to use a single intern pool for better memory usage. -- MOS_MIGRATED_REVID=118780952
* Make local_repository's interpretation of 'path' match the other rules'Gravatar Kristina Chodorow2016-04-01
| | | | | | | | | Noticed this while messing with remote runfiles. RELNOTES: local_repository can take a relative path. -- MOS_MIGRATED_REVID=118780775
* Make worker more defensive about NPEsGravatar Kristina Chodorow2016-04-01
| | | | | | | | | I was messing with filesystem dirtiness checkers and got an NPE, and I guess it was so early in the build that the buildRequest didn't have WorkerOptions yet, so I got a NPE from WorkerModule.buildComplete masking the actual NPE. -- MOS_MIGRATED_REVID=118780059
* Allow .o files as sources for objc builds.Gravatar Cal Peyser2016-04-01
| | | | | -- MOS_MIGRATED_REVID=118727286
* Support labels in new_{http,local}_repository's build_file attributeGravatar Damien Martin-Guillerez2016-03-31
| | | | | | | Fixes #855. -- MOS_MIGRATED_REVID=118711400
* Change --experimental_oom_more_eagerly_threshold default to 100, so that it ↵Gravatar Janak Ramakrishnan2016-03-31
| | | | | | | is a no-op by default. -- MOS_MIGRATED_REVID=118705742
* Don't try to clean up (and potentially exit with minor IOException exit ↵Gravatar Janak Ramakrishnan2016-03-31
| | | | | | | code) if we actually crashed. -- MOS_MIGRATED_REVID=118702463
* Make the Attribute.Builder.cloneBuilder() also copy aspects.Gravatar Carmi Grushko2016-03-31
| | | | | -- MOS_MIGRATED_REVID=118699141
* Remove the directories field from RepositoryFunction.Gravatar Ulf Adams2016-03-31
| | | | | | | | This is problematic for coming up with a less stateful BlazeModule API, which - in turn - is a requirement for running multiple commands in the same server. -- MOS_MIGRATED_REVID=118696892
* Update the java flags to default as follows:Gravatar Luis Fernando Pino Duque2016-03-31
| | | | | | | | | | | | | | | | --java_langtools=@bazel_tools//tools/jdk:langtools --javabuilder_top=@bazel_tools//tools/jdk:JavaBuilder_deploy.jar --singlejar_top=@bazel_tools//tools/jdk:SingleJar_deploy.jar --genclass_top=@bazel_tools//tools/jdk:GenClass_deploy.jar --ijar_top=@bazel_tools//tools/jdk:ijar --javac_bootclasspath=@bazel_tools//tools/jdk:bootclasspath --javac_extdir=@bazel_tools//tools/jdk:extdir --java_toolchain=@bazel_tools//tools/jdk:toolchain --host_javabase=@bazel_tools//tools/jdk:jdk --javabase=@bazel_tools//tools/jdk:jdk -- MOS_MIGRATED_REVID=118692509
* Drop unused skylarkinterface dependency from bazel-rules.Gravatar Julio Merino2016-03-31
| | | | | -- MOS_MIGRATED_REVID=118683261
* experimental UI: present SUBCOMMANDS in the scroll-back bufferGravatar Klaus Aehlig2016-03-31
| | | | | | | | | | | | | Treat SUBCOMMAND events the same way as ERROR/WARNING/INFO commands in the experimental UI: they provide information the user wants to look up later. As SUBCOMMAND events are only raised if that information is requested by the user, providing this information does run the risk over overwhelming the user with information. -- Change-Id: Ia7adeb2d5b92f18b8b88b026f790ac2af47b59d4 Reviewed-on: https://bazel-review.googlesource.com/#/c/3201 MOS_MIGRATED_REVID=118678085
* Set --fat_apk_cpu=armeabi-v7a by default and remove ↵Gravatar Luis Fernando Pino Duque2016-03-31
| | | | | | | Constants.ANDROID_DEFAULT_FAT_APK_CPUS. -- MOS_MIGRATED_REVID=118675987
* Reimplement the configuration sanity check as a per-fragment check.Gravatar Ulf Adams2016-03-31
| | | | | | | | | This is conceptually better (because each fragment should be self-contained), and gives us better performance, as we no longer need to load all explicit labels. -- MOS_MIGRATED_REVID=118674470
* Optionally stop adding headers from transitive modules as mandatory ↵Gravatar Googler2016-03-31
| | | | | | | | | | compilation inputs. As of r264664, Clang doesn't require files in module maps to be present anymore if the sources are instead available in the corresponding .pcm files. -- MOS_MIGRATED_REVID=118670841
* Set --package_path=%workspace% by default and remove ↵Gravatar Luis Fernando Pino Duque2016-03-31
| | | | | | | Constants.DEFAULT_PACKAGE_PATH. -- MOS_MIGRATED_REVID=118666575