am 6b37c7b3: Merge "Allow USE_GOMA=true without explicit USE_NINJA=true"

* commit '6b37c7b32af667f6a42b397a27add34d3c59e5fa':
  Allow USE_GOMA=true without explicit USE_NINJA=true

Merge "Allow USE_GOMA=true without explicit USE_NINJA=true"

am c180dd0b: Merge "Revert "disable Krait for Mac builds to avoid assembler issues""

* commit 'c180dd0bfc55ae222f143dc86a333caf000cbf03':
  Revert "disable Krait for Mac builds to avoid assembler issues"

Merge "Revert "disable Krait for Mac builds to avoid assembler issues""

Revert "disable Krait for Mac builds to avoid assembler issues"

This doesn't seem to be a problem any more, and mac builds
that differ from linux builds is causing other problems.

This reverts commit 7c40fb95dc90ed092f435c769b4bbb494d453674.

Bug: 24339250
Change-Id: I8a0da11df42d58ba9c3ded4571f1db870f400a21

am fdec88c9: Merge "Generate breakpad symbols"

* commit 'fdec88c9726687f98448ccff7670bc843f77bf66':
  Generate breakpad symbols

am 92facb1f: Merge "Substitue "/" with "-" in the MODULES-IN phony goal names."

* commit '92facb1fed26492746aca267df51b6082286521c':
  Substitue "/" with "-" in the MODULES-IN phony goal names.

Merge "Generate breakpad symbols"

Merge "Substitue "/" with "-" in the MODULES-IN phony goal names."

Generate breakpad symbols

When BREAKPAD_GENERATE_SYMBOLS is set to true, generate breakpad
symbols for binaries.

Bug: 23900817
Change-Id: I7b992d819350f4f80df5868d16f13016502dfca0

Substitue "/" with "-" in the MODULES-IN phony goal names.

This avoids Ninja's path canonicalization on those goal names.
Without this, Ninja complains about duplicate rules for
MODULES-IN/a/b/.. and MODULES-IN/a.

Bug: 24309760
Change-Id: Id3aa9da5d3887a2cd1426cbb5fdf35659c65805e

am 43bbbf8a: Merge "Remove more of the pathmap."

* commit '43bbbf8a15cfc03ce99d0b6a49992b25cf51fc39':
  Remove more of the pathmap.

Merge "Remove more of the pathmap."

am 880f05f4: Merge "Use exported includes for libc++."

* commit '880f05f4f28df672eb4ff4ce6b7735480deb13ae':
  Use exported includes for libc++.

am aeaa0f5c: Merge "Turn shared library multilib warnings into errors"

* commit 'aeaa0f5c580356c0d74dd11e00fb29ae67d97f2b':
  Turn shared library multilib warnings into errors

Merge "Use exported includes for libc++."

Use exported includes for libc++.

Change-Id: Ia5d9c857a43975448a8c2b489cc29be0fa67ddf6

Merge "Turn shared library multilib warnings into errors"

Turn shared library multilib warnings into errors

When caught later, these just get turned into warnings with
unpredictable results, or non-obvious errors.

Change-Id: Ie7ba88e97dc5bedd6cf850748d79693711ac41da

am 3f107055: Merge "The pattern %tests is too broad."

* commit '3f10705579deae60d9b903a4587a66a26112ee4a':
  The pattern %tests is too broad.

Merge "The pattern %tests is too broad."

The pattern %tests is too broad.

We had fixed the filtering of %tests in art/Android.mk.
Actually the %tests leads to false build<suffix>.ninja when you run:
$ mmma cts/tests/tests/nativeopengl/libnativeopengltests
because the new mmma calls make with goals like
MODULES-IN/cts/tests/tests/nativeopengl/libnativeopengltests .

Bug: 24309760
Change-Id: I35af2e8d046ae40f3b90ec0a92473fcab8ff5db5

Remove more of the pathmap.

I've fixed libpagemap to export its headers, the only user of the
libnativehelper entry was in this file anyway, and no one was using
the libhost entry.

Change-Id: Id0d1773e55c0114ccc56cefd783689193a6204f9

Allow USE_GOMA=true without explicit USE_NINJA=true

Now ninja build is disabled only with USE_NINJA=false. So goma.mk
should fail with an error only when USE_NINJA=false is explicitly
specified.

Change-Id: I2fb913433d5c1e93ed2a9bd93bb5000cabd835a4

resolved conflicts for 964f06b1 to stage-aosp-master

Change-Id: I0cebad2e440ec7d605afafac893352f06ca2f419

am bd1b9e33: Merge "Sort all files found via "

* commit 'bd1b9e3332de46e1f08b4b5ca8f53d0a096b4793':
  Sort all files found via $(shell find)

am b6a3ede5: Merge "Remove trailing / for HOST_TOOLCHAIN_FOR_CLANG"

