Merge \"Do not force turn off of Jack warning\"
am: 0a26fc0ace

Change-Id: Iddfabd648876281596bc8c571968d18589f1f581

Merge "Do not force turn off of Jack warning"

Merge \"Redirect subprocess stderr to stdout in verbose mode.\"
am: 1318ec93e7

Change-Id: If411ad0cd8aba960e4ab8600a5dd1532e5bd9dca

Merge "Redirect subprocess stderr to stdout in verbose mode."

Redirect subprocess stderr to stdout in verbose mode.

This patch uses subprocess.communicate instead of subprocess.wait to
prevent deadlock if any of the child processes outputs too much data,
and redirects the subprocess output to stdout when running in verbose
mode.

With this patch `ota_from_target_files -v` prints the delta_generator
output in stdout, and no output if '-v' is not passed.

Bug: None
TEST=ota_from_target_files -v ...

Change-Id: Id66e4f3360a6f91d61a3ce96d53afbccdaa19da5

Merge \"goldfish_setup: grant /system/bin/sh exec access\"
am: 7a866d783c

Change-Id: I340d78d01171de2eda4f25a2b4dbe0d637f35d59

Merge "goldfish_setup: grant /system/bin/sh exec access"

goldfish_setup: grant /system/bin/sh exec access

The goldfish_setup shell script needs the ability to execute
the shell script interpreter. Allow it.

Addresses the following denial:

avc: denied { getattr } for pid=1220 comm="init.goldfish.s"
path="/system/bin/sh" dev="vda" ino=442 scontext=u:r:goldfish_setup:s0
tcontext=u:object_r:shell_exec:s0 tclass=file permissive=0

(cherrypicked from commit 501c88c0290eb2308c110398f106411260863bb8)

Bug: 28941573
Change-Id: I22d26e90f107c8d801229354a5e0513c37e6c31d

Merge \"Add variable PRODUCT_SHIPPING_API_LEVEL and make files with the new read only product property ro.product.first_api_level.\"
am: e5c1af2dd0

Change-Id: I0dd1cd7ebe6b7ede5f019566d844b12fe40fb40c

Merge "Add variable PRODUCT_SHIPPING_API_LEVEL and make files with the new read only product property ro.product.first_api_level."

Merge \"Rewrite LDLIBS and SHARED_LIBRARIES\"
am: a8d13592ac

Change-Id: Ic1cdfd510ff26c279a2628b58c3a1db7ef172b5c

Merge "Rewrite LDLIBS and SHARED_LIBRARIES"

Merge \"APK Signature Scheme v2 APK verifier.\"
am: 4f8bde47ae

Change-Id: I0a9476be7b5da7166a1d20c2b64b82fcdfae4973

Merge "APK Signature Scheme v2 APK verifier."

Merge \"Fix DSA APK signatures for API Level 8 and lower.\"
am: ebe68ec237

Change-Id: I1bb11e4d47a572f5e0b9e9205910ca4dc675f671

APK Signature Scheme v2 APK verifier.

This adds the ApkVerifier class which verifies APKs using APK
Signature Scheme v2 only. In a follow-up commit this class will be
extended to verify APKs using JAR signature scheme when necessary.

The APK verifier is designed to not just verify an APK, but also
report errors, warnings, and information about signers in a structured
way, to enable tools to surface this information to users in various
ways.

Bug: 27461702
Change-Id: I10c6ba436021d86b6dbf6d3cf44494652adacb66

Merge "Fix DSA APK signatures for API Level 8 and lower."

Do not force turn off of Jack warning

When ANDROID_JACK_EXTRA_ARGS is defined there is no reason to still turn
off those warnings.

Change-Id: Ic7ac02c72ed3b7b0eaca9394c9fc7d92d5dfa871

Rewrite LDLIBS and SHARED_LIBRARIES

LOCAL_LDLIBS was the only correct way to use NDK libraries, but few used
it correctly. It also often got confused with LOCAL_LDFLAGS, so move the
flags to the correct variable.

For binaries that weren't using the NDK (empty LOCAL_SDK_VERSION), it
was never valid to use LOCAL_LDLIBS, as dependencies would not be
properly set up, and could lead to random build failures. So convert any
-l linker flags to using LOCAL_SHARED_LIBRARIES automatically.

