Merge "enable -fstack-protector-strong" am: 2a49aba35b am: c5c2e60707
am: a799c4beab

* commit 'a799c4beabeae4335a4b3d7fd262468ef04953c7':
  enable -fstack-protector-strong

Merge "enable -fstack-protector-strong" am: 2a49aba35b
am: c5c2e60707

* commit 'c5c2e60707cb1b5623705a78724e27efdf3cb7d5':
  enable -fstack-protector-strong

Merge "enable -fstack-protector-strong"
am: 2a49aba35b

* commit '2a49aba35b9b9216df1da245f1af13c6faa5a850':
  enable -fstack-protector-strong

Merge "enable -fstack-protector-strong"

enable -fstack-protector-strong

This results in nearly all functions with the possibility of stack
corruption getting stack canaries, because it applies to any function
taking a reference to the frame or with a local array rather than just
the functions with arrays larger than 8 bytes. It was developed for use
in Chrome (and Chrome OS) and has also been adopted by various other
distributions (Arch, Fedora, Ubuntu, etc).

The code size increase ranges from ~1.5% to ~2.5%, compared to ~0.3% to
~0.7% with the more conservative switch. The increase in the performance
loss is usually minimal. The overall size increase once everything other
than C and C++ code is taken into account is minimal, and it greatly
improves the mitigation of stack buffer overflow vulnerabilities.

https://lwn.net/Articles/584225/

Change-Id: Ie83d381e09d94deb34c5ff142a101ba3d14cf3ba

Merge "Gmake doesn\'t like newline in macro expansion result." am: c7d5e54dfe am: fe27825ca5
am: 82b304264e

* commit '82b304264ea590c6d331e4d87db3167875b31c10':
  Gmake doesn't like newline in macro expansion result.

Merge "Sanity check LOCAL_SRC_FILES_EXCLUDE" am: cc3b2a5854 am: a4769d2956
am: 2b661fe7e0

* commit '2b661fe7e03b73be082b1d5be0d21e9c64393f4b':
  Sanity check LOCAL_SRC_FILES_EXCLUDE

Merge "Gmake doesn\'t like newline in macro expansion result." am: c7d5e54dfe
am: fe27825ca5

* commit 'fe27825ca548e3f0afd869fd9dca91716c578457':
  Gmake doesn't like newline in macro expansion result.

Merge "Gmake doesn\'t like newline in macro expansion result."
am: c7d5e54dfe

* commit 'c7d5e54dfe71766dc60adcf06e6ec1a2b12cdda9':
  Gmake doesn't like newline in macro expansion result.

Merge "Gmake doesn't like newline in macro expansion result."

Gmake doesn't like newline in macro expansion result.

When $(call func) in the makefile text expansion includes newline, gmake
reports error "missing separator". But it's fine if the expansion includes
only spaces. That's a bit odd.
This change fixes the "missing separator".

Change-Id: I094e5ae1725699ebfa4ed66544eef037dd68714c

Merge "Sanity check LOCAL_SRC_FILES_EXCLUDE" am: cc3b2a5854
am: a4769d2956

* commit 'a4769d2956fab9750d856c4f0990f81ff9f6759c':
  Sanity check LOCAL_SRC_FILES_EXCLUDE

Merge "Sanity check LOCAL_SRC_FILES_EXCLUDE"
am: cc3b2a5854

* commit 'cc3b2a58548832ab7a6620d6b607d315f06dfc06':
  Sanity check LOCAL_SRC_FILES_EXCLUDE

Merge "Sanity check LOCAL_SRC_FILES_EXCLUDE"

Merge "Make signapk align .so entries to 4096 bytes." am: dbb5527a9e am: 7f1eda800d
am: 3ad587e9ff

* commit '3ad587e9ff6b90b80b30b82d0bc51cc40b95c85a':
  Make signapk align .so entries to 4096 bytes.

Merge "Remove extra definition of AIDL" am: 77f411e116 am: b275dd1623
am: 934e131465

* commit '934e1314658b30aa8ca22766a3404ff6955d4ece':
  Remove extra definition of AIDL

Merge "Make signapk align .so entries to 4096 bytes." am: dbb5527a9e
am: 7f1eda800d

* commit '7f1eda800d8cdb793b521e20798eaacb3fe410ad':
  Make signapk align .so entries to 4096 bytes.

Merge "Make signapk align .so entries to 4096 bytes."
am: dbb5527a9e

* commit 'dbb5527a9ef27f292794d1e51ae56d1df68c328b':
  Make signapk align .so entries to 4096 bytes.

Merge "Make signapk align .so entries to 4096 bytes."

