Merge "Don't prune $(OUT_DIR) from findleaves.py" am: 6338f23a36
am: bad4b072de

Change-Id: Ie8fe8c7e3172f25f751a21276807c3046c795d93

Merge "Don't prune $(OUT_DIR) from findleaves.py"
am: 6338f23a36

Change-Id: I9dc9f1c7380e579b54828b5b6dfbe8865ab84726

Merge "Don't prune $(OUT_DIR) from findleaves.py"

Don't prune $(OUT_DIR) from findleaves.py

Pruning $(OUT_DIR) can be overaggressive if $(OUT_DIR) matches the name
of any directory in the source tree that contains Android.mk files.
Pruning $(OUT_DIR) is no longer necessary, as one of the first things
the build system does is put an empty Android.mk file in there to avoid
recursing into it.

Bug: 31941856
Test: m -j OUT_DIR=angler
Change-Id: I7c5e95816f35bd5906845716d56cca76de7fa1ad

Merge "Make room for AVB hashtree and metadata." am: 70209ad9f0
am: b5b638f7fa

Change-Id: I2bdfd752ff2082b241ad5fbf88e89644d9cd2f3f

Merge "Make room for AVB hashtree and metadata."
am: 70209ad9f0

Change-Id: Ib9da55a7dd11d94f2044493ed932069de6c6eb74

Merge "Accept environment variables DEFAULT_*_TIDY_CHECKS" am: 4b4b2da13d
am: 8d4da5f521

Change-Id: If8c4a4d844d4ac2edbd8e173030902cab5ee0f7f

Merge "Make room for AVB hashtree and metadata."

Merge "Accept environment variables DEFAULT_*_TIDY_CHECKS"
am: 4b4b2da13d

Change-Id: I61b05ed9a74875c43ef268d67ee9e2e5d494c2ce

Merge "Accept environment variables DEFAULT_*_TIDY_CHECKS"

Make room for AVB hashtree and metadata.

While the system.img images currently built with AVB support verify
correctly, mounting the filesystem content fails. This is because
'avbtool add_hashtree_footer' used to claim some of the unused /
DONT_CARE space for stashing the verity tables and this resulting in the
mapped device ending up being smaller causing the mount failure.

Fix this by leaving enough room for AVB hashtree and metadata before
building the image. This is achieved by moving the AVB hashtree support
into build_image.py and using a just added '--calc_max_image_size'
option to 'avbtool add_hashtree_footer' to figure out how much space to
leave out.

This depends on https://android-review.googlesource.com/#/c/281821/

Bug: 31264226
Test: Mounting dm-verity set up from system.img now works.

Merged-In: I4c5de1004c1059f8c582e76b3b8517d427aa1a87

Change-Id: I945a5f1f6782791736cd319f216cfa6b448fb04d

Merge "Enable toc optimization for host builds" am: 5d8c73e373
am: 4276c12d20

Change-Id: I5bd72f5d2f8e689152d7f496b39afcacf5850371

Merge "Enable toc optimization for host builds"
am: 5d8c73e373

Change-Id: I0dae6e9a2d7e759b8c3860d3b4e7a85f04f5aecb

Merge "Enable toc optimization for host builds"

Accept environment variables DEFAULT_*_TIDY_CHECKS

* Define DEFAULT_GLOBAL_TIDY_CHECKS/DEFAULT_EXTERNAL_VENDOR_TIDY_CHECKS
  only if it is not already defined.

Bug: http://b/27779618
Test: build with WITH_TIDY=1 and define those variables.
Change-Id: I201c9da3769d6ef658a75d4a1bf42ec6816d4069

Merge "Fix for non-empty a[name] tags in openJdk based javadocs." am: a8c8dadd09
am: 8dd4374689  -s ours

Change-Id: I57a1638e7d632f6dd50b2d47c078d9371ff2968c

Merge "Fix for non-empty a[name] tags in openJdk based javadocs."
am: a8c8dadd09

