Merge "Add dummy description for GET-INSTALL-PATH"

Add dummy description for GET-INSTALL-PATH

kati takes the first @echo command in a rule and turns it in to a ninja
description that gets pretty-printed.  The GET-INSTALL-PATH is only
@echo commands and is parsed by development/testrunner/runtest.py, so
it fails to parse the pretty-printed first test.

Add a dummy @echo to GET-INSTALL-PATH for kati to use as the
description, so the rest of the @echo commands are left unmodified.

Change-Id: I192fde48b26726a2f6760448f819243bd2f21312

Merge "Normalize java source file paths before running "sort -u"."

Normalize java source file paths before running "sort -u".

We rely on "sort -u" to dedupe aidl/logtags generated java files added
by both from $(all_java_sources) and from "find
$(PRIVATE_SOURCE_INTERMEDIATES_DIR) -name '*.java".
But "sort -u" doesn't work if any of the aidl/logtags source file path
has "../" in it.
This change fixes this issue by normalizing the source file paths before
passing them to "sort -u".

Change-Id: I12d2c4e0397bed9f426a1ed9b13608d72d01e0df

Merge "Bump up the BBOTA version to 4."

Merge "Make sure R.stamp is always created"

Make sure R.stamp is always created

When an app has a resource directory, but the
directory contain no resources, R.stamp is not created.

This leads to that such apps are always rebuilt,
even when there are no changes.

This patch changes so that an empty R.stamp is
created even if there are no resources.
This leads to that affected apps are not rebuilt
unless changed.

Change-Id: Ia63e5b1913a4456402700b615ba7f9d56d2da852

Bump up the BBOTA version to 4.

To accommodate new changes (such as error correction in [1]) to BBOTA
in N release. We bump up the version to keep the OTA script backward
compatible.

Needs the matching CL in commit
1fdec8685af858c5ff4f45d2e3059186ab5ed2ab.

[1]: commit 0a7b47397db3648afe6f3aeb2abb175934c2cbca

Change-Id: Ib9158b455cd5905fe2d4742ce81feb1b7583054f

Merge "Rename UPDATE_AB_PARTITIONS to AB_OTA_PARTITIONS."

Rename UPDATE_AB_PARTITIONS to AB_OTA_PARTITIONS.

In order to have all the A/B updater variables in the same AB_OTA_
namespace we rename UPDATE_AB_PARTITIONS to AB_OTA_PARTITIONS.

Bug: None
Test: `make dist` includes the partitions.txt file if properly configured.

Change-Id: Ied4faabd7e2f0ab8a5007afb44ad2f38df8d630d

Merge "Move x86_64 target to clang by default."

Move x86_64 target to clang by default.

Bug: 23163853
Change-Id: I0587ec75c0000ac99562233de33147ec8e120e65

Merge "use `find -exec ... +` everywhere"

Merge "Fix calculation of stashed blocks."

Merge "blockimgdiff.py: Fix the script error for BBOTA v1."

use `find -exec ... +` everywhere

This is more efficient as find itself can handle correctly passing
arguments down to the command, and this is in POSIX.  The current
versions of OS X also support this form.

BUG=None
TEST=mgrep still works

Change-Id: Ie570867609113868daaf068658c4f7e6f4ae42fa

Merge changes from topic 'jack-server-1.3-a3'

* changes:
  Remove LOCAL_JACK_EXTRA_ARGS
  Allow control of jack used for compilation
  Clean call-jack and its callers

Fix calculation of stashed blocks.

Change-Id: Ic3d607cb6691fc58e189a54a7244942cf3f3c387
Signed-off-by: caozhiyuan <cao.zhiyuan@zte.com.cn>

blockimgdiff.py: Fix the script error for BBOTA v1.

CL in [1] added an assertion to make sure an empty RangeSet cannot call
RangeSet.to_raw_string(). However, we may unintentionally hit that when
generating BBOTA v1 script (e.g. for "new" commands that always have
empty src_ranges).