For binaries built using the NDK (LOCAL_SDK_VERSION set), they were
required to use LOCAL_LDLIBS for prebuilt NDK libraries, otherwise they
would get headers and dependencies to the platform versions. Any
non-prebuilt LOCAL_LDLIBS would miss dependencies. So move the NDK
prebuilt libraries to LDLIBS from SHARED_LIBRARIES, and move everything
else to SHARED_LIBRARIES.

So now, for device modules, LOCAL_SHARED_LIBRARIES should always be
used, and we'll do the right thing. LOCAL_LDLIBS should only be used for
host libraries from the system.

Change-Id: Ide34c7afdcfb6507a378d45a42471729e489a9e0

Merge "Add NATIVE_TESTS class, move host native tests"
am: e72fc63901

* commit 'e72fc639014809b4d4239d7f7754c6950f1ad546':
  Add NATIVE_TESTS class, move host native tests

Change-Id: Ifb0401242afe1e6e16666001faf3d55471200c3f

Merge "Add NATIVE_TESTS class, move host native tests"

Merge "Switch some build tools to Soong"
am: c52d09e694

* commit 'c52d09e6944e531e02242cf971020cb2c0cf9844':
  Switch some build tools to Soong

Change-Id: I4ba945cad1eda7c696cd770243e226a09ff735ac

Merge "Switch some build tools to Soong"

Fix DSA APK signatures for API Level 8 and lower.

This modifies JAR signing code to produce DSA signatures which are
accepted by all Android platforms rather than only API Level 9 and
higher.

The issue is that by default Bouncy Castle uses OID 1.2.840.10040.4.3
(dsaWithSha1) in PKCS #7 CMS SignerInfo whereas Android accepts that
only since API Level 9. However, OID 1.2.840.10040.4.1 (dsa) is
accepted by all Android platforms.

Bug: 27461702
Change-Id: I24256a255bcdc2108bdb447557af7568a2c096e3

Switch some build tools to Soong

In preparation for including them in prebuilts/build-tools.

acp: We use a prebuilt in prebuilts/sdk, but it's not part of the sdk.

ijar: We use the host libstdc++ to workaround the lack of libc++ on
some unbundled branches.

ziptime: We disable this on unbundled builds, due to the lack of libc++.

Change-Id: Ib9766b1dbddd151c38ff27c529865200ab37fce1

Merge "Remove dead rgb2565"
am: a571e83095

* commit 'a571e83095a72d595e5e4e365e0ae52aefd64356':
  Remove dead rgb2565

Change-Id: I6dae3bbd804591a0e91f3467f021c5ee2dde4bf7

Merge "Remove dead rgb2565"

Merge "Drop CLANG and CLANG_CXX in the environment"
am: 27a55cd3c7

* commit '27a55cd3c703c190f933630e503d963ef7478183':
  Drop CLANG and CLANG_CXX in the environment

Change-Id: I3850b1783fb47cda05f1a7799ff8ceddb589e033

Merge "Drop CLANG and CLANG_CXX in the environment"

Merge "Improve efficiency of using the DataSource abstraction."
am: 7353eabd14

* commit '7353eabd14cbfae2c123a58cae4ce1e770ee351d':
  Improve efficiency of using the DataSource abstraction.

Change-Id: Ia1ab6c7d0b94c59ab6d7b41828e8e5d676c212e5

Merge "Improve efficiency of using the DataSource abstraction."

Remove dead rgb2565

Change-Id: I2397dc97768f24a7c416bff96dd983fc796cbb53

Drop CLANG and CLANG_CXX in the environment

We'll explicitly pass these variables to the static analyzer builds in
binary.mk

Bug: 29071982
Change-Id: Ifc7633d36f5785fe16b63f54b885396c2cab542a

Merge "Switch signapk to apksigner-core."
am: 1a39e8eb34

* commit '1a39e8eb3408f45d5f75b261fc073843ab83645b':
  Switch signapk to apksigner-core.

Change-Id: I39af8650adfc9ee12f5d6256e4a2083d4f87acfb

