Merge "Fix shared library install dependencies outside of /system"
am: 0b3f97b0d6

Change-Id: Ia2ffd13b26825c495559565f3c4542815956f660

Merge "Fix shared library install dependencies outside of /system"

Fix shared library install dependencies outside of /system

For properly picking the shared library install dependencies, HOST_OUT
and HOST_CROSS_OUT are necessary so that we don't add a dependency
between Linux and Windows. But for target modules, we must use
TARGET_OUT_ROOT instead of TARGET_OUT, or modules in /data and /vendor
won't have proper install dependencies.

Bug: 31462199
Test: m bionic-unit-tests; manual diff of generated ninja file
Change-Id: I828fa2664613296cc6c431813f5657abffdd2b8f

Merge "Implement path-based enabling of code coverage."
am: 1e1ecd5d07

Change-Id: I5f60032cfdb365a6f93bb9ca3617cbd08f6b4b21

Merge "Implement path-based enabling of code coverage."

Merge "Fix building static linux binaries"
am: 8bb71cef9f

Change-Id: Ied0e1f1d6ed4432536baeae74858f3f524a2ac65

Merge "Fix building static linux binaries"

Fix building static linux binaries

BUILD_HOST_static has been broken because I was stripping
-Wl,--start-group/-Wl,--end-group into a separate set of ldflags instead
of keeping them wrapped around some of the ldlibs. So don't pass some of
the build system flags through the ldlibs checker.

Bug: 30898128
Change-Id: I332b42679695cdc5da3cb0036290b6a3544699c0

Implement path-based enabling of code coverage.

Native coverage is enabled by setting NATIVE_COVERAGE to true
and specifying a list of paths in the COVERAGE_PATHS
environment variable. Files are exported to a zip file in the
target out directory.

Change-Id: I66a2ddd88e849bec1cc0cdae1b51fe18a007e2c3

Merge "Remove incremental workaround for aapt overlays"
am: 946e58bdff

Change-Id: I44a7d1be8ea317e3ce861b2ce0418b112f80a323

Merge "Remove incremental workaround for aapt overlays"

Remove incremental workaround for aapt overlays

This workaround to clean package intermediates is no longer necessary,
as ninja reruns commands whose command line changes.

Test: switch between product overlays that use add-resource, verify
      R.java is regenerated.
Change-Id: Icb1f76360e39882e4401180414a031956777b020

Merge "Remove speex from the pathmap"
am: fbfff3129e

Change-Id: I3e1460124b235555860296363c065e24f872991c

Merge "Remove speex from the pathmap"

Remove speex from the pathmap

libspeexresampler uses LOCAL_EXPORT_C_INCLUDE_DIRS to export this path
to all of its users already.

Test: m libaudiopreprocessing libaudioutils
Change-Id: I682303142840d6f0f02c990de8e713620840bbda

Merge "Update NDK gtest lib names."
am: 56c2a18a50

Change-Id: Ib8b6996027020fa8bc1d17f8d1286915c534d94b

Merge "Update NDK gtest lib names."

Merge "Fix libgcov.a dependency"
am: f9d5fb97cb

Change-Id: I38c4783f80976cc6cdf1dd3066d9bd9680efb4a8

Merge "Fix libgcov.a dependency"

Fix libgcov.a dependency

Now that Soong is generating libgcov.a, we need a dependency here.

Change-Id: Ie8d97339fd584eed178cc57b9333d38754ab1135

Merge "Update build rules for VDEX"
am: 3a04e1b82f

Change-Id: I6f50ba38c35277892ecc1fcc262eed9ffa533aa5

Merge "Update build rules for VDEX"

Merge "Remove global linker search paths"
am: 39c1c9ae12

Change-Id: I3eddb29f5e0c0d4fe3d5bc012c27c11509e3de8d

Merge "Remove global linker search paths"

Remove global linker search paths

Remove the global linker search path, as it can cause spurious build
failures.  If a library with the same name as a system library is in the
process of being written to the directory in the global search path, and
the linker may try to read the partially-written built one instead of the
system one.  We already use full paths to libraries for target builds,
do the same for host builds.  Also remove the normalize library
functions, they are no longer necessary.

Test: m -j checkbuild
Bug: 31393456
Change-Id: If9fc631e111f568c700fd73e103445c30d7e9d11

Update build rules for VDEX

ART now produces a new output file called VDEX, produced alongside the
existing OAT file. This patch updates the relevant build rules.

Bug: 30937355
Change-Id: Ib6be9f8f36c8f061dbbebd72cb55a5d4b6dc27eb

Update NDK gtest lib names.

The NDK gtest libs are always suffixed to simplify the make macro
that generates the test modules.