Change-Id: Ide86ed97b14d3c34c6210b9ea9d24b8f3733c090

Merge "Fix for non-empty a[name] tags in openJdk based javadocs."

Fix for non-empty a[name] tags in openJdk based javadocs.

openJdk based android javadoc contains <a name=... tags
that are non-empty. Currently they are being hidden,
causing badly formatted pages for some java.* classes
(missing text and random whitespace blocks).

This change makes the existing a[name] hiding rule
apply only to empty tags.

Test: make docs
Bug: 31700998
Change-Id: Ifa75bbabe308d7aed0cce6165c1113e5a382a121
(cherry picked from commit a7edd92c457879f2719afe6b5d119886b869182f)

Enable toc optimization for host builds

The toc optimization had been disabled for host builds to ensure that
the timestamp of the final binary changed whenever its implementation
changed, in order to support rerunning host tools that were modified
during incremental builds.  However, only the final install rule must be
re-run to update the timestamp, and not the link rule.

Update the shared library install dependencies to use normal
dependencies instead of order-only dependencies for host modules, and
then enable the the toc optimization for host modules.  If the
implementation of a library changes it will be reinstalled, and
libraries or binaries that depend on it will also be reinstalled.

Bug: 26015464
Test: m -j; touch art/disassembler/disassembler_x86.cc; m -j, verify
      out/host/linux-x86/bin/oatdump is updated
Change-Id: I0a14decc1994eb55ad269d841943aef66e320c63

Merge "Propagate signals through makeparallel" am: e0f42b41f6
am: 6f4807da4b

Change-Id: Ia2a896df2782603b68c4dd5d348bcc7f5d661933

Merge "exec makeparallel" am: cf8afef2eb
am: 46c08db556

Change-Id: I8de8bb26ba46583072dc38248cb51552ae724a0f

Merge "Revert "Remove -d keepdepfile from ninja command line"" am: 6548b54dfd
am: 88f14b6bb7

Change-Id: Ic92ca83c27066a9f21c11a3b4aec99f3a178e56f

Merge "Propagate signals through makeparallel"
am: e0f42b41f6

Change-Id: Ia68f218edd5e57f82bea3269a8fb863077e28082

Merge "exec makeparallel"
am: cf8afef2eb

Change-Id: I77aaad7879d65b233e17a90f0196021105596c6e

Merge "Revert "Remove -d keepdepfile from ninja command line""
am: 6548b54dfd

Change-Id: I82aa023b881f277d419fdcdf09cd510b804217c2

Merge "Propagate signals through makeparallel"

Merge "exec makeparallel"

Merge "Revert "Remove -d keepdepfile from ninja command line""

Propagate signals through makeparallel

Set up a signal handler in makeparallel that will forward SIGHUP,
SIGINT, SIGQUIT, and SIGTERM to the child process.

Bug: 31907490
Test: m -j & killall make; pgrep -a ninja
Test: make makeparallel_test
Change-Id: I306e5335ed1b2c7056804d5da377a2f283877f30

exec makeparallel

bash doesn't kill children when it receives SIGTERM.  Since makeparallel
is the last command run by the shell, run it with exec to replace the
shell with makeparallel so make can send the signal directly to
makeparallel.

Bug: 31907490
Test: m -j & killall make; pgrep -a makeparallel
Change-Id: If8aeb51ec224234da5f05eae3382ce77f48a57e2

Revert "Remove -d keepdepfile from ninja command line"

This reverts commit 4aca5715523d1b8a85abde00d7ccc2989c90a96b.

Test: builds
Change-Id: Iddb3517a4596e84404db1738fa95eb15f435a42d

Merge "Optimize rebuilds by reducing $(shell) usage" am: f009531772
am: 466f4f2b9e

Change-Id: Ic26ab0de0b14a400c11fe96c679b2ac60d76aade

Merge "Optimize rebuilds by reducing $(shell) usage"
am: f009531772

Change-Id: I765a59826d058867abe8dd7a851ed06936fd4321