Merge "Remove synchronized from the public API docs"

Sanity check LOCAL_SRC_FILES_EXCLUDE

Error out if there is a file listed in LOCAL_SRC_FILES_EXCLUDE but not in
LOCAL_SRC_FILES. This should catch typos or other mistakes that would
otherwise be missed.

Change-Id: Iaddf575a6ce35238998ac47b59591a7d05fbcd0d

Use prebuilt tools in place
am: 9a89ed5300  -s ours

* commit '9a89ed5300b1288f9dd4fcfc2f0191fa9772bcb7':
  Use prebuilt tools in place

Merge "Remove extra definition of AIDL" am: 77f411e116
am: b275dd1623

* commit 'b275dd1623ca5256d9dfb8542cd324c624c6e9fe':
  Remove extra definition of AIDL

Merge "Remove extra definition of AIDL"
am: 77f411e116

* commit '77f411e116981ae1de9f060b128e4df0b2facd57':
  Remove extra definition of AIDL

Merge "Remove extra definition of AIDL"

Use prebuilt tools in place

Don't install the prebuilt SDK tools for TARGET_BUILD_APPS builds, just
execute them in place.  Avoids issues with duplicate rules for the
prebuilts and source modules when building with TARGET_BUILD_APPS in a
full platform tree.

Change-Id: I3cdb4f1a8208e8750310a5cc6130657613b683d1
(cherry picked from commit fd08247bcfa47508da3cea41a004310cf28dd0fa)

Remove extra definition of AIDL

AIDL is defined in the sometimes-prebuilt tools section, remove the
extra definition that was overriding it.

Change-Id: I677bf46475fd5fd4a9258c77d91ecee806e5f205

Merge "Use prebuilt tools in place" am: 50e0646dbe am: 8581e58b3b
am: 64747f5101

* commit '64747f5101f3a4f2385faa493643a513bd67a557':
  Use prebuilt tools in place

Merge "Use prebuilt tools in place" am: 50e0646dbe
am: 8581e58b3b

* commit '8581e58b3b845905f19a66a62bb735ac7fdbcf9e':
  Use prebuilt tools in place

Merge "Use prebuilt tools in place"
am: 50e0646dbe

* commit '50e0646dbe1fb52918b38c75aefc49d24bfd9a24':
  Use prebuilt tools in place

Merge "Use prebuilt tools in place"

Merge "Remove warnings from ijar/classfile.cc" am: 128053b91b am: 8707422e4e
am: ba1277549d

* commit 'ba1277549d322091ecbfc6b3738c9590f65ef823':
  Remove warnings from ijar/classfile.cc

Merge "Remove warnings from ijar/classfile.cc" am: 128053b91b
am: 8707422e4e

* commit '8707422e4ee24ca96befbf2bf54deca2590bedfa':
  Remove warnings from ijar/classfile.cc

Merge "Remove warnings from ijar/classfile.cc"
am: 128053b91b

* commit '128053b91b959d0664c33659063c5246eb57a594':
  Remove warnings from ijar/classfile.cc

Merge "Remove warnings from ijar/classfile.cc"

Make signapk align .so entries to 4096 bytes.

This makes signapk align uncompressed .so entries to memory page
boundary (4096 bytes) to enable such libraries to be loaded at runtime
through memory-mapping the APK.

With this change in place, there should no longer be a need to run
zipalign after (or before) signapk.

Bug: 25794543
Change-Id: I74775af15a683791f57fcbd3497a79951b3f63a1

Merge "releasetools: Always use recovery mount options." am: 643d466f7b am: 1709cf3768
am: 50aa7c731b

* commit '50aa7c731b338d32d676326589652f21249dc2c9':
  releasetools: Always use recovery mount options.

Merge "releasetools: Always use recovery mount options." am: 643d466f7b
am: 1709cf3768

* commit '1709cf3768b54c1abc267b5ee92f57023e206c26':
  releasetools: Always use recovery mount options.

Merge "releasetools: Always use recovery mount options."
am: 643d466f7b

* commit '643d466f7b853ab4009723e2da200e3a6642d2ad':
  releasetools: Always use recovery mount options.

Merge "releasetools: Always use recovery mount options."

Merge "Provide path to mainDexClasses" am: 62e99632c6 am: 4e7bf63314
am: 7e68aa8251

* commit '7e68aa8251afde9f885b7961aff5bbbabe840031':
  Provide path to mainDexClasses

Merge "Provide path to mainDexClasses" am: 62e99632c6
am: 4e7bf63314

* commit '4e7bf63314c8684bf26219c67d220341a03e4789':
  Provide path to mainDexClasses