* commit 'b6a3ede5506bec89618519bf4a8adb635e570e89':
  Remove trailing / for HOST_TOOLCHAIN_FOR_CLANG

am 58350dee: Merge "Don\'t add CFLAGS to CPPFLAGS"

* commit '58350dee77bbfadaea7bce9ee83555dba4963d92':
  Don't add CFLAGS to CPPFLAGS

Merge "Re-implement mma/mmma using make goals."

Merge "Sort all files found via $(shell find)"

Merge "Remove trailing / for HOST_TOOLCHAIN_FOR_CLANG"

Merge "Don't add CFLAGS to CPPFLAGS"

am 782b98ea: Merge changes I3ca9ed36,Ibd7eec5c

* commit '782b98eaa1c02d935b338f7317fef139067291bb':
  Revert "Default to hiding libgcc symbols in each object."
  Revert "Don't apply --exclude-libs for the host."

Remove trailing / for HOST_TOOLCHAIN_FOR_CLANG

The darwin version doesn't have a trailing slash. The common case is to
append a path to it:

  $(HOST_TOOLCHAIN_FOR_CLANG)/lib/...

Which means we end up with two slashes.

Change-Id: I74e88924ecfd092c5f7871e188ede0aab29cf65b

Merge changes I3ca9ed36,Ibd7eec5c

* changes:
  Revert "Default to hiding libgcc symbols in each object."
  Revert "Don't apply --exclude-libs for the host."

Re-implement mma/mmma using make goals.

Previously we use command line variable BUILD_MODULES_IN_PATHS which
causes kati to regenerate build.ninja if any variable value changes.

TODO: Remove the old rules of BUILD_MODULES_IN_PATHS in
build/core/main.mk and ninja.mk once users have re-sourced envsetup.sh.
Probably in a few weeks.

Caveat: With this change, if a user sources the new envsetup.sh, mma/mmma
won't work in old branches from the same shell.

Bug: 24309760
Change-Id: I212e82aeebaaa2bd4114e2066f1995da8886b5e2

Don't add CFLAGS to CPPFLAGS

Except for kati, we never use COMMON_GLOBAL_CPPFLAGS without also using
COMMON_GLOBAL_CFLAGS, which leads to duplicate command line entries.

Change-Id: I25d625924e9d157175a9e88bc3f15b7214cfb4ed

Revert "Default to hiding libgcc symbols in each object."

This reverts commit 021b7651a5fcb806d1b8a767c7f4a0a81bc8e4ff.

Bug: http://b/24166967
Change-Id: I3ca9ed36fa3b55c7eb129c209903676879720acd

Revert "Don't apply --exclude-libs for the host."

This reverts commit 1576da248c4a061d906778c2c1dac7478abcc510.

Bug: http://b/24166967
Change-Id: Ibd7eec5c261e85a0474243991be7ee1860c9e14d

Sort all files found via $(shell find)

Don't rely on filesystem ordering to make these the same for all builds.

Change-Id: I7313062157764091acecf45f4b57405c28858546

am 87e0e8a7: Merge "Revert "Only put PARSE_TIME_MAKE_GOALS into KATI_TARGETS""

* commit '87e0e8a7642bd1f5bfdb7b8313df60a77119690a':
  Revert "Only put PARSE_TIME_MAKE_GOALS into KATI_TARGETS"

Merge "Revert "Only put PARSE_TIME_MAKE_GOALS into KATI_TARGETS""

Revert "Only put PARSE_TIME_MAKE_GOALS into KATI_TARGETS"

If there are any goals on the command line, kati hides targets that are
not dependencies of those goals.  Revert this until kati is modified to
include all targets.

This reverts commit b71d24931c7a18799249e3c7f7f6de20ac76cc75.

Bug: 24339367
Change-Id: I7c3a950aef8efc9a0eb72bd29ee07710d2faca99

am f8579330: Merge "Only put PARSE_TIME_MAKE_GOALS into KATI_TARGETS"

* commit 'f8579330a5f6dea88a4d96e4d34ec165a7563030':
  Only put PARSE_TIME_MAKE_GOALS into KATI_TARGETS

Merge "Restore AOSP-style version numbers - DO NOT MERGE" into stage-aosp-master

Merge "Only put PARSE_TIME_MAKE_GOALS into KATI_TARGETS"

am a41dd0a9: Merge "Don\'t apply --exclude-libs for the host."

* commit 'a41dd0a973ad9b890a4e2fe558ad6ed038bacdef':
  Don't apply --exclude-libs for the host.

Merge "Don't apply --exclude-libs for the host."

Don't apply --exclude-libs for the host.

The Mac linker doesn't support this flag, and we don't actually need
it there anyway because we link dynamically to the system's compiler
runtime lib.

Bug: http://b/24166967
Change-Id: I62a926ed39d9fc487638e0c1a172762503dd633e

