Merge "Don't build windows modules with clang, fix flags"

Don't build windows modules with clang, fix flags

shamu checkbuilds set USE_CLANG_PLATFORM_BUILD, which shouldn't apply to
modules built for windows. Also fix some flags that were being set
improperly.

Bug: 23566667
Change-Id: Id4c5b7cc59966328483d90f2b7be3f35e439ecee

Merge "releasetools: Fix the expected total blocks."

releasetools: Fix the expected total blocks.

With the change in [1], it zeroes out extra blocks that might be read
by dm-verity. We should update the expected number of blocks that get
touched.

[1] commit e9b619108dbe0c358ba6c14e27c92a623a7c1059.

Bug: 23786644
Change-Id: I7a7c3531e5af6bf381d3a14832e77abfa9a8efce

Merge "Add HOST_CROSS_OS"

Merge "Fix name for NDK protobuf libraries."

Add HOST_CROSS_OS

Instead of using recursive make to change the HOST_OS when building the
windows SDK under linux, add the concept of cross-building to another
host os.

Bug: 23566667
Change-Id: I6dc525b601b6251d458d197c30bf4660d7485502

Merge "releasetools: Don't write empty groups."

releasetools: Don't write empty groups.

For a DataImage, we may have no mapping of ZERO or NONZERO blocks. Don't
write those groups if they are empty.

Bug: 19024504
Change-Id: I1e921effcdbf0661d4f4b541c104dd456ac0c582

Fix name for NDK protobuf libraries.

Name them *-ndk so it's clear which library is which.

Change-Id: I53544a8a97488f902e00d1aedaa021d09ee2f649

Merge "releasetools: Always write the last block if it's padded."

Merge "Use $(BUILD_MODULES_IN_PATHS) as a part of ninja suffix"

releasetools: Always write the last block if it's padded.

In BBOTAs if the last block of a DataImage is padded, we should always
write the whole block even for incremental OTAs. Because otherwise the
last block may be skipped if unchanged, but would fail the post-install
verification if it has non-zero contents in the padding bytes.

Bug: 23828506
Change-Id: I4b0af7344d18261258cd48d18c029c089d6ff365

Merge "Fix LOCAL_MODULE_HOST_OS"

Use $(BUILD_MODULES_IN_PATHS) as a part of ninja suffix

So that switching between two different mmma targets won't
need regenerating ninja files.

Change-Id: Ifddd75ec49abd3ab2a54c6afbb902b0b32359db8

Fix LOCAL_MODULE_HOST_OS

Make sure LOCAL_IS_HOST_MODULE is set before including
module_arch_supported.mk

Bug: 23566667
Change-Id: I28ef10f093407eb3a6a83574ecc3f098eb775241

Merge "Move part of logtags stuff back to base_rules.mk."

Merge "releasetools: Fix the bug when deleting files."

Move part of logtags stuff back to base_rules.mk.

Otherwise event tags for non-Java modules
(e.g. logcat, libsurfaceflinger) are not generated.

BUG: 23803975
Change-Id: I6fd53fe2ca39102787fc513aa8c23f906b94f253

releasetools: Fix the bug when deleting files.

For file-based OTAs, we used to remove unneeded files in ascending
order, which failed to delete non-empty directories. Reverse the order
to fix the issue.

For example, now we have the following in our generated script:
  delete("/system/app/Calculator/arm/Calculator.odex",
         "/system/app/Calculator/arm/",
         "/system/app/Calculator/Calculator.apk",
         "/system/app/Calculator/");

Bug: 22960996
Change-Id: I0d36d29b7862fb53bf55bf5685a990180f9c0b3b

Merge "Add LOCAL_MODULE_HOST_OS"

Add LOCAL_MODULE_HOST_OS

Instead of wrapping a host module definition in 'ifeq($(HOST_OS),...)'
in the Android.mk files, define which hosts are supported using
LOCAL_MODULE_HOST_OS.

A blank LOCAL_MODULE_HOST_OS means that linux and darwin are supported.
A non-empty LOCAL_MODULE_HOST_OS lists the supported HOST_OSs.

Change-Id: I1e342d1908cfa00aef2c39c145b4f5f81c373bc6

Merge "move init.trace.rc to frameworks/native"

move init.trace.rc to frameworks/native

init.trace.rc will be renamed to atrace.rc and use the LOCAL_INIT_RC
mechanism to be included on /system appropriately.