Merge "Provide path to mainDexClasses"
am: 62e99632c6

* commit '62e99632c61ef74fb04cf2bf6e079844e5dd8ae1':
  Provide path to mainDexClasses

Use prebuilt tools in place

Don't install the prebuilt SDK tools for TARGET_BUILD_APPS builds, just
execute them in place.  Avoids issues with duplicate rules for the
prebuilts and source modules when building with TARGET_BUILD_APPS in a
full platform tree.

Change-Id: I3cdb4f1a8208e8750310a5cc6130657613b683d1

Merge "Provide path to mainDexClasses"

Provide path to mainDexClasses

Set MAINDEXCLASSES to the path to mainDexClasses.

Change-Id: I82532e8ea8dc7349178347e080fa0c63078ca9f5

releasetools: Always use recovery mount options.

Change-Id: I6e96c417ffb5ad36405f4a6f5342f80f358f375f

Merge "releasetools: Abort the update on apply_patch failures." am: 2a23c0c859 am: d77af53fc6
am: 21362f024b

* commit '21362f024b62e9ebab89c9a813229b4bd0bc9369':
  releasetools: Abort the update on apply_patch failures.

Merge "releasetools: Abort the update on apply_patch failures." am: 2a23c0c859
am: d77af53fc6

* commit 'd77af53fc673229a1e9b59d99592272f689df10f':
  releasetools: Abort the update on apply_patch failures.

Merge "releasetools: Abort the update on apply_patch failures."
am: 2a23c0c859

* commit '2a23c0c85919bada137985f9e3b32514185b38c5':
  releasetools: Abort the update on apply_patch failures.

Merge "releasetools: Abort the update on apply_patch failures."

releasetools: Abort the update on apply_patch failures.

We used to silently ignore the return value from apply_patch() even if
it had failed. It gives us more trouble to investigate the failure when
the affected file/partition gets touched in subsequent OTAs. This CL
adds the checking of the return value and aborts the update accordingly.

Bug: 25893277
Change-Id: Ie5e1c563576e503343e6a5b28ed4d7039f6f919c

Merge "make USER limited to 6 characters when generating BF_BUILD_NUMBER" am: 88ca8aff3e am: 79215cdeab
am: 4c6b66e617

* commit '4c6b66e6172ffcc879ff48f2a245f670471f52e2':
  make USER limited to 6 characters when generating BF_BUILD_NUMBER

Merge "make USER limited to 6 characters when generating BF_BUILD_NUMBER" am: 88ca8aff3e
am: 79215cdeab

* commit '79215cdeabbc43e47cbfe212eed02e6958ebfb8b':
  make USER limited to 6 characters when generating BF_BUILD_NUMBER

Merge "make USER limited to 6 characters when generating BF_BUILD_NUMBER"
am: 88ca8aff3e

* commit '88ca8aff3e4c6f776444ca8267b044b314f6aa0f':
  make USER limited to 6 characters when generating BF_BUILD_NUMBER

Merge "make USER limited to 6 characters when generating BF_BUILD_NUMBER"

make USER limited to 6 characters when generating BF_BUILD_NUMBER

the value of USER is dependent from the compilation environment,so
when compiling one same device project, the BUILD_FINGERPRINT may
exceed 91 characters because ${USER} is long, but with short ${USER}
the compilation can pass.

Signed-off-by: wei qiao <qiaowei224@gmail.com>
Change-Id: Ia0f7dfa9cf7d605f1f2603f70dd0e6877482eb8a

Remove warnings from ijar/classfile.cc

Also add -Werror.

Change-Id: I2f9fda288d31bd34735856ade33916c93c45922a

Merge "Stop packaging obsolete OTA tools." am: b98fa19703 am: e82ea963e8
am: 2867533add

* commit '2867533add946cd9d2294ed3b4976f5ecf7aa76e':
  Stop packaging obsolete OTA tools.

Merge "Stop packaging obsolete OTA tools." am: b98fa19703
am: e82ea963e8

* commit 'e82ea963e8cf5d7eb2dd6d2c7aacee86ddfba853':
  Stop packaging obsolete OTA tools.

Merge "Stop packaging obsolete OTA tools."
am: b98fa19703

* commit 'b98fa19703ed0c7608f926007b1df257127ea777':
  Stop packaging obsolete OTA tools.

Merge "Stop packaging obsolete OTA tools."

Merge "Make export_includes a restat rule" am: 615d66e2c2 am: b16a63cbb0
am: ceac25c571

* commit 'ceac25c5719aab39d7a15fd1c1578d8839a86911':
  Make export_includes a restat rule