am c496b259: Merge "Default to hiding libgcc symbols in each object."

* commit 'c496b259b8dddf0eac71a50f5e258da4a353d6bb':
  Default to hiding libgcc symbols in each object.

Merge "Default to hiding libgcc symbols in each object."

Restore AOSP-style version numbers - DO NOT MERGE

Change-Id: Icc9bc2c03d47e1c02d3b535719862e55a9f9b7cd

am 56dca5b5: Merge "Use md5sum as the build.ninja filename suffix if the suffix gets too long."

* commit '56dca5b5408834ff36455d8112e737e968bb6320':
  Use md5sum as the build.ninja filename suffix if the suffix gets too long.

Default to hiding libgcc symbols in each object.

Bug: http://b/24166967
Change-Id: I3dba264ae70a2e13b3edee7575139e86ef49be71

Merge "Use md5sum as the build.ninja filename suffix if the suffix gets too long."

Use md5sum as the build.ninja filename suffix if the suffix gets too long.

Use md5sum as the build.ninja filename suffix if the suffix is longer
than 64 characters. Also store the original suffix in a corresponding
.suf file for human lookup.
This fixes the "File name too long" error when kati tries to write to
build.ninja with the long original suffix.

Bug: 24302576
Change-Id: Ie04b0c14e1a9d593aa5c2a6b357b19a9e8a7158e

am 17a41e36: Merge "Add ld.mc as a requirement for core_tiny.mk as well."

* commit '17a41e36dbe75b94bde05a1c511903ae2162aea8':
  Add ld.mc as a requirement for core_tiny.mk as well.

Merge "Add ld.mc as a requirement for core_tiny.mk as well."

Only put PARSE_TIME_MAKE_GOALS into KATI_TARGETS

Only goals that modify the parsing need to be passed to kati, filter out
everything else and only pass it to ninja.

Change-Id: I28788a3409d601fe838749bcf15394bc8f3d775b

Add ld.mc as a requirement for core_tiny.mk as well.

Bug: 24171451

I missed this when I initially added ld.mc as a requirement for
core_minimal.mk. This is required for RenderScript linking on the
device.

Change-Id: Ie3ffa2454214f886c38387f45b34df2dcbebd6e6

am 9412efdc: Merge "Don\'t build otatools.zip if we don\'t build the ota package."

* commit '9412efdc1b9f54e1bc8df3ea3e8e34b7d899cc70':
  Don't build otatools.zip if we don't build the ota package.

Merge "Don't build otatools.zip if we don't build the ota package."

Don't build otatools.zip if we don't build the ota package.

Also added flag TARGET_SKIP_OTA_PACKAGE to allow product to skip ota
package.

Bug: 24295339
Change-Id: Ib613ed747528661a6572b577cd54ccb6c2d1fdc8

am 9ff24387: Merge "Make USE_NINJA=true the default"

* commit '9ff24387c205da60e3d7deb042530b1d0b2d881c':
  Make USE_NINJA=true the default

Merge "Make USE_NINJA=true the default"

am 30b32620: Merge "Revert "Don\'t re-export libgcc symbols from every object.""

* commit '30b326206ba0148da32c1419f210bd6e6890236d':
  Revert "Don't re-export libgcc symbols from every object."

Merge "Revert "Don't re-export libgcc symbols from every object.""

Revert "Don't re-export libgcc symbols from every object."

For some reason shamu doesn't boot after this change.

Bug: http://b/24166967

This reverts commit e92d25f296af266ebf15bafcc599a91f1f87bd38.

Change-Id: Ibd90c0fccb1fe7446f959c937963d1aa4e43c37c

am 86e1aa1a: Merge "Move python-adb to development/python-packages."

* commit '86e1aa1a1d3593861f3a127e337fbdb8d965ee4f':
  Move python-adb to development/python-packages.

Merge "Move python-adb to development/python-packages."

am 3c4c3418: Merge "Don\'t add build number to apps\' version name for platform build."

* commit '3c4c34183a10fd3deac81a8fc1fed8c774f12190':
  Don't add build number to apps' version name for platform build.

Merge "Don't add build number to apps' version name for platform build."

am 0881ce18: Merge changes Ic1be7572,Id6481430

* commit '0881ce1817395d62d2746a14350767bf411f89d4':
  Pass --ninja to makeparallel for ninja
  makeparallel: improve support for wrapping ninja

am 9bc462c1: Merge "Add the common directory back to kernel headers."

* commit '9bc462c12a5c4519b00e988b738c029c688c12d5':
  Add the common directory back to kernel headers.

Don't add build number to apps' version name for platform build.

Bug: 24201956
Change-Id: I4a4bb483bb7b1bf7b7a856050d548bee4db0fe93

Merge changes Ic1be7572,Id6481430

