Merge "Add --dir option to findleaves.py"
am: 40623e2a99

* commit '40623e2a995247907028b12538a2732ac107a057':
  Add --dir option to findleaves.py

Change-Id: I6708e21ca14a5d4e7218c792b6bc6ee0fd6818e8

Merge "Add --dir option to findleaves.py"

Add --dir option to findleaves.py

findleaves.py needs to be able to search multiple directories for
multiple filenames (Android.bp and Android.mk).  Add support for a new
--dir= option to specify one or more directories to search, and then
treat any remaining options as filenames.  If no directories are
specified, fall back to treating only the last option as a filename and
the rest as directories.

Change-Id: Ie879facd49192c6b4fafae070ad00eb21b1faa32

Merge "Improve DataSource interface."
am: ce4c9d7108

* commit 'ce4c9d71080d27ce321b1aa49e2189ee55bb25fa':
  Improve DataSource interface.

Change-Id: I8a1df06ee64785ae48414b2468e760c41d7f41f3

Merge "Store entry alignment information in APK."
am: 85f004c6c5

* commit '85f004c6c5995da4f9175eb65b90f48f601d658d':
  Store entry alignment information in APK.

Change-Id: I937cd36edbce072bcf329699f2bd3265205a8a11

Merge "Improve DataSource interface."

Merge "Store entry alignment information in APK."

Improve DataSource interface.

1. This adds way to obtain a DataSource which represents a region of
   data contained in the DataSource.
2. This fixes a design bug in "feed" method where the size parameter
   was an int instead of long.
3. This fixes a bug in ByteBufferDataSource where its mSize field was
   a long instead of an int (ByteBuffer's length cannot be more than
   2^31).

Bug: 27461702
Change-Id: Ib0812784beb581f19d2412e667b8bd018f0a3c78

Merge "Fix mm and mmm in Android.bp directories"
am: 8ed91b6624

* commit '8ed91b66240d3071a42365737f4cd5c7dbe601e1':
  Fix mm and mmm in Android.bp directories

Change-Id: Id8d80175a2ab27c6ea5a969eb37ad12726e68376

Merge "Fix mm and mmm in Android.bp directories"

Fix mm and mmm in Android.bp directories

mm and mmm were not parsing out/soong/Android.mk, so they were ignoring
all modules defined in an Android.bp file.  Always include
out/soong/Android.mk when ONE_SHOT_MAKEFILE is set, and convert the mm
and mmm functions to build MODULES-IN-$DIR instead of all_modules so
that only the subset of modules defined in out/soong/Android.mk for the
requested directory are built.

Bug: 28986194
Change-Id: I6e1ff8cacfc668a4154a0d5937450db840bb7a6b

Merge "Remove unnecessary variables"
am: 2eb45d9ab2

* commit '2eb45d9ab29b0923aeeea51ae0c58e32fea23941':
  Remove unnecessary variables

Change-Id: I645c34edb93ce262d245867c4efea2d409366a09

Merge "Remove unnecessary variables"

Remove unnecessary variables

These are all either recently unused due to the removal of CFLAGS/etc,
or have been exported by Soong and are no longer necessary.

Change-Id: I5930d43fda21acc8202b3d8ea010fbefb6ae4cf1

Merge "Remove USE_SOONG=false option"
am: 5413fb4b3e

* commit '5413fb4b3e2310be8568f5024fdc4d0900bf9f9e':
  Remove USE_SOONG=false option

Change-Id: I83c44d57917a10838dd9626d382b4e992c58f822

Merge "Remove USE_SOONG=false option"

Remove USE_SOONG=false option

Soong is always required to build now.

Change-Id: I4e310e98e912b370ebf980bf609f04212a7e9a50

Store entry alignment information in APK.