Merge "Optimize rebuilds by reducing $(shell) usage"

Optimize rebuilds by reducing $(shell) usage

$(shell) isn't particularly fast in Kati, and they have to be executed
both when reading the makefiles and determining whether the ninja file
needs to be regenerated.

Right now, the regen time is mostly hidden because we run them in
parallel. We've also configured it to ignore any commands that contain
"echo", "date", or the output directory. That happens to remove most
commands that contain side effects, so running them in parallel is fine.

But the side effects contain some important things, like the clean up
necessary when switching products. So I'm removing those filters, and
then we'll need to run the shell commands in sequence, since there will
be side-effects. That makes regen take longer though, so use pure-Make
implementations instead of $(shell) where possible.

This set of changes reduces aosp/master aosp_arm64-eng build $(shell)
usage and time by 2/3:

*kati*: func shell time: 3.135095 / 709
*kati*: func shell time: 1.067331 / 236

Bug: 30947985
Test: Manual test lines for math functions
Test: Compare build-aosp_arm64.ninja before/after
Change-Id: I4fc9d6318957992921972994f277c17918e7e1eb

Merge "Remove LOCAL_CTS_GTEST_LIST_EXECUTABLE" am: 847d965b9e
am: 8c42bd3ae4

Change-Id: I01c6bb2333ac66f266dac5755c73a1ab0f5161d1

Merge "Remove LOCAL_CTS_GTEST_LIST_EXECUTABLE"
am: 847d965b9e

Change-Id: Iab6362844a327f559a5471605ba176e075703a72

Merge "Remove LOCAL_CTS_GTEST_LIST_EXECUTABLE"

For ds builds only, emit a separate metadata file for samples only.
am: a9fcf48078

Change-Id: Ic2b786401a3431896e6441c8c2c61545a6aaea1b

For ds builds only, emit a separate metadata file for samples only.

Change-Id: I95c03a10cca586c6675f98664479ba8bf2e43576

Merge "releasetools: Clean up sign_target_files_apks.py." am: ad49deb5e6
am: 4299e48f0c

Change-Id: Ic59f6a2839502c4007f2e5dc51ac4a8eb2477b2d

Merge "releasetools: Clean up sign_target_files_apks.py."
am: ad49deb5e6

Change-Id: I9ffb1325d349e82bd08b5d0ee99db640a11a36bd

Merge "releasetools: Clean up sign_target_files_apks.py."

Merge "Use DEFAULT_EXTERNAL_VENDOR_TIDY_CHECKS for vendor/google_devices" am: 3d38042a00
am: c3118d2cd2

Change-Id: I699d22bd5cf4593e31a702aa81c98104ab84874a

Merge "Use DEFAULT_EXTERNAL_VENDOR_TIDY_CHECKS for vendor/google_devices"
am: 3d38042a00

Change-Id: I0f25786d7ef6e860093ce7c4760269f77065170a

Merge "Use DEFAULT_EXTERNAL_VENDOR_TIDY_CHECKS for vendor/google_devices"

releasetools: Clean up sign_target_files_apks.py.

Remove the obsolete workaround for API 24. Also make it pylint clean.

Test: 1. Sign a target_files.zip and get identical results.
      2. `pylint --rcfile=pylintrc sign_target_files_apks.py` gives 10.00/10.

Change-Id: I21736f959c5182486fd8ccebea9bbc594edef9fb

Use DEFAULT_EXTERNAL_VENDOR_TIDY_CHECKS for vendor/google_devices

Many source files under vendor/google_devices
are old third party code with too many clang-tidy
misc-macro-parentheses warnings.

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

Merge "releasetools: Fix the reference to OPTIONS.info_dict." am: 0a5b59e78b
am: 8b6d59e110

Change-Id: If2385afbd245654f27a96f02414856a7ce546a00

Merge "releasetools: Fix the reference to OPTIONS.info_dict."
am: 0a5b59e78b

Change-Id: Id4f6669baa861f606d960bc9bb371c6ef728aa36