Bug: http://b/16574165
Change-Id: I1c9647e90aff2f1e9da4a88cad30199b1e423757

Merge "Prevent Kati from seeing MAKEFLAGS"
am: 6b363e24bd

Change-Id: I8bc18aa66c7b621ed90d82412b6c29e72334f3f1

Merge "Prevent Kati from seeing MAKEFLAGS"

Prevent Kati from seeing MAKEFLAGS

We never want Kati to see MAKEFLAGS, as forcefully overriding variables
is terrible. The variables in MAKEFLAGS are still available in the
environment, so if part of the build wants input from the user, it
should be explicitly checking for an environment variable or using ?=

This commit shouldn't actually change any behavior -- the version of
Kati checked into our tree doesn't actually read MAKEFLAGS (the latest
version does, which is why this change is necessary). We're also already
clearing MAKEFLAGS during makeparallel, so the USE_GOMA case already
doesn't pass through MAKEFLAGS.

Test: Manual $(warnings $(origin MYTEST) $(MYTEST)) with old and new Kati
Change-Id: I5e4ffa66b3539a539cc7a26da24645b11445c469

Merge "Blacklist the GOMACC_PATH environment variable"
am: b2498a22c2

Change-Id: I987d05196b3b15f5019f7a9497867fa9c39cec12

Merge "Blacklist the GOMACC_PATH environment variable"

Merge "Define DEFAULT_TIDY_HEADER_DIRS."
am: 9d66e2c2ef

Change-Id: Ifbcded2a3d1f0852ba348fc7954e835c05760d5a

Merge "Define DEFAULT_TIDY_HEADER_DIRS."

Blacklist the GOMACC_PATH environment variable

The clang compiler wrapper will be looking for this environment
variable, but it should only ever be set by the `gomacc` in CC_WRAPPER.
Make sure that it isn't otherwise set in the environment.

Change-Id: I4b26c7183bc47f71dffa7dfcc2e5b08e840a11b1

Merge ""SignApk" in Created-By header"
am: 6e85eeaf53

Change-Id: I5b87f9655b88bdeed5ccfccbf5b76e94243592f6

Merge ""SignApk" in Created-By header"

Merge "Build symlinks for -host and -target phony modules"
am: d81559e6ac

Change-Id: I133cae0be0a8328af1b9b186e7d47e4f46a11406

Merge "Build symlinks for -host and -target phony modules"

Build symlinks for -host and -target phony modules

Move extra module dependencies like symlinks from dependencies of
$(my_register_name) to $(my_all_targets), and make $(my_register_name)
and other convenience phony modules depend on $(my_all_targets).  Move
$(LOCAL_MODULE)-host/-target to depend on $(my_all_targets) instead of
directly depending on the built and installed modules.

Change-Id: I3b093986db7a1bff118f1f2482275f0997a98fb6

"SignApk" in Created-By header

The switch to apksig changed the Created-By header value in .SF file
from "1.0 (Android SignApk)" to "1.0 (Android apksigner)". This commit
reverts the value back to "1.0 (Android SignApk)".

Change-Id: I2fc462cade40a5b31bb6191996fd6f18fabbf08f

Merge "Add tool to copy files in to fat16 images"
am: bfea84dabc

Change-Id: I9647fc5f7b96370a74e603cdd3fb29021e7905a6

Merge "Add tool to copy files in to fat16 images"

Merge "Recognize new clang-tidy and Java warnings."
am: 0c9a2ee864

Change-Id: I81f6d78600d0a168dec75171bee5de01c5a068b2

Merge "Recognize new clang-tidy and Java warnings."

Recognize new clang-tidy and Java warnings.

Test: run warn.py with build.log
Change-Id: I11ff844bcfd7fb13aef8909307bc7eb1f5ee9b2a

Define DEFAULT_TIDY_HEADER_DIRS.

Let clang-tidy report warnings in header files in the
system and current directories but not the external
and vendor directories.

Test: build with WITH_TIDY=1
Change-Id: I1723b8eb95948961fb7164a19989e0a926e5bf8a

Merge changes I65c3fa9a,Id1ea5e8b
am: 126ea5a55c

Change-Id: If6d054b480245d50315938a65dd07aca32449c84

Merge changes I65c3fa9a,Id1ea5e8b

* changes:
  Pair mips64r6 with mips32r2 on Aosp
  Allowing pairing mips64r6 with mips32r1 and r2

Merge "Use full path in symbols zip file"
am: 72dcbeb4bc

Change-Id: I6f5db95ec6f13d8562c14a484751c9d659eb226d

Merge "Use full path in symbols zip file"

Use full path in symbols zip file