Data of uncompressed APK entries is often aligned to a multiple of 4
or 4096 in the APK to make it easier to mmap the data. Unfortunately,
the current method for achieving alignment suffers from two issues:
(1) the way it uses the Local File Header extra field is not compliant
with ZIP format (for example, this prevents older versions of Python's
zipfile from reading APKs: https://bugs.python.org/issue14315), and
(2) it does not store information about the alignment multiple in the
APK, making it harder/impossible to preserve the intended alignment
when rearranging entries in the APK.

This change solves these issues by switching to a different method for
aligning data of uncompressed APK entries. Same as before, alignment
is achieved using Local File Header entry field. What's different is
that alignment is achieved by placing a well-formed extensible data
field/block into the extra field. The new field/block contains the
alignment multiple (e.g., 4 or 4096) as well as the necessary padding
(if any). Compared to the original alignment method, the new method
uses 6 more bytes for each uncompressed entry.

Bug: 27461702
Change-Id: I8cffbecc50bf634b28fca5bc39eb23f671961cf9

Merge "Double quote some statements for zsh compatibility"
am: 576fd3ebb9

* commit '576fd3ebb92d2200ea8c02cd6f3b7ba21ced9030':
  Double quote some statements for zsh compatibility

Change-Id: I30ff4c9d527907772d11d557097fafb2cbeeb590

Merge "Double quote some statements for zsh compatibility"

Merge "Error on USE_CLANG_PLATFORM_BUILD="
am: 7b31dd1094

* commit '7b31dd10948abccc9c8b2f949edbe84de15702d5':
  Error on USE_CLANG_PLATFORM_BUILD=

Change-Id: Iea742a5cd8230587656460cc5ad6b979ba2189a5

Merge "Error on USE_CLANG_PLATFORM_BUILD="

Merge "Add LOCAL_CTS_GTEST_LIST_EXECUTABLE"
am: c11ae6c569

* commit 'c11ae6c569c26607a165e6fe491008bdd272d981':
  Add LOCAL_CTS_GTEST_LIST_EXECUTABLE

Change-Id: Id305d60c8db698fa90b09fb547f606b62bb37ad9

Merge "Add LOCAL_CTS_GTEST_LIST_EXECUTABLE"

Merge "Improve concurrency for JAR entry inspection requests."
am: ac68c28831

* commit 'ac68c28831f3947e47a924cb677b740975dbbc6f':
  Improve concurrency for JAR entry inspection requests.

Change-Id: I144f7c3f60e8a95785fcdd380e98df867a84bd70

Merge "Improve concurrency for JAR entry inspection requests."

Merge "Move to a single clang unknown flags list"
am: 06935290ac

* commit '06935290ac12d03a5a4f287f4743ada996305235':
  Move to a single clang unknown flags list

Change-Id: Ia02030229a089e531c7258780fb7837240e2b47f

Merge "Move to a single clang unknown flags list"

Move to a single clang unknown flags list

Soong is using a single list of unknown clang flags for all
architectures, instead of the per-architecture list here. Collapse this
down to a single list as well so that it can be verified against the
Soong list, and eventually replaced by the Soong version.

Change-Id: If43cd892105da5155907c05965a74b835920a369

Merge "Remove (C|CPP|CONLY|LD)FLAGS checked by Soong"
am: 15409e8f20

* commit '15409e8f20195edce9ccda48836082e4c6bad4f8':
  Remove (C|CPP|CONLY|LD)FLAGS checked by Soong

Change-Id: I412ec02d9825f6f6c857df84744b198e7ce9901a

Merge "Remove (C|CPP|CONLY|LD)FLAGS checked by Soong"

Error on USE_CLANG_PLATFORM_BUILD=

A blank USE_CLANG_PLATFORM_BUILD= on the command line for make causes
confusion between make and kati.  make ignores writes to variables set
on the command line, so envsetup.mk cannot set it to true,
USE_CLANG_PLATFORM_BUILD is still empty in soong.mk, and soong.variables
gets DeviceUsesClang: false.  kati gets USE_CLANG_PLATFORM_BUILD= in its
environment instead of on its command line, so envsetup.mk when parsed
by kati sets USE_CLANG_PLATFORM_BUILD=true.

Error out early if USE_CLANG_PLATFORM_BUILD is not true or false after
attempting to set it to true if it is empty.

Change-Id: I0e2731315711db9a6ac6ea2bf49c88bccacd35f7

Double quote some statements for zsh compatibility

Two eval commands did not enclose their arguments in double quotes, and
this broke compatibility on zsh - preventing lunch from running to
completion. This CL adds the double quotes and fixes the issue.

Change-Id: I5ed884b455e2e61182f6c29fb807ed92372af039

Merge "Windows: Stop adding to GLOBAL_LD_DIRS"
am: e23a49157f

* commit 'e23a49157f34a3c3f82115a117908d32b14a6262':
  Windows: Stop adding to GLOBAL_LD_DIRS

Change-Id: I8b834feed668c311169ec5da294414f47d4175be

Remove (C|CPP|CONLY|LD)FLAGS checked by Soong

At the beginning of every build, Soong has exported its version of these
variables, and has been ensuring that when sorted, the result is the
same. So these variables all have the same flags of the Soong ones, but
may be in different orders. We don't believe any of the remaining order
differences matter. As we remove the Make definitions, the exported
Soong variables will take over.

This only removes lines that change one of these variables:

  [2ND_][CLANG_]HOST_CROSS_GLOBAL_CFLAGS
  [2ND_][CLANG_]HOST_CROSS_GLOBAL_CONLYFLAGS
  [2ND_][CLANG_]HOST_CROSS_GLOBAL_CPPFLAGS
  [2ND_][CLANG_]HOST_CROSS_GLOBAL_LDFLAGS
  [2ND_][CLANG_]HOST_GLOBAL_CFLAGS
  [2ND_][CLANG_]HOST_GLOBAL_CONLYFLAGS
  [2ND_][CLANG_]HOST_GLOBAL_CPPFLAGS
  [2ND_][CLANG_]HOST_GLOBAL_LDFLAGS
  [2ND_][CLANG_]TARGET_GLOBAL_CFLAGS
  [2ND_][CLANG_]TARGET_GLOBAL_CONLYFLAGS
  [2ND_][CLANG_]TARGET_GLOBAL_CPPFLAGS
  [2ND_][CLANG_]TARGET_GLOBAL_LDFLAGS

Many other variables are unnecessary now too, but those will be removed
in another change. For those we can ensure the build.ninja file does not
change, but we expect it to change here due to the ordering differences.

Change-Id: I0bd0778706d02ee27b2784dd8dc6b2c71d37bd3a

Merge "Windows: Stop adding to GLOBAL_LD_DIRS"

Merge "Remove trailing newline from TARGET_C_INCLUDES"
am: 6864e3790c

* commit '6864e3790c80d3044307f0d260cf0e2b962ae82d':
  Remove trailing newline from TARGET_C_INCLUDES

Change-Id: I615050a58cca90291f2c07cae4397a711c69e585

Windows: Stop adding to GLOBAL_LD_DIRS

Soong does not have the idea of global linking directories, so move the
windows prebuilt library directory to GLOBAL_LDFLAGS instead.

Change-Id: Ie101146f4682cd85924463aa5acaa46869938e6c

Merge "Remove trailing newline from TARGET_C_INCLUDES"

Remove trailing newline from TARGET_C_INCLUDES

In preparation for comparing with the Soong variable.

Change-Id: I159507757ed7678a33a3c94e6d01d170485e251a

Merge "Add fake libart"
am: 49a4be8130

* commit '49a4be81304d2292bb6bd67ddaabe759d9e8c273':
  Add fake libart

Change-Id: Iacc59ada5b3323c81a8c70464f5634ab722c4f2b

Merge "Add fake libart"

Merge "Remove (TARGET|HOST)_NO_UNDEFINED_LDFLAGS"
am: ac7a7bb0ba

* commit 'ac7a7bb0ba7b864a07ceb159dcd6a8b177866fb3':
  Remove (TARGET|HOST)_NO_UNDEFINED_LDFLAGS

Change-Id: I7e880084b72d90dc4de16fd674692b53b1e67de2

Merge "Remove (TARGET|HOST)_NO_UNDEFINED_LDFLAGS"

Remove (TARGET|HOST)_NO_UNDEFINED_LDFLAGS

We never use it for host modules, and all the target configurations use
the same linker argument. This matches Soong.

Change-Id: Ibcba9a4ce340c7a12306d1fee620a04027c0e292

Add fake libart

Bug: 27775991

Change-Id: I54cbdf2340dcfdd9b3973d9d10c8bd94492736fb
(cherry picked from commit 6da74d8f1f0a06366fe93b885dc26f53401ec6b4)

Merge "Remove make variables exported from Soong"
am: 30b57cd7d5

* commit '30b57cd7d5459c69fbd4332b988b5e636b529f4d':
  Remove make variables exported from Soong

Change-Id: I2b7a6baf5d8a8b3b0bb52e8fc4b16a5616f8f939

Merge "Pass BUILD_HOST_static to Soong"
am: 3b0990cbdc

* commit '3b0990cbdc941b584fb3000f76607c2a8065b603':
  Pass BUILD_HOST_static to Soong

Change-Id: I8292e35b7ca3fe14133b8675dbf8d6c54736e5aa

Merge "Remove make variables exported from Soong"

Merge "Pass BUILD_HOST_static to Soong"

Pass BUILD_HOST_static to Soong

Change-Id: I0ac43a1dedf20e5251e05f12d120d0e6e2e1b1ee

Merge "Remove redundant clang cppflags"
am: 8bbed5a635

* commit '8bbed5a6350a2a7ca1eb7d0886b783fa5b681180':
  Remove redundant clang cppflags

Change-Id: Ic45e02c74ec154bc74b42dd12df2c162b4a862ce

Merge "Remove redundant clang cppflags"

Remove redundant clang cppflags

Cppflags always get added to cflags, so we don't need to duplicate clang
cppflags extras that are already in the clang cflags extras

Change-Id: Ic099f565f20fd993fc0713c033fbc5154373c98f

Merge "Make apks depend on their certificates"
am: f0a465c2cb

* commit 'f0a465c2cb5921762d7d17d65cd7352d0f64c261':
  Make apks depend on their certificates

Change-Id: I1f89af20d7d4cb337a91203631ff68eb654b0e0c

Merge "Make apks depend on their certificates"

Remove make variables exported from Soong

These have all been strictly checked by Soong, so the values currently
match. Just remove them so that the make variables exported by Soong
will take over.

This also removes some of the GCC reconfiguration, since we haven't
implemented that in Soong. If it becomes useful in the future, we'll
implement it there.

Some things like TARGET_TOOCHAIN_ROOT can nearly be removed, but are
used before the Soong-exported make_vars.mk is loaded. In that case,
added to the clang cflags, so it can be removed once the cflags are
removed.

Others, like LLVM_PREBUILTS_VERSION are loaded even earlier -- in
envsetup.sh before we've even configured Soong. So for now, keep the
Make definition, and continue ensuring it's the same as the Soong
version.

Change-Id: I554b27062ac43610828a8c06d640d919a2dc21ca

Merge "Remove TARGET_LIBGCC/LIBATOMIC/LIBGCOV"
am: a794f4db22

* commit 'a794f4db220d0f357773a037d842f40ab463485f':
  Remove TARGET_LIBGCC/LIBATOMIC/LIBGCOV

Change-Id: I7b1fcc43e7c2260632c61e29f0d6bfb41df1b812

Merge "Remove TARGET_LIBGCC/LIBATOMIC/LIBGCOV"

Remove TARGET_LIBGCC/LIBATOMIC/LIBGCOV

Instead, use the libgcc/libatomic/libgcov from the static libraries dir,
which is provided by Soong. Copy the libraries using the Soong script if
Soong is disabled - this can be removed once USE_SOONG is removed.

Change-Id: Iad2ad20ad5c3cfc48bf1e46e594a482609098d7a

Merge "No need to JAR-sign OTA update packages."
am: 094caf99aa

* commit '094caf99aab48c7ef62fd635503a0c70c18ecfde':
  No need to JAR-sign OTA update packages.

Change-Id: I0a0e73cc5f64ba20600c30b9f304b0ff06705693

Merge "No need to JAR-sign OTA update packages."

Improve concurrency for JAR entry inspection requests.

This moves the creation of potentially expensive objects from the
thread which creates JAR entry inspection requests, to the thread
which fulfills these requests, increasig concurrency opportunities.

Bug: 27461702
Change-Id: If753b2de7fb04ee5e2e4bbcb27d42269d7fa5def

Merge "Soong: Read Android.soong.mk if skipping Android.mk"
am: ea71c96894

* commit 'ea71c968943209f41191a1f164f899dd002b9119':
  Soong: Read Android.soong.mk if skipping Android.mk

Change-Id: I76aa5c350a26ea718df36f1df4ba5ade3f49a478

Merge "Soong: Read Android.soong.mk if skipping Android.mk"

Soong: Read Android.soong.mk if skipping Android.mk

If Soong is enabled, and we're skipping an Android.mk because there is
an Android.bp file, check for an Android.soong.mk file and read that
instead. This will allow us to temporarily define modules or recurse
into subdirectories that soong does not yet support.

Change-Id: Ifdb2f0204a38a5069e53527f66ffcfb8008c11a4

Merge "Default USE_SOONG to true"
am: ac33935ac1

* commit 'ac33935ac1721c640ce39b722cb7471ea9a5949d':
  Default USE_SOONG to true

Change-Id: Ie75c39b174b20af51e13bd7f6e0dfdb50d903233

Merge "Default USE_SOONG to true"

Merge "Don\'t use GTEST_OS_LINUX on Darwin"
am: 439f4dd189

* commit '439f4dd1895eef8684b3257fe5631a45d51635d6':
  Don't use GTEST_OS_LINUX on Darwin

Change-Id: Ie05e2df72a2cbfa52d6b767cd009c36a640ef23e

Merge "Pass USE_SAFESTACK to Soong."
am: 00bccf54d4

* commit '00bccf54d4b3847cf54f6507c8fa29eabf26f2e7':
  Pass USE_SAFESTACK to Soong.

Change-Id: I9258135b2debee97b21de27232a7530780a63f5d

Merge "Don't use GTEST_OS_LINUX on Darwin"

Merge "Pass USE_SAFESTACK to Soong."

Pass USE_SAFESTACK to Soong.

Change-Id: I51a94da0942aa1a1e003c134a30e8da82e1270b8

Don't use GTEST_OS_LINUX on Darwin

Change-Id: If77118591daa7b474e6989e03109ea32ddd357b2

Merge "Make Error Prone warnings in sync with Error Prone documenation."
am: 86c3b72d9c

* commit '86c3b72d9c9cb46acd7b8339dc3ce0a2f869fb2f':
  Make Error Prone warnings in sync with Error Prone documenation.

Change-Id: I78b111feb108096a9b38c798dd4ee666a6852797

Merge "Make Error Prone warnings in sync with Error Prone documenation."

Merge "Use fs_config files for mksquashfs"
am: 4730facd73

* commit '4730facd7399ead0d753d7b588001faf6469e041':
  Use fs_config files for mksquashfs

Change-Id: I3c7c7c618e18598cda56f153d9ac8a988ef85510

Merge "Use fs_config files for mksquashfs"

Make Error Prone warnings in sync with Error Prone documenation.

Use summary information from annotation within Error Prone to auto-generate
the warnings list.

Bug: 28681096
Change-Id: Ib3a5f734bd229726fcd7332eaa9e0e1bd1200c25

Make apks depend on their certificates

Bug: 27954979
Change-Id: I02db576cb80cbb9dd684d931da02e1b3b3efdf37

Merge "Follow symlinks when using find for assets"
am: 4c4c06016b

* commit '4c4c06016bfe62fde1aaf2fff33d6f6c55f78b89':
  Follow symlinks when using find for assets

Change-Id: I62fc48be86bbbac6e0d2f09e398a4922fe21668a

Merge "Follow symlinks when using find for assets"

Use fs_config files for mksquashfs

BUG: 27467028
Change-Id: I7648030ad4b31d70a5d439e9552fd2cbfe288b74
Signed-off-by: Mohamad Ayyash <mkayyash@google.com>
BUG: 28704419

Merge "Read Soong-exported configuration"
am: c7b610a67e

* commit 'c7b610a67ebff333cede41d5f336165485c9ae5d':
  Read Soong-exported configuration

Change-Id: I0890077c980e0e9d863809bc02fbb8e2f5aa8018

Merge "Read Soong-exported configuration"

Merge "Use product-specific Soong outputs"
am: f100d01dd5

* commit 'f100d01dd5dde712a19bd75f8f6895b21641bd62':
  Use product-specific Soong outputs

Change-Id: I530778730be7424be3d3fbe88fd65fc70d2a0d12

Read Soong-exported configuration

This is to ensure that Make and Soong agree on configuration variables
like TARGET_GLOBAL_CFLAGS. Only a few variables are actually checked
currently until we make more actually the same.

Bug: 23566674
Change-Id: Ibede43d933ede4d470e182f9490ea6ec4ef52fbc

Merge "Use product-specific Soong outputs"

Merge "Offer an ApkSignerEngine implementation."
am: 6975f03fd8

* commit '6975f03fd8b65c294a08ebc40be7da871e9e74d7':
  Offer an ApkSignerEngine implementation.

Change-Id: I7ab04e1e9699219bb5204c2b656b47583a399e31

Merge "Offer an ApkSignerEngine implementation."

No need to JAR-sign OTA update packages.

This removes the logic for JAR signing from -w (whole-file signing)
mode. This mode is designed specifically for OTA update packages. When
such packages are verified, their JAR signatures are ignored. Thus,
there is no need to JAR-sign in -w mode.

For context, OTA update packages are protected by a special signature
residing in the ZIP End of Central Directory record (at the very end
of the file). This is the signature verified when update packages are
being applied to Android.

Change-Id: Ia852a11ed6774ce746087cdd7f028b191ef6bc8b

Offer an ApkSignerEngine implementation.

This adds an implementation of ApkSignerEngine to the apksigner-core
library.

Bug: 27461702
Change-Id: I5f977b98555ca507a0dfcd3e92eecb9758aa8370

Follow symlinks when using find for assets

This should have been done in
https://android-review.googlesource.com/#/c/43901/

Bug: 27954979
Change-Id: I663b5e87e0d844d37a59e404219ff5e7e364df74

Merge "Correct some dependencies around zip packages"
am: b6f2c8fac8

* commit 'b6f2c8fac8246ed7e6a1d25bf9cc0e658504813a':
  Correct some dependencies around zip packages

Change-Id: I59e2597e955e69a35edbd0fa13bb1eb4d076b456

Merge "Correct some dependencies around zip packages"

Merge "Include BRILLO_VENDOR_PARTITIONS in target zip generation"
am: 648ea82b04

* commit '648ea82b048a83dfcc68aa3419523502b3a1485f':
  Include BRILLO_VENDOR_PARTITIONS in target zip generation

Change-Id: I9aa55d82d8121b66dd853cf434b2a68b4dbbab68

Merge "Add misc-macro-parentheses to default tidy checks."
am: a55b7ec4c3

* commit 'a55b7ec4c38f5de172d51e805215505196c7702d':
  Add misc-macro-parentheses to default tidy checks.

Change-Id: I2e93d50500ee3c6746aa805f0f6b268c07687c7f

Merge "Include BRILLO_VENDOR_PARTITIONS in target zip generation"

Merge "Add misc-macro-parentheses to default tidy checks."

Add misc-macro-parentheses to default tidy checks.

* Disable this check in external projects.

Bug: 28705665
Change-Id: Ia44a15765fd637dae36b5e0f2b59ee4280b139c9