Bug 23186545

Change-Id: Ibb86761d3e8d3c6d194ddb1220f93a71a8c6675b

Merge "releasetools: Fix the bug in symlink deletion."

releasetools: Fix the bug in symlink deletion.

For file-based OTAs, symlinks in the source build but not in the target
build will be deleted. However, if a symlink is replaced by a regular
file in the target build, the file will be accidentally deleted when
applying (resuming) the same package again.

Verify the checksum of a symlink that will be unpacked or renamed to.
Delete the file only if it doesn't have the target checksum.

Bug: 23646151
Change-Id: Ie322abb6022b6fa812c6b36a3127872d9614fc3b

Merge "Clean up Javac a little bit."

Merge "Add AddressSanitizer linker flags for x86."

Add AddressSanitizer linker flags for x86.

This is the 1/4 change required to get AddressSanitizer
for Android x86 to work.

Change-Id: I848805af34fdd32a313c991504d6014f9b97ab88

Clean up Javac a little bit.

- Added GLOBAL_JAVAC_DEBUG_FLAGS and merge it to
  PRIVATE_JAVACFLAGS/PRIVATE_JACK_FLAGS to get rid of
  PRIVATE_JAVAC_DEBUG_FLAGS/PRIVATE_JACK_DEBUG_FLAGS.
- With Java rules out of base_rules.mk we can get rid
  of java_alternative_checked_module now.

Change-Id: I1a14716c785e3d49330a75044107662ce96a9307

Merge "Set --ninja_suffix based on make/mm/mmm targets"

Set --ninja_suffix based on make/mm/mmm targets

This fixes two issues:

- Switching make target didn't work well when the latter
  make target is not a subset of the first make target.
- Switching between m and mm/mmm forced build.ninja to be
  regenerated.

Bug: 23685536

Change-Id: I3c3550d6cca3007d166a13367d6bc22cc3feede2

Merge "Show error message when resource limit too low for USE_GOMA=true"

Merge "Blacklist libc++ from windows host builds"

Merge "Add ahat to runtime_libart product."

Merge "Cleaned the outdated incrementaljavac."

Cleaned the outdated incrementaljavac.

Nobody is using this feature and now we switched to jack.

Change-Id: I749b486eb347cbf3ee7b107565fc800eeb238c44
(cherry-picked from commit 90598cb70c3a6e65223f02c4104e823024caf693)

Merge "Move Java stuff out of base_rules.mk"

Show error message when resource limit too low for USE_GOMA=true

This should be particularly helpful for Mac users, because the
default limits of Mac are too low.

Change-Id: I03f0f76a0707b4ce5ef9cddf5df6e3857a215996

Move Java stuff out of base_rules.mk

- Moved the target-only .aidl and .logtag rules to java.mk.
- Moved the .proto rules,  Java library dependency calculation,
  Java resource processing and Java-related
  target-specific and global variables setup to a new java_common.java,
  which is shared by both target and host Java modules.
- Minor cleanup for the moved code.

Bug: 23597357
Change-Id: Ic3eb72e26bb1a663e5739abe0a935f96b88bc352

Add ahat to runtime_libart product.

Change-Id: Iae5837f884c926a9f0e57df56bd21b6260ee8b2d

Merge "Remove AndroidConfig.h."

Merge "Use the basename of file_contexts from the info dict."

Use the basename of file_contexts from the info dict.

Changes in [1] break the backward compatibility of the OTA scripts. Fix
the issue by extracting the file basename from the info dict in
META/misc_info.txt.

[1] commit d3a803e6680e86f7b4960d51ab0b620728b793ca

Bug: 23620049
Change-Id: I40a5625bbe0beb295f34b2328906a4a4283fe59b

Remove AndroidConfig.h.

This change leaves soong references.

Change-Id: Iee32b6b97ad9711f69e999c9d01f01d71562b63f

Merge "Add support for LOCAL_SRC_FILES_(host)_(arch)"

Add support for LOCAL_SRC_FILES_(host)_(arch)

For boringssl, which has architecture specific host assembly files

Bug: 23566667

Change-Id: Ia4602d28e93a1b3d1400cf95417b5e446c5fb84b

Merge "Add _(darwin|linux|windows)"

Add _(darwin|linux|windows)

So that we can support building both linux and windows binaries at the
same time on a linux host. This replaces the ifeq($(HOST_OS),...) checks
in Android.mk files.

