Merge "Do not run zipalign when building APK (except prebuilts)."

Merge "Fix TARGET_ASAN build."

Merge "Include libGLESv3.so to PRODUCT_PACKAGES"

Fix TARGET_ASAN build.

Bug: 26044735
Change-Id: I9c6b3b16637ca2d0da4765e896a9f03c1d5dc51c

Merge "enable -fstack-protector-strong"

enable -fstack-protector-strong

This results in nearly all functions with the possibility of stack
corruption getting stack canaries, because it applies to any function
taking a reference to the frame or with a local array rather than just
the functions with arrays larger than 8 bytes. It was developed for use
in Chrome (and Chrome OS) and has also been adopted by various other
distributions (Arch, Fedora, Ubuntu, etc).

The code size increase ranges from ~1.5% to ~2.5%, compared to ~0.3% to
~0.7% with the more conservative switch. The increase in the performance
loss is usually minimal. The overall size increase once everything other
than C and C++ code is taken into account is minimal, and it greatly
improves the mitigation of stack buffer overflow vulnerabilities.

https://lwn.net/Articles/584225/

Change-Id: Ie83d381e09d94deb34c5ff142a101ba3d14cf3ba

Merge "Gmake doesn't like newline in macro expansion result."

Gmake doesn't like newline in macro expansion result.

When $(call func) in the makefile text expansion includes newline, gmake
reports error "missing separator". But it's fine if the expansion includes
only spaces. That's a bit odd.
This change fixes the "missing separator".

Change-Id: I094e5ae1725699ebfa4ed66544eef037dd68714c

Merge "Sanity check LOCAL_SRC_FILES_EXCLUDE"

Do not run zipalign when building APK (except prebuilts).

This changes the build system to no longer run zipalign except for
those prebuilts which do not need to re-signed.

Running zipalign is no longer needed because signapk takes care of
alignment. For those prebuilts which do not need to be re-signed
signapk is not invoked and thus zipalign is still needed because the
prebuilts may not have been aligned properly. This exemption will go
away once prebuilts are properly aligned.

Bug: 25794543
Change-Id: I31f10af80b9baa3c9481cb36f41e5e5046e852f9

Merge "Make signapk align .so entries to 4096 bytes."

Sanity check LOCAL_SRC_FILES_EXCLUDE

Error out if there is a file listed in LOCAL_SRC_FILES_EXCLUDE but not in
LOCAL_SRC_FILES. This should catch typos or other mistakes that would
otherwise be missed.

Change-Id: Iaddf575a6ce35238998ac47b59591a7d05fbcd0d

Merge "Remove extra definition of AIDL"

Remove extra definition of AIDL

AIDL is defined in the sometimes-prebuilt tools section, remove the
extra definition that was overriding it.

Change-Id: I677bf46475fd5fd4a9258c77d91ecee806e5f205

Merge "Use prebuilt tools in place"

Merge "Remove warnings from ijar/classfile.cc"

Make signapk align .so entries to 4096 bytes.

This makes signapk align uncompressed .so entries to memory page
boundary (4096 bytes) to enable such libraries to be loaded at runtime
through memory-mapping the APK.

With this change in place, there should no longer be a need to run
zipalign after (or before) signapk.

Bug: 25794543
Change-Id: I74775af15a683791f57fcbd3497a79951b3f63a1

Merge "releasetools: Always use recovery mount options."

Use prebuilt tools in place

Don't install the prebuilt SDK tools for TARGET_BUILD_APPS builds, just
execute them in place.  Avoids issues with duplicate rules for the
prebuilts and source modules when building with TARGET_BUILD_APPS in a
full platform tree.

Change-Id: I3cdb4f1a8208e8750310a5cc6130657613b683d1

Merge "Provide path to mainDexClasses"

Provide path to mainDexClasses

Set MAINDEXCLASSES to the path to mainDexClasses.

Change-Id: I82532e8ea8dc7349178347e080fa0c63078ca9f5