Merge "releasetools: Fix the reference to OPTIONS.info_dict."

Remove LOCAL_CTS_GTEST_LIST_EXECUTABLE

The one user has now been removed, so this is no longer necessary.

Test: None
Change-Id: I8fb9e7b90f664af616368685d81f8b10d0687306

releasetools: Fix the reference to OPTIONS.info_dict.

sign_target_files_apks.py calls common.GetBootableImage() but without
calling 'OPTIONS = common.OPTIONS' first. In common.GetBootableImage(),
we should use the local info_dict parameter instead of OPTIONS.info_dict.

Test: sign_target_files_apks.py generates signed-TF.zip successfully.

Change-Id: Ia3d32b88691c26e5fb98feea709e3e3c3eb70fdb

Merge "Add LOCAL_EXPORT_C_INCLUDE_DEPS" am: 1c0b5eaf83
am: b1fdb9745f

Change-Id: I3388804b77e7125526bf6ea2814bbeb38bbc5f31

Merge "Add LOCAL_EXPORT_C_INCLUDE_DEPS"
am: 1c0b5eaf83

Change-Id: I356a040d02e2838fada7ae8472078579b516e8b0

Merge "Add LOCAL_EXPORT_C_INCLUDE_DEPS"

Add LOCAL_EXPORT_C_INCLUDE_DEPS

This shouldn't be necessary for most makefiles, since we add
$(LOCAL_GENERATED_SOURCES) to this already included, but for prebuilts
that aren't actually prebuilts (Soong modules), we need to pass
dependencies so that the generated headers are built before being used.

Bug: 31742855
Test: Use, inspect build-*.ninja
Change-Id: I8d9f675af639d3f40780d48a016fc079a82531c2

Merge "Update for new Android Verified Boot (AVB)." am: 8ee6d119d1
am: 621a3dbf7a

Change-Id: I1e7101853ba81ba21f68b9a72993dc731653692a

Merge "Update for new Android Verified Boot (AVB)."
am: 8ee6d119d1

Change-Id: I4445455b1b16c6c920c39d7a1d66520eeda385cf

Merge "Update for new Android Verified Boot (AVB)."

Update for new Android Verified Boot (AVB).

This updates the build system for the new Android Verified Boot
codebase. As this is based on Brillo Verified Boot, this change replaces
the existing BVB support.

Android Verified Boot is enabled by the BOARD_AVB_ENABLE variable

 BOARD_AVB_ENABLE := true

This will make the build system create vbmeta.img which will contain a
hash descriptor for boot.img, a hashtree descriptor for system.img, a
kernel-cmdline descriptor for setting up dm-verity for system.img and
append a hash-tree to system.img.

Additionally, the descriptors are left in boot.img and system.img so a
third party can create their own vbmeta.img file linking - using the
option --chain_partition - to these images. If this is not needed
footers can be erased using the 'avbtool erase_footer' command. It's
also harmless to just leave them in the images.

By default, the algorithm SHA256_RSA4096 is used with a test key from
the AVB source directory. This can be overriden by the
BOARD_AVB_ALGORITHM and BOARD_AVB_KEY_PATH variables to use e.g. a
4096-bit RSA key and SHA-512:

 BOARD_AVB_ALGORITHM := SHA512_RSA4096
 BOARD_AVB_KEY_PATH := /path/to/rsa_key_4096bits.pem

To prevent rollback attacks, the rollback index should be increased on a
regular basis. The rollback index can be set with the
BOARD_AVB_ROLLBACK_INDEX variable:

 BOARD_AVB_ROLLBACK_INDEX := 5

If this is not set, the rollback index defaults to 0.

The variable BOARD_AVB_MAKE_VBMETA_IMAGE_ARGS can be used to specify
additional options passed to 'avbtool make_vbmeta_image'. Typical
options to be used here include '--prop', '--prop_from_file', and
'--chain_partition'.