[1] commit e8f756137c39fc3292014f2eab5f84c6e5135797

Change-Id: I1723606967f354ace9286d2b888fdbc80ba10e39

Merge "allow qemu_props to set opengles.version"

allow qemu_props to set opengles.version

Addresses the following denial:

  init: avc:  denied  { set } for property=opengles.version scontext=u:r:qemu_props:s0 tcontext=u:object_r:default_prop:s0 tclass=property_service

Bug: 25148690
Change-Id: I4b197eeabfe37e794104e4e686e9e388b5bc3e0c

Merge "Revert "Switch from clang 3.6 to new clang repository (with 3.8).""

Revert "Switch from clang 3.6 to new clang repository (with 3.8)."

This reverts commit 1332828b6ea25250c497609a1d4b2c7e28227ff9.

Bug: 25141123

Change-Id: Idd5d8757095b2b370046a84aea76bc95e16f3876

Remove LOCAL_JACK_EXTRA_ARGS

Use of DEFAULT_JACK_EXTRA_ARGS and LOCAL_JACK_FLAGS provides all needed
support.

(cherry picked from commit d2b2983cc791f90a0d052699603723335ab94f5a)

Change-Id: I1cd76836792e7332bc3750408a860400b0957485

Allow control of jack used for compilation

Version of Jack used to compile a module can now be control from
a dedicated mk located in prebuilts/sdk/tools

Bug: 24564814

(cherry picked from commit b68c9a498d1e2c631eb6e8c9092d4d5775402083)

Change-Id: I7809dc966026ff94b5cb2daa7e2c63762ca4768d

Clean call-jack and its callers

This is possible now that we have the new Jack server.
Also fix dependency from dex/jack target to jack script
instead of jack.jar.

(cherry picked from commit 5e50b342218fe2b9c1af20fd7da3c173bfac0499)

Change-Id: I8e7aa5087ba8d73cc93140ddfca8f4ab2f505175

Merge "Switch from clang 3.6 to new clang repository (with 3.8)."

Merge changes I2ba90314,I5f47307d

* changes:
  makeparallel: prepend flags to ninja command line
  Put NINJA_ARGS last

makeparallel: prepend flags to ninja command line

Ninja stops parsing top level options after -t is used to select a
tool.  Put any inserted command line options at the beginning of the
command.

Change-Id: I2ba903143366aaded63e21d749476248617c8962

Merge "Build: Fix DEX2OAT_TARGET_CPU_VARIANT setting"

Put NINJA_ARGS last

Put NINJA_ARGS at the end of the command line so it can contain tools,
which expect to be last.

Change-Id: I5f47307d01aa52eb74a333b19ffe3a7cd8950b01

Merge "Typos in envsetup.sh."

Merge "Stop encoding absolute paths in symbols"

Merge "Move x86 target to clang by default."

Move x86 target to clang by default.

Bug: 23163853
Change-Id: Ib5e9beddbeca8968d6e4365abeb44ae97cbe9981

Typos in envsetup.sh.

Change-Id: I31cc811e1352b9c924bdf164561f334b032d6f5c

Build: Fix DEX2OAT_TARGET_CPU_VARIANT setting

The secondary variant fallback was not set. The build system expects
the variants, so system properties were malformed for x86-64:

  [dalvik.vm.isa.x86.variant]: [dalvik.vm.isa.x86.features=default]

Bug: 24989387
Change-Id: Idbf54f81494f2650ad56080305807fd6b7ed4161

Merge "Invoke the aidl-cpp compiler to generate C++"

Invoke the aidl-cpp compiler to generate C++

Bug: 23599697
Test: test application compiles generated code

Change-Id: I35255517dd0b7ef7a82134be1c862093eaa3ce6b

Merge "envsetup: Show error when supplied dir isn't present with mmm"

Merge "Add support for LOCAL_SRC_FILES_EXCLUDE"