releasetools: Always use recovery mount options.

Change-Id: I6e96c417ffb5ad36405f4a6f5342f80f358f375f

Merge "releasetools: Abort the update on apply_patch failures."

releasetools: Abort the update on apply_patch failures.

We used to silently ignore the return value from apply_patch() even if
it had failed. It gives us more trouble to investigate the failure when
the affected file/partition gets touched in subsequent OTAs. This CL
adds the checking of the return value and aborts the update accordingly.

Bug: 25893277
Change-Id: Ie5e1c563576e503343e6a5b28ed4d7039f6f919c

Merge "make USER limited to 6 characters when generating BF_BUILD_NUMBER"

make USER limited to 6 characters when generating BF_BUILD_NUMBER

the value of USER is dependent from the compilation environment,so
when compiling one same device project, the BUILD_FINGERPRINT may
exceed 91 characters because ${USER} is long, but with short ${USER}
the compilation can pass.

Signed-off-by: wei qiao <qiaowei224@gmail.com>
Change-Id: Ia0f7dfa9cf7d605f1f2603f70dd0e6877482eb8a

Remove warnings from ijar/classfile.cc

Also add -Werror.

Change-Id: I2f9fda288d31bd34735856ade33916c93c45922a

Merge "Stop packaging obsolete OTA tools."

Merge "Make export_includes a restat rule"

Make export_includes a restat rule

There is currently an intentional incremental rebuild issue with
import_includes.  export_includes might get updated with an identical
version, but we don't want to force everything downstream of it to
rebuild.

When BUILDING_WITH_NINJA==true, only update export_includes if it
changes, and use .KATI_RESTAT to only run downstream rules if it
changes.  import_includes will only be updated if one of the
export_includes files is updated, so object files can have a normal
dependency on import_includes instead of an order-only dependency.
All downstream object files will now be recompiled if their imported
include paths change.

Bug: 25910568
Change-Id: I626f3b24ac02ac1309049cf1ce66cfe8ec816513

Merge changes I1de10391,I4399ca26,Iec3b2b0b

* changes:
  Use libstdc++ for ijar
  Build ijar for apps build
  Use .KATI_RESTAT to reduce unnecessary rebuilds of .jar files

Stop packaging obsolete OTA tools.

applypatch, applypatch_static and sqlite3 are no longer needed.

Bug: 24621915
Change-Id: I6de5bc45bc8434b9c387e1d3d1b6c347c23a5290

Merge "releasetools: Allow logging the diff for incrementals."

releasetools: Allow logging the diff for incrementals.

Add an option "--log_diff <filename>" to ota_from_target_files.py
script. When enabled, it logs the differences between the source
and target builds into <filename> when generating incremental OTAs.

Also move target_files_diff.py into releasetools/ so that it can be
packed into otatools.zip.

Bug: 25372309
Change-Id: Ifd4ed0f2f12ef040ee377621ec8c35a873cec34f

Use libstdc++ for ijar

For some reason ijar won't build against libc++ for TARGET_BUILD_APPS
builds, but does build with libstdc++.

(cherry picked from commit 718bab6aecfe7e23a6afa1361da64c8e0c832705)

Bug: 25904002
Change-Id: I1de103918faa5bb574af6f12cc954e7fed44dc62

Build ijar for apps build

This fixes git_master-linux-apps build.

(cherry picked from commit 90847057446374f818b49a742c567f82db02438e)

Change-Id: I4399ca2662e08c9b3cc2ad5a317e84d91b728115

Use .KATI_RESTAT to reduce unnecessary rebuilds of .jar files

The same optimization was done for binaries in

https://android-review.googlesource.com/#/c/175250/

To create a TOC file from .jar files, this change introduces
ijar, which is designed for this purpose. Only #include lines
were modified from the original version.

https://github.com/bazelbuild/bazel/tree/master/third_party/ijar

Performance:

$ m && touch
frameworks/base/core/java/com/google/android/util/Procedure.java && time
m
Before: 4m30s (1580 targets)
After: 3m57s (772 targets)

Unfortunately, the improvement is small yet, but local
experiments showed we can cut ~2 more minutes if the similar
optimization is done for .dex files.

(cherry picked from commit c1f5d9c203324b9435414e789c02394ca99c98f7)

Bug: 24597504
Change-Id: Iec3b2b0b0e674bee5d80cce3c300dc8fad6e7c13

Merge "Revert "Use .KATI_RESTAT to reduce unnecessary rebuilds of .jar files""

Merge "Revert "Build ijar for apps build""

Revert "Use .KATI_RESTAT to reduce unnecessary rebuilds of .jar files"

This reverts commit c1f5d9c203324b9435414e789c02394ca99c98f7.

Change-Id: I7e9969a060e3f1380154da65e76134168aee15d8

Revert "Build ijar for apps build"

This reverts commit 90847057446374f818b49a742c567f82db02438e.

Change-Id: I8a9958788c51dcec5028ef396cb67abd541ce31f

Merge "Revert "Use libstdc++ for ijar""

Revert "Use libstdc++ for ijar"

This reverts commit 718bab6aecfe7e23a6afa1361da64c8e0c832705.

Change-Id: I9aa222fc2936d7d89ae0700eeef505c6b317d777

Merge "Use libstdc++ for ijar"

Use libstdc++ for ijar

For some reason ijar won't build against libc++ for TARGET_BUILD_APPS
builds, but does build with libstdc++.

Change-Id: I8e900b0f764f0bb8f827705cb9173f07e4f33862

Merge "Build ijar for apps build"

Merge "Fix generated-sources-dir-for for COMMON"

Build ijar for apps build

This fixes git_master-linux-apps build.

Change-Id: I1a77b2495d98d89f570cb165d7f25f617c9af9aa

Merge "Use .KATI_RESTAT to reduce unnecessary rebuilds of .jar files"

Fix generated-sources-dir-for for COMMON

When the 4th argument specified is non-empty then it attempts to
use either HOST_OUT_GEN_COMMON or TARGET_OUT_GEN_COMMON
depending on whether the 3rd argument is non-empty or not
respectively. Unfortunately, those two variables do not exist,
the correct names for those variables is HOST_OUT_COMMON_GEN and
TARGET_OUT_COMMON_GET.

Change-Id: I66edb02824c06e0f504ebe04ff80ddbd77a16c95

Merge "target_files_diff: Allow writing outputs to file."

target_files_diff: Allow writing outputs to file.

Also clean up some pylint warnings.

Bug: 25372309
Change-Id: I226bfd72cb1310ba77352f8d367803e815e1cfa3

Merge "Fix typo."

Fix typo.

Change-Id: I299076791c620968fa7a5c420b5ca21ea4d30b51

Merge "Don't delete META-INF in jack-java-to-dex when packing resources."

Don't delete META-INF in jack-java-to-dex when packing resources.

LOCAL_DONT_DELETE_JAR_META_INF is meant for deleting resources carried
by static Java libraries, see comment in clear_vars.mk.
For a module's own resources, we should pick up whatever in
LOCAL_JAVA_RESOURCE_DIRS.
The same applies  when building .jack from a  prebult jar in
transform-jar-to-jack.

Bug: 25860887
Change-Id: I20c120e039342a1124362c5f8747eace94b03931
(cherry-pick from commit 996ae38ffd552be2a8420de6651c1982e0723e02)

Merge "Move mips64 target to clang by default."

Merge "Move mips target to clang by default."

Merge "Clean up compiler warnings in signapk."

Clean up compiler warnings in signapk.

Bug: 25794543
Change-Id: Ia7da4fbaed77af4020e5aa0b14fe5e1bd8521edb

Merge "Prevent findleaves.py from traversing copies of $(OUT_DIR)"

Move mips64 target to clang by default.

Bug: 23163853
Change-Id: Ia3d449f908b09ce95b3de246c8c699d171dcbaa3