* changes:
  Pass --ninja to makeparallel for ninja
  makeparallel: improve support for wrapping ninja

Make USE_NINJA=true the default

Build with kati + ninja by default, unless specifically disabled with
USE_NINJA=false in the environment or on the make command line.

Bug: 23565895
Change-Id: I1fa9f5c6eb4d4ed3f5b81454f89d19fd2f1f6abb

Pass --ninja to makeparallel for ninja

When parallelism is being handled by ninja, pass --ninja to makeparallel
to have it translate -j with no number to default ninja parallelism, and
-k to -k0.

Bug: 24199503
Change-Id: Ic1be7572f93b33e4a71ed23e9e73dea1d4255703

makeparallel: improve support for wrapping ninja

Allow makeparallel to pass better -j and -k arguments to ninja if the
first argument to makeparallel is --ninja.  Uses getopt to parse
MAKEFLAGS to get values for --jobserver-fds, -k, and -j, and uses the
result to not pass any -j argument to ninja for make -j with no number,
and pass -k0 to ninja for make -k.

Also improve the test makefile to provide many more tests.

Bug: 24199503
Change-Id: Id6481430f77e9e952213be58a98fe78c46ee5d6a

Merge "Add the common directory back to kernel headers."

Move python-adb to development/python-packages.

Nothing else was being imported from system/core yet, and a path under
development makes more sense for this.

Bug: http://b/22881740
Change-Id: Ib0beb02137805b3e382535514a2bc067b7fd1d33

am 908260a5: Merge "Add support of SANITIZE_TARGET=\'address coverage\' for fuzzing."

* commit '908260a56631cad51196c7b197f720bc88d7f5c7':
  Add support of SANITIZE_TARGET='address coverage' for fuzzing.

Merge "Add support of SANITIZE_TARGET='address coverage' for fuzzing."

Add support of SANITIZE_TARGET='address coverage' for fuzzing.

Also, add trace-cmp instrumentation to fuzz_test and host_fuzz_test.

Bug: 22850550
Change-Id: Ifff7b8be693ae991feb0a64e19439370a19b2748

am 000e1a97: Merge "Don\'t re-export libgcc symbols from every object."

* commit '000e1a973ec83cb2af644ce72a46f1c026299a7d':
  Don't re-export libgcc symbols from every object.

Merge "Don't re-export libgcc symbols from every object."

Add the common directory back to kernel headers.

There will only be a very small set of non-uapi headers. This is
mostly being done for the scsi headers since the kernel has not
made uapi versions of all of them.

Change-Id: I44904b07ff96de918dc5fcab4e5f117a34cb2a87

Don't re-export libgcc symbols from every object.

Target modules shouldn't re-export libgcc.a because we don't want
other binaries importing them when they should be getting their own
copy of the builtins.

Bug: http://b/24166967
Change-Id: Iaa208933885375da4a8e1b8935f76e9f3ea0647f

am 909fe8e0: Merge "build: support compiling .mm on host"

* commit '909fe8e0f7da8db09a004ef44a90e786c81df5e4':
  build: support compiling .mm on host

Merge "build: support compiling .mm on host"

build: support compiling .mm on host

libchrome uses .mm (Objective-C++) files to bridge C++ code with
OS X Frameworks. This adds support for compiling .mm to .o by just
using the existing C++ support.

Bug: 24168923
Change-Id: Ia65357e2e2584dfffcb6796e214fe6b27635c3a6

am 3ee9daac: Merge "Remove WITHOUT_(HOST|TARGET)_CLANG."

* commit '3ee9daac6a3bf8d2a9030b99f62595330f22cafa':
  Remove WITHOUT_(HOST|TARGET)_CLANG.

Merge "Remove WITHOUT_(HOST|TARGET)_CLANG."

am 6fb36397: Merge "Use /ninja_build to mark ninja build."

* commit '6fb36397e56336e08a75e7101d6d47a98a27aea1':
  Use $(OUT_DIR)/ninja_build to mark ninja build.

Merge "Use $(OUT_DIR)/ninja_build to mark ninja build."

Remove WITHOUT_(HOST|TARGET)_CLANG.

This is unused.

Change-Id: Ibc461fef02c82d110718003aceffb943c930ffe6

Use $(OUT_DIR)/ninja_build to mark ninja build.

Bug: 23565895
Change-Id: I000e88a45278fd52096312262eacc52511f02efb

am 45075cf6: Merge "Generate installed-files-vendor.txt"

* commit '45075cf623fc59930594ac27c675cd43e36975b1':
  Generate installed-files-vendor.txt

am 6eb3d977: Merge "Remove legacy kernel build toolchain PATH setup in envsetup.sh"

* commit '6eb3d977c5a8b7612058f1a55e648f8200de2b8f':
  Remove legacy kernel build toolchain PATH setup in envsetup.sh

Merge "Generate installed-files-vendor.txt"