envsetup: Show error when supplied dir isn't present with mmm

When a directory isn't present with mmm, don't show "No Android.mk present", rather show that the directory isn't present

Change-Id: I7259a60012c6f30c470daa60d5a5097d01ffc4c7
Signed-off-by: Abhinav1997 <abhinav.jhanwar.august2@gmail.com>

Merge "Error correction: Append codes to verified partitions"

Merge "If a console doesn't exist, read password from stdin."

Add support for LOCAL_SRC_FILES_EXCLUDE

LOCAL_SRC_FILES_EXCLUDE will be used to filter files out of
LOCAL_SRC_FILES.  A common usage will be to use
LOCAL_SRC_FILES_EXCLUDE_<arch> to remove a source file that will be
replaced with an arch-optimized version.

Change-Id: I75cc6114c47fb784bab65cae8f618c4f395f07bb

Merge "common.py: Fix the build."

common.py: Fix the build.

Should only use source_info_dict when creating incremental OTAs.

Change-Id: I6a4e5f8fa3613a1b36563ec915e433247965a2b8

Merge "Pass source's info_dict to vendor's releasetools.py."

Merge "Use direct dependency on the JNI so files."

Use direct dependency on the JNI so files.

With the order-only dependency if the jni so files is updated,
make won't rebuild the system.img, which has dependency only on the apk
file.

Bug: 24865400
Change-Id: I9d5bee82b8a712a2c24dabaa0cd4c50174ea219f

Merge "Use libgcov OR libclang_rt.profile, not both."

Merge "Remove duplicate local_time.default"

Merge "Add the Product ID to target_files.zip"

Merge "generic_x86: delete system_server execmem"

generic_x86: delete system_server execmem

https://android-review.googlesource.com/175922 removed all uses
of system_server execmem and neverallowed it. The x86 emulator policy
inappropriately includes this rule. Delete it.

Fixes the following build breakage:

  libsepol.report_failure: neverallow on line 473 of external/sepolicy/system_server.te (or line 12452 of policy.conf) violated by allow system_server system_server:process { execmem };
  libsepol.check_assertions: 1 neverallow failures occurred
  Error while expanding policy

Change-Id: I7fbfaa0a09e8f4e8a372d2f1a64bbe58d5302204

Merge "Export include path to D-Bus headers for multilib builds"

Merge "Use @arg-lsit-file for jar in add-carried-jack-resources"

Use @arg-lsit-file for jar in add-carried-jack-resources

Don't pass the arg list as argument in any bash command;
Instead store the arg list to the arg list file and
run "jar @arg-lsit-file".
Note we need to quote the arguments in the arg list file.

This fixed "argument list too long" build error on Mac.

Bug: 24789595
Change-Id: I718f84e540d82bb83516824894a473b2961da278

Stop encoding absolute paths in symbols

GCC & Clang encode the current working directory into the symbol files.
Even if the binaries are stripped before being installed, the build-id
hash will be different. This means that two different checkouts will
produce different installed binaries, even if the sources are identical.

To prevent this, set PWD=/proc/self/cwd, so that it replaces the
absolute path with the static path /proc/self/cwd. With GCC, use
-fdebug-prefix-map to remove that static path. Our clang does not yet
support this flag, but it was merged in r250094.

This does change the behavior when using gdb -- you now need to be in
$ANDROID_BUILD_TOP before gdb will load the sources.

Bug: 24204119
Change-Id: I792d65b042b7a3fdc78d567c5a6e93cdf0141b9b

Export include path to D-Bus headers for multilib builds

Bug:None

Change-Id: I6e3898c71f54ede08a27f5be93e95a2b219e5ea6

Merge "Revert "Use @arg-lsit-file for jar in add-carried-jack-resources""

Revert "Use @arg-lsit-file for jar in add-carried-jack-resources"

This reverts commit 16f93d5661afe23229ab3b08ddd5fde9bc9c7eb8.