This was accidentally changed when moving to soong_zip.

Change-Id: I5f889c3c83b6f888e765d4c35486576fa11f6945

Merge "Remove HOST_PREFER_32_BIT"
am: 182ff5e2e7

Change-Id: I1fdb2a963d2e2dded21ee91998388a68b9ea03a4

Merge "Remove HOST_PREFER_32_BIT"

Merge "Create sub-directory for fuzzers under nativetest."
am: 0464662888

Change-Id: I16b6652473326811c0ceac3cb9997ff82c6ada73

Merge "Create sub-directory for fuzzers under nativetest."

Add tool to copy files in to fat16 images

fat16copy.py can be used to copy files or directories in to the root of
a fat16 image.

Usage: fat16copy <image> <file> [<file> ...]

Test: Used to compose a Raspberry Pi 3 boot image
Bug: 28912590
Change-Id: I29c9eec3786e2c5cc94f9a160360bac850809a93

Create sub-directory for fuzzers under nativetest.

This CL changes BUILD_FUZZ_TEST to generate binaries in a
sub-directory called fuzzers/ under the nativetest directory. This
allows fuzzers to be distinguished from the other native tests and
makes it easier to perform automated fuzz-testing.

Bug: 25714111
Change-Id: I452e43f15728ec9168dd44776b6ffcea6037fa7f

Remove HOST_PREFER_32_BIT

HOST_PREFER_32_BIT was used during the switch to 64-bit host tools to
keep the SDK building as 32-bit, but is never set any more.

Change-Id: I874f89c7d1e9cd8c9d4c879048d81a4362ce4dce

Merge "build_image: add padding to match partition size"
am: ef5751c1ec

Change-Id: Ia06afb7351d15b6726bd92cdbe92fca8b5116da3

Merge "build_image: add padding to match partition size"

build_image: add padding to match partition size

For some partition sizes, we currently build an image that's 1-2
blocks smaller than the actual partition, which causes fs_mgr to
not find metadata. This change adds padding to FEC metadata that
correctly positions the metadata header at the end.

Bug: 28865197
Change-Id: Ie0e044715a9c5ae8ba395e7d2ff9fbd7cffc0b4c

Merge "Walk up tree to find makefiles in mma"
am: 5ccafd0f29

Change-Id: Ia43e06a1f7169f9ed709fbbb3ef7763b9155daf2

Merge "Walk up tree to find makefiles in mma"

Walk up tree to find makefiles in mma

Call findmakefile in mma to walk up the tree to find the nearest
makefile.  Fixes mma in subdirectories of directories that have
Android.mk or Android.bp files.

Change-Id: I6fa509171699071fc34a2a75dd7e191d4ddca31a

Merge "Canonicalize mmm path"
am: a5b02ff1ef

Change-Id: Ib9d8de38a7b5d63a2be023b538797df17c5451d5

Merge "Canonicalize mmm path"

Merge "Don't double-wrap with gomacc"
am: b7a12806b1

Change-Id: If5e9b3a9dabfd3db9b16fe37e5ed0ffad956cfbb

Merge "Don't double-wrap with gomacc"

Canonicalize mmm path

Canonicalize the mmm path relative to the top level before creating the
MODULES-IN-dir target.  Fixes cd frameworks/base; mmm -j .

Test: cd frameworks/base; mmm -j
Change-Id: If7bc321e8a87b76121f876bf4bc296b0f9b80578

Merge "Create more shortcut phony targets"
am: 44303e9f26

Change-Id: I0b42e1f01e7a0eb1b731459c581155b952e67a19

Merge "Create more shortcut phony targets"

Don't double-wrap with gomacc

config.mk is parsed by make and then kati, and goma.mk appends to
CC_WRAPPER.  Only export it from make so gomacc doesn't appear twice on
the command line.

Bug: 31142427
Change-Id: I5ad97ee3a00b013faa620215c839499b48528e00

Merge "Enable goma in soong"
am: 753ed863c5

Change-Id: I10cd9f5f447124b0aff58b8aaf7a0fadafe3ac59

Merge "Enable goma in soong"

Enable goma in soong

Pass the USE_GOMA flag to soong, and export the CC_WRAPPER environment
variable after adding gomacc to it.

Bug: 31142427
Change-Id: I6de698a40817c0615b03fa17bd27075465d79ffd

Merge "Simplify Soong bootstrapping"
am: 2e5dc07c1f

Change-Id: I90835790fe8a2b6d526a8c345638c8380972beaa

Merge "Simplify Soong bootstrapping"

Simplify Soong bootstrapping

Now we only run the "main" stage from Soong once, and only when combined
with the Kati ninja file. So we can now depend on pools or rules defined
by Kati in Soong.