Merge "Switch signapk to apksigner-core."

Improve efficiency of using the DataSource abstraction.

This adds getByteBuffer and copyTo methods to the DataSource
abstraction. These methods enable the client to avoid unnecessary
copying of the data source's data.

Bug: 27461702
Change-Id: If4e9f902ea75c1ca5c7be0e20c0e7218faf9c504

Merge "Error on external includes"
am: 4b33f51043

* commit '4b33f510436f88774f5217b472835973fd8d5b2b':
  Error on external includes

Change-Id: Ib2fb1470cee531cb043e7fe4a8e310fb026dcb3c

Merge "Error on external includes"

Add variable PRODUCT_SHIPPING_API_LEVEL and make files with the
new read only product property ro.product.first_api_level.

PRODUCT_SHIPPING_API_LEVEL declares which API level a device first
shipped with. We use this variable to set a read-only property that
can be used in CTS/GTS tests.

Change-Id: If54202075d3b92ed380655cde8b3fc50c5ba73b3

Merge "Default to enable tidy warnings in header files."
am: 6a7cf55a7d

* commit '6a7cf55a7dfe844b4df173520890b4daee60ea63':
  Default to enable tidy warnings in header files.

Change-Id: Ib4ca529f2f81d05db8392a4f501bea3400f17d57

Merge "Default to enable tidy warnings in header files."

Merge "Add support for TARGET_RECOVERY_BRICK."
am: daf55776be

* commit 'daf55776beefeb3f59d2f7ca61796ad07fa1d738':
  Add support for TARGET_RECOVERY_BRICK.

Change-Id: I70a4682caca250d557975dcf19bfbf63bc567a24

Merge "Add support for TARGET_RECOVERY_BRICK."

Default to enable tidy warnings in header files.

Change-Id: I3831f03facbbd6d17117d82f9ccac49bc2645237

Add support for TARGET_RECOVERY_BRICK.

TARGET_RECOVERY_BRICK specifies the file that lists all the partitions
to be bricked under recovery. The file, if present, will be copied to
'$(TARGET_RECOVERY_ROOT_OUT)/etc/recovery.brick'.

Bug: 27253717
Change-Id: Id84195b5ee1031a83bf7a077be070a3728416580
(cherry picked from commit 3051f4f11aef75af1d7cf623506ad6b4b648f3bd)

Merge "Add LOCAL_LOGTAGS_FILES that prebuilts can use"
am: 476b5b1616

* commit '476b5b161613cc352961414b549a0553361d533b':
  Add LOCAL_LOGTAGS_FILES that prebuilts can use

Change-Id: Ib0b5bb5aac8b2d1c9896d58bb2c7928a63de4241

Merge "disable unpriv perf by default in user{,debug} builds"
am: 4bc8280a98

* commit '4bc8280a9886d93870fab02eea73986d8ca08965':
  disable unpriv perf by default in user{,debug} builds

Change-Id: Icb4e1ad8ec0bd491a99b541ac98dc95bf509aa63

Merge "Add LOCAL_LOGTAGS_FILES that prebuilts can use"

Merge "disable unpriv perf by default in user{,debug} builds"

Add LOCAL_LOGTAGS_FILES that prebuilts can use

Soong modules can define logtags files that need to be combined into
/system/etc/event-log-tags, so add a new LOCAL_LOGTAGS_FILES variable
that can be used to specify *.logtags file outside of LOCAL_SRC_FILES.

Bug: 28989759
Change-Id: I53c5d396dfb7c6006806758f351eb5cdde90fe74

Merge "Update default NINJA_STATUS to show finished edges"
am: 27032630c0

* commit '27032630c0683c30fa78a7bf9432c5b651707f7b':
  Update default NINJA_STATUS to show finished edges

Change-Id: I8e9989722cc5855640a0fcd68597926ce7f0dfda

Merge "Update Android.mk finder to work with Android.bp files"
am: 520442f053

* commit '520442f0537c72276147e86116a1170ea1ddc9d7':
  Update Android.mk finder to work with Android.bp files

Change-Id: Ic737bc2d3e693dc7398fff665f7557c3e2e2a72b