Move mips target to clang by default.

Bug: 23163853
Change-Id: I408ffd90ff2ff075e03962fffd1681fccf0319d3

Merge "Use sensible default locations for Java"

Merge "releasetools: Fix the bug with TARGET_NO_RECOVERY."

Merge "Increase droiddoc -J-Xmx to 1600MB."

Increase droiddoc -J-Xmx to 1600MB.

As we occasionally see  java.lang.OutOfMemoryError when running
doc-comment-check-docs.

Bug: 25787786
Change-Id: I6c505c96ec36d09d28d400e03027b169870c49bb

releasetools: Fix the bug with TARGET_NO_RECOVERY.

The packaging script is broken when we set TARGET_NO_RECOVERY and
SYSTEM_ROOT_IMAGE both. With TARGET_NO_RECOVERY set, we don't have
RECOVERY/RAMDISK/etc/recovery.fstab. It thus fails the assertion in
common.LoadRecoveryFSTab() that "/" must exist in fstab when enabling
SYSTEM_ROOT_IMAGE.

Change-Id: Id509d724f18be7cf7389dd786924ead894ac0cc1

Merge "Fix race in C++ AIDL file generation"

Fix race in C++ AIDL file generation

The export_includes file for a library needs to express a dependency on
all generated exported headers.  For aidl generated headers, express a
dependency on the .cpp file instead, since the generator promises to
generate this file last.  Unfortunately, the C++ headers generated from
a .aidl file depend on the contents of the file.

Change-Id: I9402b364e4538b502c0958ac8c7bd72cb0add724

Merge "Switch to using kati env script"

Merge "Export generated AIDL headers"

Export generated AIDL headers

It is common for developers to generate/compile AIDL in a static
library, then link that library into an executable.  When doing this,
developers need to export the generated headers.

Bug: 25779424
Test: a refactoring of the aidl Android.mk shows this works

Change-Id: I4f7d471a601d2a683cb5a9da5e02e3fab576c26a

Use sensible default locations for Java

Use sensible default locations for Java based on
the EXPERIMENTAL_USE_JAVA8 flag.

Bug: 25786468
Change-Id: I7ec035b5329785efc41b3f971d5e3ae202293576

Merge "Rename AArch64 ASan target runtime library to match upstream convention."

Prevent findleaves.py from traversing copies of $(OUT_DIR)

by putting blank Android.mk and CleanSpec.mk in $(OUT_DIR).

Bug: 25732342
Change-Id: I3fcc4efddb9ee5ce309502e978f0589335af8371

Use .KATI_RESTAT to reduce unnecessary rebuilds of .jar files

The same optimization was done for binaries in

https://android-review.googlesource.com/#/c/175250/

To create a TOC file from .jar files, this change introduces
ijar, which is designed for this purpose. Only #include lines
were modified from the original version.

https://github.com/bazelbuild/bazel/tree/master/third_party/ijar

Performance:

$ m && touch
frameworks/base/core/java/com/google/android/util/Procedure.java && time
m
Before: 4m30s (1580 targets)
After: 3m57s (772 targets)

Unfortunately, the improvement is small yet, but local
experiments showed we can cut ~2 more minutes if the similar
optimization is done for .dex files.

Bug: 24597504
Change-Id: Id54953aea25e497c8ebbe783b03f13b488577c6a

Rename AArch64 ASan target runtime library to match upstream convention.

Rename from
 libclang_rt.asan-arm64-android.so
to
 libclang_rt.asan-aarch64-android.so

Change-Id: I6d73f458f437440c1de5bc72d2188d392c80cb98

Merge "Really allow overriding the LLVM prebuilts path."

Really allow overriding the LLVM prebuilts path.

Not sure why my previous testing was working without this, but kati is
a bit pickier about ?= vs :=.

Change-Id: I80243a3c93dec7fd9b8e6f92e2c6cc956706ace0