Change-Id: I925dae6a2d07792c61fcd3b6fd8718391ab993f8

Add the Product ID to target_files.zip

When using the A/B updater, include the product id string to
target_files.zip.
This is needed by the Omaha backend.

Bug: 24946811
Change-Id: I2ce5f6cffe440c29a1820348ee215c71cec07861

Merge "Use @arg-lsit-file for jar in add-carried-jack-resources"

Merge "Disable verity metadata for eng builds"

Merge "Let get-prebuilt-src-arch return empty if the input is empty"

Error correction: Append codes to verified partitions

Append error-correcting codes to verified partitions provided that
PRODUCT_SUPPORTS_VERITY_FEC is true.

This moves verity metadata to be after the hash tree, and requires
matching changes from
  Ide48f581bbba77aed6132f77b309db71630d81ed

Bug: 21893453
Change-Id: I6945cbab99e214566a1f9d3702333f2dbbc35816

Disable verity metadata for eng builds

We don't have dm-verity enabled on eng builds, so don't waste time
generating metadata for images.

Change-Id: Ib2c8d459bb50c30dc32a4ea1fdedc152c09a3a0f
(cherry picked from commit beae6395fc934ae24955a5e96ec04cda5535d952)

Let get-prebuilt-src-arch return empty if the input is empty

The list of archs of a pure java apk (no native libs) is empty.
In this case get-prebuilt-src-arch still return 'none'.
If it's set to LOCAL_MODULE_TARGET_ARCH, it makes the apk
be incompatible with the building target. It's not good.

Just return empty in this case.

Change-Id: Ide75cdb08c8480c8e7d3a0ac59f41831fbc18bd5

Use @arg-lsit-file for jar in add-carried-jack-resources

Don't pass the arg list as argument in any bash command;
Instead store the arg list to the arg list file and
run "jar @arg-lsit-file".

This fixed "argument list too long" build error on Mac.

Bug: 24789595
Change-Id: I3e5cd18fd988f12f0876b152b8bf07d07b56119e

Merge "Allow to build against Mac OS X SDK 10.11."

Allow to build against Mac OS X SDK 10.11.

Bug: 24739158
Change-Id: I7c57e141dd02a0d2de5a7a1ccbc72ffe422a8d6f

Pass source's info_dict to vendor's releasetools.py.

When building incremental packages, the info_dict from the source build
should be the one in use. We have done that for most of the partitions
(system and etc.), and should pass that to vendor's script as well.

Bug: 24898607
Change-Id: Ie2973d41b905637862616286663baf80df83bd88

Merge "Add the product version to target_files.zip."

Merge "Fix for Java 8 toolchain builds"

Merge "Stop passing --gen_all_phony_targets to kati"

Merge "Add all-named-(dirs|files)-under and related"

Add all-named-(dirs|files)-under and related

To consolidate the number of places that we're using 'find' in the tree,
add some more helpers:

  all-named-dirs-under
  all-subdir-named-dirs
  all-named-files-under
  all-subdir-named-files

This change also makes many of the current helpers use these helpers
instead of using their own implementation.

The 'dirs' helpers are using '-type d' so that they only output
directories. It's probably safe to use '-type f' for the files helpers,
but that increased the kati load time by ~20%.

Bug: 24204119
Change-Id: I3312e2fe8c146f10955e1d986ad15d9c8be494e1

Fix for Java 8 toolchain builds

We don't need to assume the OpenJDK (i.e. for Mac builds).

Change-Id: I1ac1856813045c7b1478dee6b08ad7b49300e4d7

Add the product version to target_files.zip.

When using the A/B updater, include the product version string to
target_files.zip.
This is needed by the Omaha backend.

BUG: 24743570
Change-Id: Idaba06624c1de452b1e1f9d3ebec851229af987a

Use libgcov OR libclang_rt.profile, not both.

Using both can cause duplicate symbol errors.

Change-Id: Id8ee13a81d32e3248ff1cdc468f49a26ecde6b57