The variable BOARD_AVBTOOL_BOOT_ADD_HASH_FOOTER_ARGS can be used to
specify additional options passed to 'avbtool add_hash_footer' for
boot.img. Typical options to be used here include '--hash_algorithm' and
'--salt'.

The variable BOARD_AVBTOOL_SYSTEM_ADD_HASHTREE_FOOTER_ARGS can be used
to specify additional options passed to 'avbtool add_hashtree_footer'
for systems.img. Typical options to be used here include
'--hash_algorithm', '--salt', and '--block_size'.

BUG=31264226
TEST=Manually tested on edison-eng by inspecting {boot, system,
  vbmeta}.img in out/ directory as well as their counterparts in
  the IMAGES/ directory of edision-target_files-eng.zeuthen.zip

Merged-In: Ic9a61cfc65c148b12996e57f04da5432eef6b982

Change-Id: I97042655bca15e7eac899f12c5bada2f6184d307

Merge "Only run warn.py's main function if it is the main program." am: 312c310f7c
am: ce4e16d835

Change-Id: I44973ef0f9ff3f8e1bc641db592b1ac3c96d051a

Merge "Only run warn.py's main function if it is the main program."
am: 312c310f7c

Change-Id: I4b2c7dce195e62172a81154e261ced167d997ab5

Merge "Only run warn.py's main function if it is the main program."

Merge "Switch to using clang-3289846." am: 7ec2a39c3f
am: 523543b17c

Change-Id: Ifa9434cb0ca7ea0f57b436305b72f4d22af5b4b1

Merge "Switch to using clang-3289846."
am: 7ec2a39c3f

Change-Id: I67851b34b3c901d8bab93c9ba6f5456eca2aba91

Merge "Switch to using clang-3289846."

Only run warn.py's main function if it is the main program.

Test: run warn.py --byproject build.log
Change-Id: I155cce8c952d6d8a02c8e1809bf61a46c847c26e

Update Security String to 2016-11-05 on klp-dev am: 5ecba8e0cc am: b404280313 am: b6621dcce4 am: 34a991851b am: 19f744cb11 am: e38c8eff17  -s ours am: f34934a398 am: bbde7f4d61 am: b29e4b5072 am: bf456ce691
am: c66709e250

Change-Id: I7ce273af0c92ff86f021728f776d38429689a268

Update Security String to 2016-11-05 on klp-dev am: 5ecba8e0cc am: b404280313 am: b6621dcce4 am: 34a991851b am: 19f744cb11 am: e38c8eff17  -s ours am: f34934a398 am: bbde7f4d61 am: b29e4b5072
am: bf456ce691

Change-Id: I322af25b017e60b81202de28ee03ec0657e4db61

Update Security String to 2016-11-05 on klp-dev am: 5ecba8e0cc am: b404280313 am: b6621dcce4 am: 34a991851b am: 19f744cb11 am: e38c8eff17  -s ours am: f34934a398 am: bbde7f4d61
am: b29e4b5072

Change-Id: Ic2e3c2cae87d0ef219c57ec45fe953ab89a92a27

Update Security String to 2016-11-05 on klp-dev am: 5ecba8e0cc am: b404280313 am: b6621dcce4 am: 34a991851b am: 19f744cb11 am: e38c8eff17  -s ours am: f34934a398
am: bbde7f4d61

Change-Id: I2cd175f60442ff8359e17e5a5d6b12407fc0ee93

Update Security String to 2016-11-05 on klp-dev am: 5ecba8e0cc am: b404280313 am: b6621dcce4 am: 34a991851b am: 19f744cb11 am: e38c8eff17  -s ours
am: f34934a398

Change-Id: Id21149fb75f586dbd8a5bd25eadaf0c76699bd50

Update Security String to 2016-11-05 on klp-dev am: 5ecba8e0cc am: b404280313 am: b6621dcce4 am: 34a991851b am: 19f744cb11
am: e38c8eff17  -s ours

Change-Id: Ie6a3cba97e517bbe4e0e2d3154523dbe8c28ba9d