Change-Id: I7a8fd699ddc0d41bbcbbf3c6f2edb9e26ccfc4c9

Create more shortcut phony targets

Create shortcut phony targets for the host and device halves of
modules.

Change-Id: I1cf3a49118db830d78a1b83b2177715175949871

Merge "Check USE_GOMA consistently"
am: 049dd57872

Change-Id: Ia838646334435a7bdbd862db2c7b2a697c791265

Merge "Check USE_GOMA consistently"

Merge "Switch to prebuilt clang-3217047"
am: 3500ddfe1b

Change-Id: I9d9e8ef2be4158afa814279b073a1109a12f1970

Merge "Switch to prebuilt clang-3217047"

Check USE_GOMA consistently

Currently, the only way to properly disable goma is to unset USE_GOMA.
If you set USE_GOMA=false, we won't turn on goma, but we'll pass -j500
to ninja.

Test: USE_GOMA={x,false} m -j48 showcommands (inspect)
Change-Id: I6b327d08ebadbe614a3bf7646fd597054bbfaaa2

Merge "Identify modules ready to be converted to Soong"
am: c6bada8acd

Change-Id: Ia180357da71fbce20e0b06503fbc29481b2160af

Merge "Identify modules ready to be converted to Soong"

Switch to prebuilt clang-3217047

Bug: http://b/30421084
Change-Id: I41acd016992a0c5043b6ea018f205fad44c88b9e
Test: Tested build, boot and common usage for several devices in AOSP
and internal branch.

Identify modules ready to be converted to Soong

The output will be in the `m dist` results as soong_to_convert.txt, or
can be built using:

  $ m $OUT/soong_to_convert.txt

The output is a list of modules that are probably ready to convert to
Soong:

  # Blocked on Module (potential problems)
           283 libEGL (srcs_dotarm)
           246 libicuuc (dotdot_incs dotdot_srcs)
           221 libspeexresampler
           215 libcamera_metadata
               ...
             0 zram-perf (dotdot_incs)

The number at the beginning of the line shows how many native modules
depend on that module.

All of their dependencies have been satisfied, and any potential
problems that Make can detect are listed in parenthesis after the
module:

  dotdot_srcs: LOCAL_SRC_FILES contains paths outside $(LOCAL_PATH)
  dotdot_incs: LOCAL_C_INCLUDES contains paths include '..'
  srcs_dotarm: LOCAL_SRC_FILES contains source files like <...>.c.arm
  aidl: LOCAL_SRC_FILES contains .aidl sources
  dbus: LOCAL_SRC_FILES contains .dbus-xml sources
  objc: LOCAL_SRC_FILES contains Objective-C sources
  proto: LOCAL_SRC_FILES contains .proto sources
  rs: LOCAL_SRC_FILES contains renderscript sources
  vts: LOCAL_SRC_FILES contains .vts sources

Not all problems can be discovered, but this is a starting point.

Change-Id: I45674fe93fd267d4d1fb0bc3bc9aa025e20c5ac6

Merge "Export TARGET_PREFER_32_BIT_EXECUTABLES to soong"
am: fe3163400e

Change-Id: I27cb5c1cd17444a68b6fe4842d58e8ac3cf75979

Merge "Export TARGET_PREFER_32_BIT_EXECUTABLES to soong"

Export TARGET_PREFER_32_BIT_EXECUTABLES to soong

Change-Id: I1427d88c7539c5ed0c4c238bad4a28f73d29ef0f

Merge "Install symlinks when using shortcut phony targets"
am: 6f1cf129f2

Change-Id: I73386358b3b74ed71f0e3640eb06b74eb923fb5a

Merge "Install symlinks when using shortcut phony targets"

Merge "Suppress clang-tidy checks in frameworks/compile/mclinker."
am: a9ee94f441

Change-Id: I9b3d7949d58fe6f6c80b7717c61e8efe8b5bc706

Merge "Suppress clang-tidy checks in frameworks/compile/mclinker."

Install symlinks when using shortcut phony targets

Make symlinks an order-only dependency of the module name phony target
so that they get installed with make module or make MODULES-IN-path.

Test: mmma -j art/dalvikvm, remove symlink in $OUT and retry
Change-Id: Iae472a1c8fa1e8386cc9556cfe819bd4fca99428

Merge "Don't regenerate userdata.img when signing"
am: a7adc06e68

Change-Id: Ib6139262f366fa864e03d98796e5384fa12c7c55

Merge "Don't regenerate userdata.img when signing"

Merge "Recognize new warning messages."
am: b8f6f5dced

Change-Id: I1d3ee3bed661be31f03b76e5c2abfd3e56d18bff