Merge "Ensure dump-words-to-file creates the target"

Remove duplicate local_time.default

Seems it's copied from target/product/generic_no_telephony.mk
which had duplicate local_time.default before.
(fixed in commit 567ea28838678f1cea789538dbf79cd94db1f2d9)

Change-Id: I09dcdffc14de08b3d25a5fd08364d38b2712bb08

Stop passing --gen_all_phony_targets to kati

https://github.com/google/kati/commit/6ff74ce8ee4a618060a5bc7dac408be909151518

should have made this workaround unnecessary. Now, even with
old build/core, "m -j32 PRODUCT-aosp_mips64-eng dist" does not
output a ninja file with "$(cat )".

Bug: 24384320
Change-Id: I262edf00dce39f39ec4e55d5ed7b1b6f8d931f33

Ensure dump-words-to-file creates the target

Even if word list is empty.

Not creating the file was causing print of error messages that were
not preventing compilation to continue when compiling Java module with
no source.

Change-Id: I095b85f85ed6fcef5882afa39d4af51b78124c7c

am 46fbaa56: Merge "Exclude OTA/bin tools when using A/B OTA updater"

* commit '46fbaa564071e682e3cef7580723249af153ca71':
  Exclude OTA/bin tools when using A/B OTA updater

Merge "Exclude OTA/bin tools when using A/B OTA updater"

am 6b02b76a: Merge "Fix gcc color diagnostics"

* commit '6b02b76ab9ebb5c21743abc5616379d19bc1c304':
  Fix gcc color diagnostics

Exclude OTA/bin tools when using A/B OTA updater

The OTA/bin tools are not needed in the target files zip when using
the A/B OTA updater.

Bug: 24577767
Change-Id: I7938a906e4d73709aaa944e8065f922536072e27

Merge "Fix gcc color diagnostics"

Fix gcc color diagnostics

gcc color diagnostics stopped working when the flag was moved to
TARGET_GLOBAL_CFLAGS, as that gets overwritten by combo/select.mk.  Put
it back in COMMON_GLOBAL_CFLAGS, and then let the windows build filter
it out, similar to the way clang deals with unknown flags.

Change-Id: I2db221edb893d81f199494e7515d1b9282c12fae

am b9599804: Merge "Move the dist-as-droid handling from make to ninja"

* commit 'b9599804877f00fd1efddb18f779216efae14d4f':
  Move the dist-as-droid handling from make to ninja

Merge "Move the dist-as-droid handling from make to ninja"

am e89dd69c: Merge "Pack META/ab_partitions.txt when using the A/B OTA updater."

* commit 'e89dd69c7632e23bf1cf7db5be1cf1b0f6f345f8':
  Pack META/ab_partitions.txt when using the A/B OTA updater.

Merge "Pack META/ab_partitions.txt when using the A/B OTA updater."

Pack META/ab_partitions.txt when using the A/B OTA updater.

The list of A/B partitions updated is product-specific. Normally you
would update boot and system, but the product could require to update
other partitions during a system update. This patch packages the list
of A/B partitions configured to be updated when using the A/B updater.
The list can be set in the product makefile by setting the make
variable UPDATE_AB_PARTITIONS.

Bug: 24387863
Test: `make dist` creates the META/ab_partitions.txt in the target_files.zip

Change-Id: I4226ab05c7c7cd74721b06f1da64c6cf4d0e52ac

am 6ec320df: Merge "Have all targets in build.ninja"

* commit '6ec320df174b6f1dd17023e20fc642f89bb83b4c':
  Have all targets in build.ninja

Merge "Have all targets in build.ninja"

am 9df1cb50: Merge "Remove old grandfathered ALL_PREBUILTS"

* commit '9df1cb507d4d3442c75ca083670bf06938c7d66e':
  Remove old grandfathered ALL_PREBUILTS

Merge "Remove old grandfathered ALL_PREBUILTS"