Merge "Make export_includes a restat rule" am: 615d66e2c2
am: b16a63cbb0

* commit 'b16a63cbb0582637b0e6f28792ae6d8de718e357':
  Make export_includes a restat rule

Merge "Make export_includes a restat rule"
am: 615d66e2c2

* commit '615d66e2c21603552dc952db9493f43bbcc7f2b1':
  Make export_includes a restat rule

Merge "Make export_includes a restat rule"

Make export_includes a restat rule

There is currently an intentional incremental rebuild issue with
import_includes.  export_includes might get updated with an identical
version, but we don't want to force everything downstream of it to
rebuild.

When BUILDING_WITH_NINJA==true, only update export_includes if it
changes, and use .KATI_RESTAT to only run downstream rules if it
changes.  import_includes will only be updated if one of the
export_includes files is updated, so object files can have a normal
dependency on import_includes instead of an order-only dependency.
All downstream object files will now be recompiled if their imported
include paths change.

Bug: 25910568
Change-Id: I626f3b24ac02ac1309049cf1ce66cfe8ec816513

Merge changes I1de10391,I4399ca26,Iec3b2b0b am: bff3c9b4c1 am: bae8872294
am: dbeab8de65

* commit 'dbeab8de6543db1d0393789cb18ee7fd7ed8fedd':
  Use libstdc++ for ijar
  Build ijar for apps build
  Use .KATI_RESTAT to reduce unnecessary rebuilds of .jar files

Merge changes I1de10391,I4399ca26,Iec3b2b0b am: bff3c9b4c1
am: bae8872294

* commit 'bae88722945130a0c8ebbb1ef27eaa845e9f03fd':
  Use libstdc++ for ijar
  Build ijar for apps build
  Use .KATI_RESTAT to reduce unnecessary rebuilds of .jar files

Merge changes I1de10391,I4399ca26,Iec3b2b0b
am: bff3c9b4c1

* commit 'bff3c9b4c10dcb3ce3820d3a5e144e3df20313dc':
  Use libstdc++ for ijar
  Build ijar for apps build
  Use .KATI_RESTAT to reduce unnecessary rebuilds of .jar files

Merge changes I1de10391,I4399ca26,Iec3b2b0b

* changes:
  Use libstdc++ for ijar
  Build ijar for apps build
  Use .KATI_RESTAT to reduce unnecessary rebuilds of .jar files

Merge "Copy video stop sound"

Stop packaging obsolete OTA tools.

applypatch, applypatch_static and sqlite3 are no longer needed.

Bug: 24621915
Change-Id: I6de5bc45bc8434b9c387e1d3d1b6c347c23a5290

Merge "releasetools: Allow logging the diff for incrementals." am: 6d6e0481fa am: e88245ddf0
am: 94792ca281

* commit '94792ca281e1c87ccdefb3f8c0e91000a6e1b6f5':
  releasetools: Allow logging the diff for incrementals.

Merge "releasetools: Allow logging the diff for incrementals." am: 6d6e0481fa
am: e88245ddf0

* commit 'e88245ddf0a4920ff90866b9d66b2a3e4bd89f18':
  releasetools: Allow logging the diff for incrementals.

Merge "releasetools: Allow logging the diff for incrementals."
am: 6d6e0481fa

* commit '6d6e0481fa86cb552e1b386e5449fe1fde2325cf':
  releasetools: Allow logging the diff for incrementals.

Merge "releasetools: Allow logging the diff for incrementals."

releasetools: Allow logging the diff for incrementals.

Add an option "--log_diff <filename>" to ota_from_target_files.py
script. When enabled, it logs the differences between the source
and target builds into <filename> when generating incremental OTAs.

Also move target_files_diff.py into releasetools/ so that it can be
packed into otatools.zip.

Bug: 25372309
Change-Id: Ifd4ed0f2f12ef040ee377621ec8c35a873cec34f

Use libstdc++ for ijar

For some reason ijar won't build against libc++ for TARGET_BUILD_APPS
builds, but does build with libstdc++.

(cherry picked from commit 718bab6aecfe7e23a6afa1361da64c8e0c832705)

Bug: 25904002
Change-Id: I1de103918faa5bb574af6f12cc954e7fed44dc62

Build ijar for apps build

This fixes git_master-linux-apps build.

(cherry picked from commit 90847057446374f818b49a742c567f82db02438e)

Change-Id: I4399ca2662e08c9b3cc2ad5a317e84d91b728115

Use .KATI_RESTAT to reduce unnecessary rebuilds of .jar files

The same optimization was done for binaries in

https://android-review.googlesource.com/#/c/175250/