Merge "Update default NINJA_STATUS to show finished edges"

Merge "Update Android.mk finder to work with Android.bp files"

Merge "Add Soong tools to PATH"
am: b09e644e1d

* commit 'b09e644e1db3439d85ce6dd9e371ac6514a4343c':
  Add Soong tools to PATH

Change-Id: I7ba514ee46489e44ff64b5e1be8459567102a33c

Merge "Add Soong tools to PATH"

Add Soong tools to PATH

Change-Id: I8f8649106df4ba7cefbfa37a724ebe0635d90b76

Merge "Allow arm prebuilts for x86_arm builds"
am: fcaae6bee8

* commit 'fcaae6bee83c62703339f0e54751b244ef7fd7b7':
  Allow arm prebuilts for x86_arm builds

Change-Id: Ia8a3cd4736afbf7920f4b5383cb24c99a0f9df41

Merge "Allow arm prebuilts for x86_arm builds"

disable unpriv perf by default in user{,debug} builds

Bug: 29054680

Change-Id: If04bc63fc366006cde842a50b7c764f89d7449a6

Allow arm prebuilts for x86_arm builds

But only for static and shared libraries -- we disable building other
types of modules for arm, so don't support the prebuilts for them
either.

This is needed so that we can build these libraries under Soong, and
expose them to Make using BUILD_PREBUILT.

Bug: 27526885
Change-Id: I4f6ec922aed37591b6257b7022b81fb65d05b0fe

Update default NINJA_STATUS to show finished edges

Update default NINJA_STATUS that includes percentage to match
https://github.com/ninja-build/ninja/commit/6c2c2eecbfd2e4ddeeacb0129944618a6ce3dc4f

Change-Id: I5694308abe26b71999d944c40962d59d50a73077

Update Android.mk finder to work with Android.bp files

The Android.mk finder should stop traversing when it finds an Android.bp
file, and any Android.soong.mk file at the same path should be included.

Update filter-soong-makefiles to accept a mixed list of Android.bp files
and Android.mk files, convert Android.bp files to Android.soong.mk files
if they exist, and ignore Android.mk files that have an Android.bp file.

Update first-makefiles-under to use the new --dir syntax for
findleaves.py and search for both Android.bp and Android.mk files.

Update all-makefiles-under and all-named-subdir-makefiles to search for
both Android.bp and Android.mk files.

Change-Id: Iddc51bf84593d5de3add02f2b2c065da08962af4

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

Switch signapk to apksigner-core.

This switches signapk's APK signing from its own signing logic to that
offered by apksigner-core library. OTA update package signing logic
remains inside signapk codebase.

Bug: 27461702
Change-Id: Ibf8435c555fe3f2b621d5189e7ae44f79082c810

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

Error on external includes

To make the build hermetic, all include files should be located within
the source tree (or generated as output).

There was an Android.mk that was adding /usr/include to its include
path.

Change-Id: I4c40b908314239c8239c7835d592e409248a3150

Add NATIVE_TESTS class, move host native tests

Host native tests have been getting installed into
out/host/linux-x86/bin/..., but this pollutes the bin directory with a
lot of poorly named tests. Also, to support 32-bit and 64-bit tests, we
need to have different names with different suffixes. This causes
problems when tests expect to be named something specific (like gtest).
It's also convenient to store test data next to the test itself.

So with this change, native tests will be installed in
out/host/linux-x86/nativetest[64]/$(LOCAL_MODULE)/$(LOCAL_MODULE_STEM)
just like target tests get installed into /data/nativetest[64].

Implement this using a new NATIVE_TESTS class, which is like
EXECUTABLES, but sets up the install path differently, and configures
the rpath to load shared libraries with the proper relative path.
LOCAL_MODULE_RELATIVE_PATH can be used to control the directory name, it
will default to $(LOCAL_MODULE). This way multiple related tests can be
grouped together.

Target native tests also use NATIVE_TESTS now, but nothing should change
other than LOCAL_MODULE_RELATIVE_PATH can be used.

Change-Id: I535e42b1a6b21c5b8d6a580aa2f944d2be35e27d

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