Include libGLESv3.so to PRODUCT_PACKAGES

Bug: http://b/25754041
Change-Id: If5f5030e7e1da037d873f0252dd8beb8ce6fd4db

Merge "Support LOCAL_ACP_UNAVAILABLE for prebuilts"

Switch to using kati env script

Kati generates a shell script to run ninja, but that makes it difficult
to customize the ninja command line.  Switch to sourcing the new env
script and then starting ninja manually.

Change-Id: I7139290a463018ddf5e05b4d61a0d2f18b1a2247

Merge "releasetools: Pack updatable images into IMAGES/ for A/B update."

Merge "Remove unnecessary inheritance layer between aliases."

Support LOCAL_ACP_UNAVAILABLE for prebuilts

Files built by soong will be installed using BUILD_PREBUILT, and may
include prerequesites of the acp binary.  Allow specifying
LOCAL_ACP_UNAVAILABLE:=true on prebuilts.

Change-Id: Iccfe61092f90b61f397a45f5334b0c1c0f439b32

Remove unnecessary inheritance layer between aliases.

The extra inheritance layer may cause multiple
PRODUCT_DEVICE/PRODUCT_BRAND values for a product if it doesn't
explicitly override them, eg. gms.
The full_* are deprecated product names. We keep them just for backward
compatibility, for some tools may still use the full_* product names.

Bug: 25611987
Change-Id: I7ecebd422754c3ceb16507b8d9ced65d533fe7c3

releasetools: Pack updatable images into IMAGES/ for A/B update.

Vendor-specific images (radio, bootloader and etc) used to stay in
RADIO/ in target_files zip. A/B updater expects them to be available
under IMAGES/. Make a copy for such images when calling
add_img_to_target_files.py.

We cannot move them directly from RADIO/ to IMAGES/, because we will
delete everything under IMAGES/ when signing for release.

Bug: 25674625
Change-Id: I977c879d0ff7e45c83b3f5c63844a5b135caa383

Merge "Allow overriding the LLVM prebuilts path."

Allow overriding the LLVM prebuilts path.

The automated build will need to do a two stage build, and we want to
do that without altering the source directory. Building with
`make LLVM_PREBUILTS_BASE=/path/to/prebuilts` will build using an
arbitrary toolchain instead of the installed one.

Bug: http://b/17441393
Change-Id: I4ff5c5224fbb8ebeb3a752907d6c4cd0a925aa1e

Merge "Write AB_OTA_POSTINSTALL_CONFIG to META/postinstall_config.txt."

Write AB_OTA_POSTINSTALL_CONFIG to META/postinstall_config.txt.

The config file will be passed to delta_generator to add the postinstall
field in the update payload.

Bug: 24537566
Change-Id: I06b8d9d0b2c37f9a3e7a844ac4d9e2eead8cd4d2
TEST: make dist

Merge "Use GOMA_HERMETIC=error for USE_GOMA=true build"

Use GOMA_HERMETIC=error for USE_GOMA=true build

With this flag, goma's client (gomacc) fails when local
compiler is different from goma's. Without this, goma's
backend finds a fallback compiler which looks closer to the
local compiler using version info, etc.

Bug: 25668061

Change-Id: I64ff81751f3fe960a557ddb2ca30a090c26c4327

Merge "Set WINVER as well as _WIN32_WINNT."

Merge "Sort dex and static lib resources"

Merge "Make not having Jack non-fatal."

Merge "Allow globally disabling the relocation packer."

Make not having Jack non-fatal.

Not every build needs Java.

Bug: http://b/17441393
Change-Id: Ia55742cfd43c3a4f9c7873b87816bcce99f8156f

Allow globally disabling the relocation packer.

The relocation packer is part of prebuilts/misc, and pulling that
repository into the manifest requires pulling in a lot of other things
not needed by the simpler builds (like the clang toolchain manifest).

Bug: http://b/17441393
Change-Id: If4a94804fc1a3f81215b840247f8e332d0b510c1