To create a TOC file from .jar files, this change introduces
ijar, which is designed for this purpose. Only #include lines
were modified from the original version.

https://github.com/bazelbuild/bazel/tree/master/third_party/ijar

Performance:

$ m && touch
frameworks/base/core/java/com/google/android/util/Procedure.java && time
m
Before: 4m30s (1580 targets)
After: 3m57s (772 targets)

Unfortunately, the improvement is small yet, but local
experiments showed we can cut ~2 more minutes if the similar
optimization is done for .dex files.

(cherry picked from commit c1f5d9c203324b9435414e789c02394ca99c98f7)

Bug: 24597504
Change-Id: Iec3b2b0b0e674bee5d80cce3c300dc8fad6e7c13

Merge "Revert "Use .KATI_RESTAT to reduce unnecessary rebuilds of .jar files"" am: 2ff3ad9045 am: 51ef51c4ad
am: 371b8c7f77

* commit '371b8c7f770f8215e6bda1433e5f236c106b4e64':
  Revert "Use .KATI_RESTAT to reduce unnecessary rebuilds of .jar files"

Merge "Revert "Build ijar for apps build"" am: d17c3a57f6 am: c3725c256b
am: 15bead2c59

* commit '15bead2c598308b5a6cd17b64c839676d7286ed8':
  Revert "Build ijar for apps build"

Merge "Revert "Use .KATI_RESTAT to reduce unnecessary rebuilds of .jar files"" am: 2ff3ad9045
am: 51ef51c4ad

* commit '51ef51c4ad283198d5fb4a2ec9caecdc7a677838':
  Revert "Use .KATI_RESTAT to reduce unnecessary rebuilds of .jar files"

Merge "Revert "Build ijar for apps build"" am: d17c3a57f6
am: c3725c256b

* commit 'c3725c256b5d786007f2565324324b386287725a':
  Revert "Build ijar for apps build"

Merge "Revert "Use .KATI_RESTAT to reduce unnecessary rebuilds of .jar files""
am: 2ff3ad9045

* commit '2ff3ad90455e868a1805d065f0aa2b8babbc6660':
  Revert "Use .KATI_RESTAT to reduce unnecessary rebuilds of .jar files"

Merge "Revert "Build ijar for apps build""
am: d17c3a57f6

* commit 'd17c3a57f6f183b97cc557e10e221b4dc1acf47f':
  Revert "Build ijar for apps build"

Merge "Revert "Use .KATI_RESTAT to reduce unnecessary rebuilds of .jar files""

Merge "Revert "Build ijar for apps build""

Revert "Use .KATI_RESTAT to reduce unnecessary rebuilds of .jar files"

This reverts commit c1f5d9c203324b9435414e789c02394ca99c98f7.

Change-Id: I7e9969a060e3f1380154da65e76134168aee15d8

Revert "Build ijar for apps build"

This reverts commit 90847057446374f818b49a742c567f82db02438e.

Change-Id: I8a9958788c51dcec5028ef396cb67abd541ce31f

Merge "Revert "Use libstdc++ for ijar"" am: 41dca40376 am: 6cd88d7514
am: 06c44241da

* commit '06c44241da5ce7bb055a2ae7160daeb2bf66d85e':
  Revert "Use libstdc++ for ijar"

Merge "Revert "Use libstdc++ for ijar"" am: 41dca40376
am: 6cd88d7514

* commit '6cd88d75147217bc42fed67b558f9e4c836a20a0':
  Revert "Use libstdc++ for ijar"

Merge "Revert "Use libstdc++ for ijar""
am: 41dca40376

* commit '41dca403766d22373ba469d99ba1625d0930d02e':
  Revert "Use libstdc++ for ijar"

Merge "Revert "Use libstdc++ for ijar""

Revert "Use libstdc++ for ijar"

This reverts commit 718bab6aecfe7e23a6afa1361da64c8e0c832705.

Change-Id: I9aa222fc2936d7d89ae0700eeef505c6b317d777

Merge "Use libstdc++ for ijar" am: c97b5272d7 am: 4c1944083f
am: 8713fde1c3

* commit '8713fde1c3edf9dd6c5799cd3cc25b8f2ec2cb2d':
  Use libstdc++ for ijar

Merge "Use libstdc++ for ijar" am: c97b5272d7
am: 4c1944083f

* commit '4c1944083f03e1ddf55f16f815b89a337ffc60f6':
  Use libstdc++ for ijar

Merge "Use libstdc++ for ijar"
am: c97b5272d7

* commit 'c97b5272d715ab9f4ff52748bd2f403235f94426':
  Use libstdc++ for ijar

Merge "Use libstdc++ for ijar"