Merge "Fix pylint warning and refactor project_list." am: 6784a3fcdd
am: ac2df1c205

Change-Id: I7f2017a50ab5b5aee425e86211e0746538e94e90

Merge "Fix pylint warning and refactor project_list."
am: 6784a3fcdd

Change-Id: I6ddc536af0c9ebf735ed14032d98880ed4f2175b

Merge "Fix pylint warning and refactor project_list."

Merge "Analyze unchanged blocks in odex files." am: 6e12b72966
am: 0493339a1f

Change-Id: Icfdd5a0f02f43fdca2adf6468b106fcd403614bc

Merge "Analyze unchanged blocks in odex files."
am: 6e12b72966

Change-Id: Ic99c7293f6887ee8a4e3ad4e66ca9140e7dbeba9

Merge "Analyze unchanged blocks in odex files."

Switch to using clang-3289846.

Bug: http://b/30252254
Test: Built/boot platform using latest prebuilts.
Change-Id: I7c9357a99afec323fd766e93b5d5e941e9e06244

Merge "Fix Google pylint warnings." am: aa5f2ccde2
am: 2d06a27ae6

Change-Id: Id90eacb4b5316ac1cecd4c8578127fbf6a1c5102

Merge "Fix Google pylint warnings."
am: aa5f2ccde2

Change-Id: Ide1a5bffa1745737142ffde982268addd8c253bb

Fix pylint warning and refactor project_list.

Bug: 31377083
Test: run warn.py --byproject build.log
Change-Id: Id442cf6608bf5c14784063c05f5ff0b731f17486

Merge "Fix Google pylint warnings."

Merge "Remove WITH_SYNTAX_CHECK" am: c114bd9dbf
am: d640e062e8

Change-Id: Ia808c46ce1089210eaa5db02b446cfca0ed5cdde

Merge "Remove WITH_SYNTAX_CHECK"
am: c114bd9dbf

Change-Id: I9e6c2a9c6aa48e27efe9b32d496ca21f47d4a447

Merge "Remove WITH_SYNTAX_CHECK"

Fix Google pylint warnings.

* Rename 'severity' to 'Severity'.
* Fix bad line indentations and space.

Test: run warn.py --byproject build.log
Change-Id: I33d701cca408b5ca4715be8d44c890942b7d132a

Merge "Add buttons to group warning by project or severity." am: 9796175d0f
am: 9dee2ac43c

Change-Id: I10ff9eb5c01a9d594ccc3332b5a8daaf673705c8

Merge "Add buttons to group warning by project or severity."
am: 9796175d0f

Change-Id: I4d016ce0450870351a0454314f43d0cf75487993

Merge "Add buttons to group warning by project or severity."

Merge "Enable LOCAL_SANITIZE:=cfi and add LOCAL_SANITIZE_DIAG." am: 2358c55d70
am: 04b11c5acd

Change-Id: Iafb3730cc96056b09e26fa95e9bf9e73588d4315

Merge "Enable LOCAL_SANITIZE:=cfi and add LOCAL_SANITIZE_DIAG."
am: 2358c55d70

Change-Id: Ib36748ff1fe5f8bad82b37d27f9d1a94d5a8e0ee

Merge "Enable LOCAL_SANITIZE:=cfi and add LOCAL_SANITIZE_DIAG."

Fix for non-empty a[name] tags in openJdk based javadocs.
am: a7edd92c45

Change-Id: Iaef35c55e7af6f27a12944491026ee45b5ef8312

Merge "Remove spaces from DEFAULT_TIDY_HEADER_DIRS" am: f69e48b30b
am: 785d69a849

Change-Id: I5362a422641f40c3472d3a7506cc8a6354b8a09a

Merge "Remove spaces from DEFAULT_TIDY_HEADER_DIRS"
am: f69e48b30b

Change-Id: I63e8c0dc06332c620b16fdb27b396b39ad2d44cf

Merge "Remove spaces from DEFAULT_TIDY_HEADER_DIRS"