Bug: 23566667

Change-Id: I693e11984e36d55bb6f09fa0d49bc485463e16fb

Merge "rangelib.py: Fix the bug in monotonic check."

rangelib.py: Fix the bug in monotonic check.

RangeSet("2-10").monotonic gives a wrong result of "False". Fix the bug
and add more tests into test_rangelib.py.

Change-Id: I04780571b45ecafd34040f405756b9745a9e21a5

Merge "Add unit tests for rangelib.py."

Add unit tests for rangelib.py.

Separate the test cases in rangelib.py comments into a python unittest
file.

To invoke:
  $ python -m unittest test_rangelib

Change-Id: I59e4ce0aaccc83bcbb42fee15775683dcd559d99

Merge "Split large files for BBOTA v3."

Split large files for BBOTA v3.

For BBOTA v3, we need to stash source blocks to support resumable
feature. However, with the growth of file size and the shrink of the
cache size, source blocks that represent a file are too large to be
stashed as a whole. CL in [1] solves the issue by replacing the diff
command with a "new" command. However, it may increase the generated
package size substantially (e.g. from ~100MB to ~400MB).

With this CL, if a file spans too many blocks, we split it into smaller
pieces by generating multiple commands. For the same case above, it
reduces the package size to ~150MB.

One potential downside is that after splitting, files like .jar,
.apk and .zip can no longer use imgdiff. We may lose the potential
benefit of using imgdiff for patch size reduction.

[1] commit 82c47981bd0602a1c7b50dfabf9a6a2412993bae

Bug: 22430577
Change-Id: I5684fab0ac41c8d94fe18f52a34cef77e06f741c

Merge "Only allow toolbox exec where /system exec was already allowed."

Merge "Dist the otatools package."

Only allow toolbox exec where /system exec was already allowed.

When the toolbox domain was introduced, we allowed all domains to exec it
to avoid breakage.  However, only domains that were previously allowed the
ability to exec /system files would have been able to do this prior to the
introduction of the toolbox domain.  Remove the rule from domain.te and add
rules to all domains that are already allowed execute_no_trans to system_file.
Requires coordination with device-specific policy changes with the same Change-Id.

Change-Id: Ie46209f0412f9914857dc3d7c6b0917b7031aae5
Signed-off-by: Stephen Smalley <sds@tycho.nsa.gov>

Dist the otatools package.

Bug: 23495952
Change-Id: I1f61946580dd8ea29c3d5e1f4e859cc6d1f8554a

Merge "remove TARGET_PROVIDES_INIT_RC"

remove TARGET_PROVIDES_INIT_RC

Change-Id: I765b80d48333f54fb995e4ccfd77d38683c6cbb9

Merge "Change .dbus.xml extension to .dbus-xml"

Merge "Use goma when USE_GOMA=true is set"

Merge "Fix Linux host checkbuilds."

Fix Linux host checkbuilds.

Turns out that thanks to 9d59f417767991246848c3e101cb27d2dfea5988 we actually
build *all* the host libraries both 32- and 64-bit. Rather than fix the
libraries (or fixing things so we don't build stuff we don't need) right now,
let's just put _FILE_OFFSET_BITS and _LARGEFILE_SOURCE back.

Change-Id: I6c2d6a6919d1518f775e0d6c050e2c774994c5bd

Merge "Kill host Linux AndroidConfig.h."

Merge "Revert "Fix Windows SDK build.""

Revert "Fix Windows SDK build."

This reverts commit b8dd176441f6ba9527d24a5065d8225437fe39d5.

This wasn't the problem, and mingw doesn't even use _LARGEFILE_SOURCE. The real problem (using WIN32 instead of _WIN32 in external/llvm) is fixed now.

Change-Id: I2b708a006b530cb18d03b1425cd65edda35ee17e

Merge "Stop defining WIN32."

Stop defining WIN32.

Change-Id: Iac0b4745579e6de55bea02571617ae561308e1a9

Merge "Support LOCAL_INIT_RC."

Support LOCAL_INIT_RC.

A module can install a companion init.rc file,
by specifying:
LOCAL_INIT_RC := <init.rc-file-path-replative-to-LOCAL_PATH>
You can also use the variant with _32 or _64 suffix.

Bug: 23186545
Change-Id: I00a96509f5707ae39361a0c5555fa59d46c90322

Kill host Linux AndroidConfig.h.

We only build 64-bit now anyway.

Change-Id: I4bc9487522d215234509510bee66e4f210d86765

Merge "Also -DWIN32 to keep llvm building."

Also -DWIN32 to keep llvm building.

Change-Id: Ic8e70f35dda23226d60010a38cf5633840772310

Merge "Fix Windows SDK build."

Fix Windows SDK build.

I forgot that removing a file doesn't cause a rebuild in our lame build
system, so I hadn't actually tested my previous change.

Change-Id: Ida19d4a8fd40e37029031eac4b4ca5f0cc5c895b

Merge "Remove Windows AndroidConfig.h."

Remove Windows AndroidConfig.h.

The only things left that we're using are the Windows target version
and _FILE_OFFSET_BITS=64, and they can go in the combo .mk.

Also fix the unused Windows 64 .mk.

Change-Id: I5f2458d67c0a8201196a339573f861bbf18b7eb8

Change .dbus.xml extension to .dbus-xml

Work around gyp's inability to handle compound extensions by expecting
a similar looking simple extension for XML files definine DBus
interfaces.  We'll need to rename these sources in the places we're
using them already.

Bug: 23380180
Change-Id: Ieb2050f3ef05456cd70de65c3e128d57a6a508f8

Use goma when USE_GOMA=true is set

With USE_GOMA, the path to gomacc in $GOMA_DIR or $HOME/goma
will be appended to CC_WRAPPER and CXX_WRAPPER.

Note this works only with USE_NINJA. Unlike ninja, GNU make
cannot change the parallelism depending on targets. Specifying
-j500 to GNU make would mean you may run 500 local jobs in
parallel, but with -j32 goma will just slow down the build.

Change-Id: I0f571454fd2a5b525ee29b445f7ab8715927ca00

Merge "Add two flags for the latest ckati"

Merge "Fix sanitizer choice for global vs module."

Fix sanitizer choice for global vs module.

The sanitizer chosen by the environment (either by SANITIZE_TARGET or
SANITIZE_HOST) should be chosen over the one specified by the module.

Bug: http://b/23330588
Change-Id: I835b7d76e071fc0db2f859f98dfb9d7ff76af245

Merge "Stop defining HAVE_MS_C_RUNTIME."

Merge "Fix DBus proxies to use all definitions"

Stop defining HAVE_MS_C_RUNTIME.

Change-Id: Ie491547552ba64d60d669b74115da96a0fcced18

Fix DBus proxies to use all definitions

Bug: 22608897
Change-Id: Id17cf4b896c1c62be874599c7f346f3045e36819

Merge "Generate DBus proxies and adaptors separately"

Generate DBus proxies and adaptors separately

Enable daemons exposing an interface over DBus to easily
build client libraries.  Now daemons can write rules like:

include $(CLEAR_VARS)
LOCAL_MODULE := libdbus-binding-example-client
LOCAL_DBUS_PROXY_PREFIX := dbus-example-example
LOCAL_SRC_FILES := \
    dbus_bindings/org.chromium.Example.Manager.dbus.xml \
    dbus_bindings/dbus-service-config.json
include $(BUILD_SHARED_LIBRARY)

to expose a client library.

While here, add support for generating independent adaptor header
files on a per interface basis.

Bug: 22608897
Change-Id: I011f9afc234811c31e445898321c2731c482fa77

Merge "build: Don't call make_recovery_patch if there's no recovery."

Blacklist libc++ from windows host builds

It's not ready, and instead of each Android.mk doing this, do it in the
core build.

Change-Id: Ie80b17ffcde64e72c98d310ca986d4cf775c099e

build: Don't call make_recovery_patch if there's no recovery.

Change-Id: Iea1bd529eca9cd607da3b420bc4287f95c18219d

Merge "Remove special x86 flag that disables tail call optimization."

Merge "Rename libnanopb-c-2.8.0 to libprotobuf-c-nano."

Merge "Consolidate the nanopb-c and regular protobuf build rules."

Rename libnanopb-c-2.8.0 to libprotobuf-c-nano.

Bug: 20093047
Change-Id: I787925fde97d1a3965cb8a9bf012f3c5a45a6aed

Consolidate the nanopb-c and regular protobuf build rules.

Bug: 20093047
Change-Id: I38ebd748aacce6d215403da481ad04833810a995

Merge "Revise stash for BBOTAs when needed."

Merge "Remove legacy windows platform build support"