Don't duplicate default platform libraries into proguard flags

Test: builds
Change-Id: Ib972d954972614f3e6ce522c58fce06beeda3cea

Merge "Stop double-printing config"

Merge "Revert "Replace hardware features conf file to be emulator specific""

Merge "Prevent duplicate signals from makeparallel"

Stop double-printing config

Don't print the config in Kati when we've already printed it in soong_ui
or the make wrapper.

Test: m -j
Test: USE_SOONG_UI=true m -j
Change-Id: Ic6dce0ab29d584d85c6642c597f6e9dcd6ba9dca

Prevent duplicate signals from makeparallel

As part of the soong_ui effort, we noticed that we'd sometimes see
multiple SIGINTs from a single ctrl-c from a user. ctrl-c sends a SIGINT
to the entire process group, so make, makeparallel, soong_ui, and all of
its children would get a signal. Since makeparallel was passing it along
to it's child, soong_ui would get two signals.

So instead, follow what Make does and only pass along SIGTERM. Assume
that all other signals went to the entire process group.

Bug: 35214134
Test: Send SIGINT to process group, check in makeparallel's child for
more than one signal received.
Change-Id: I5b2a77ad0fcebbaa5087439948e71bf3b541061a

Merge "releasetools: allow for multiple OEM property values."

releasetools: allow for multiple OEM property values.

This CL changes the --oem_settings flag to allow a comma seperated list of
property files. All property values will be used when asserting properties such
as ro.product.name.

For example, if two property files are provided with ro.product.name values of
"sprout" and "sprout_a", the resulting otapackage will check that the device's
ro.product.name property matches at least one of them.

Bug: 34191373
Test: manual

Change-Id: I954673511be8f0929982235cc9cbfbd85a9ee1f4

Merge "2nd attempt to remove legacy-test from TARGET_DEFAULT_JAVA_LIBRARIES"

Merge "Revert "Revert "Add soong_javac_filter to all javac commands"""

Revert "Revert "Add soong_javac_filter to all javac commands""

This reverts commit 52d2253ff9617e0c1ad767274e8627c39776198a.

Test: pipe the grok build output through soong_javac_filter
Change-Id: I449a61ed2c95b3e1afd51a31a11baab15d97e2f2

Merge "Disable building coverage with javac"

Merge "releasetools: Allow using futility-host to sign old TF.zip."

Revert "Replace hardware features conf file to be emulator specific"

This reverts commit a118a21b009e09514f7b245f60ac4139217ce0ce.

Change-Id: I2c4140667a2c00abad3c6337c680542070dd7caa

Disable building coverage with javac

Emma is unable to process framework.jar, and making framework
classes.jar depend on the emma output causes builds to fail when
emma is enabled.  Make classes.jar depend on the jarjar output
instead, bypassing emma for now.

Test: m -j EMMA_INSTRUMENT=true EMMA_INSTRUMENT_FRAMEWORK=true check-public-api
Change-Id: I861282292816e29ca27b67a3eef978d951c1a4dc

releasetools: Allow using futility-host to sign old TF.zip.

Commit e98fb7a8d3a460891f6c21fafbe62a86b50d3d1d switched to using
futility-host instead of the prebuilt futility. This CL adds support to
handle signing old TF.zip that still says "futility=prebuilt/..." in
META/misc_info.txt.

Bug: 35467608
Test: Generate otatools.zip and sign an old ryu TF.zip.
Change-Id: I48a9cc918c7afce361e1ec9bc4f85f74fa92566e

Merge "embedded.mk: allocator service"

Merge "Build all the way to classes.dex for checkbuild"

Merge "Add a desugar step between javac and proguard"

Merge "Build FUTILITY tool from futility-host."

Merge "Remove unused zipalign macros"

Merge "Add Header Library support to Make"

Merge "Replace hardware features conf file to be emulator specific"

Build all the way to classes.dex for checkbuild

Test: m -j checkbuild
Change-Id: Ie1b7be01c9bbcf1a32399f19d050a93410bdb42b

Add a desugar step between javac and proguard

This enables support for lambdas by passing class files through the
desugar tool.

Test: m -j checkbuild tests cts
Change-Id: I14ec152355243fd67fe2f107ccbe67a1b4e7e262

Merge "Revert "Add soong_javac_filter to all javac commands""

Revert "Add soong_javac_filter to all javac commands"

This reverts commit a341bf0f8636ec42bd87058d9ba31a482cea3b6d.

Breaks grok builds that replace javac with another tool:
reading standard input: bufio.Scanner: token too long

Change-Id: I825587c465b05c1fbddb7794dc1c84f5f2c883d1

Merge changes I7e8fcd2e,Ie12d743c

* changes:
  Add soong_javac_filter to all javac commands
  Re-enable dx support

Build FUTILITY tool from futility-host.

Also pack the tool, vboot_signer.sh script and the test keys into
otatools.zip. This allows signing ryu target without a source repo.

Bug: 35467608
Test: m otatools-package and check the newly included files.
Test: Use otatools.zip to sign a ryu target.
Change-Id: I70f7b359f21a579463d5c7251daac81a97e338cc

Replace hardware features conf file to be emulator specific

full_base_telephoney.mk file gets used only for emulators which should
have bluetooth disabled. Using the default handheld_core_hardware.xml
file has bluetooth enabled. We should use the generic
handheld_core_hardware.xml file for emulators which has bluetooth
disabled.

Its presence causes problems upstream in SystemServer when
BluetoothService starts and requires an extra emulator config change
which should not be there.

Bug: 35361545
Test: Verified by launching Android Emulator. It works fine and does not
start bluetooth.

Change-Id: I484c3d5867f46853fb49f28422dd72ab362ed768

embedded.mk: allocator service

We are splitting android.hidl.memory into android.hidl.allocator and
android.hidl.memory to reflect the fact that we have two separate
interfaces which are served over different transports.

Bug: 35327976
Test: hidl_test, device boots with allocator

Merged-In: I36b1554d20f89ecd60a836c04f788ac83e0a1f5c
Change-Id: I36b1554d20f89ecd60a836c04f788ac83e0a1f5c

Remove unused zipalign macros

Test: codesearch
Test: build-marlin.ninja is identical before/after (in internal master)
Change-Id: I1c5c39cabcc166b8e8608ace3290b7027d258624

Merge "zipalign: remove support for adding members from gzip sources."

Merge "Set IS_AT_LEAST_* variables based on TARGET_PLATFORM_VERSION"

Set IS_AT_LEAST_* variables based on TARGET_PLATFORM_VERSION

Create a default list of minor versions (v, vDR1, vDR2, vMR1, vMR2)
for O and P, and propagate TARGET_PLATFORM_VERSION to IS_AT_LEAST_*
variables.

Test: manual
Bug: 34610721
Change-Id: I575caa099d85ee6fbc79538e67dc8d621e81790f

Add soong_javac_filter to all javac commands

soong_javac_filter colorizes javac output, and hides noisy messages.

Test: builds
Change-Id: I7e8fcd2e4e1ed3ff530a8ccfe931ceb7e411e0ad

Re-enable dx support

This is a partial revert of commits:
858657366fa6 Remove support of disabling Jack.
3ae78612522e Remove javac support in host dex rules.
22313f2b2a4a Remove rules for building dex with dx

Test: builds
Change-Id: Ie12d743cbe978bdeb030910848b67f5945a4fec8

zipalign: remove support for adding members from gzip sources.

This was vestigial code and never actually called anywhere.

Test: make
Bug: 35246701
Change-Id: I9297ac5fa030d369946857414155450635b65410

Add Header Library support to Make

We currently use static libraries without any source files to represent
header libraries, but Soong actually has cc_library_headers. So to
export those in a separate namespace from static libraries, implement
them in Make as well.

This also adds a nice pretty-warning / pretty-error macro that can be
used to print out standard warning messages pointing to the real source
file having the problem.

Test: Use a header library exported by Soong in a Make module
Change-Id: I3486539e247524cb82a20620745fc7be03014e14

Merge "Blacklist code for CFI."

Blacklist code for CFI.

Adds the -fsanitize-blacklist option for CFI, using the built in
blacklist at external/compiler-rt/lib/cfi/cfi_blacklist.txt.

Also refactors the CFI cflags and ldflags into ../soong/cc/makevars.go
to ensure they're consistent across Soong and make projects.

Bug: 30227045
Test: ENABLE_CFI=true m -j40 builds and boots.
Test: The blacklist prevents runtime errors that otherwise occur.
Change-Id: I0c2801ed459a3b9adeb37daff3ca212564801259

Merge "Fix init rc installation in vendor when LOCAL_MODULE_PATH is set"

Merge "Support Soong native coverage"

2nd attempt to remove legacy-test from TARGET_DEFAULT_JAVA_LIBRARIES

Bug: 30188076
Test: make checkbuild

(cherry picked from commit 01fc887a07ed8d07f6e7cd75a812b90956543338)

Change-Id: I114948232fd63eaf57a777d37623e4c931bdd0ff

Merge "Improve installclean to catch more installed files"

Merge "Share *_OUT_TESTCASES environment variables with TF"

Merge "Don't build tests in normal builds"

Merge "releasetools: Fix a bug in ReviseStashSize()."

Fix init rc installation in vendor when LOCAL_MODULE_PATH is set

The init.rc path selection uses the same partition_tag detection to
select an installation partition. So make sure that we always set
partition_tag even if we've been given an explicit module path.

Bug: 35314904
Test: set proprietary: true to Soong module with an init rc, verify
      that was the only change to build-aosp_arm64.ninja
Change-Id: Ic6ffd7a5177959db96a34a24896b242ee5b09e41

Share *_OUT_TESTCASES environment variables with TF

Instead of having test tools search the OUT_DIR for the
testcases folders (of which there can be multiple different
targets), let's have these variables become part of the runtime
environment after a developer has run lunch.

Bug: 35322559
Test: . build/envsetup.sh ; lunch ; printenv | grep TESTCASES
Change-Id: Ic4addc2b9727ff717b2e595b9be224650e2f1b1d

Improve installclean to catch more installed files

Remove more installation directories. This gets called in between each
incremental build on our build servers.

Bug: 35267243
Test: (m -j; m -j installclean; m -j) ensure no large recompiles
Change-Id: Ied6bea5cad05c1f5e98b1bd1a30fb28ff09df23b

Merge "fat16copy: Allow to copy files to an existing directory."

Merge "Disble CFI for ARM32 processes."

releasetools: Fix a bug in ReviseStashSize().

We check the needed stash size in ReviseStashSize(), and may not
generate a stash command if it would exceed the max allowed size. This
CL fixes a bug when skipping a stash operation: we shouldn't update the
'stashes' map if a stash command won't be generated.

Bug: 35313668
Test: Successfully generate the package that was failing due to the bug.
Change-Id: If0a3a5fadda9b4a4edad66a2a5826b5f978400ae

Merge "Include the LOCAL_REQUIRED_MODULES when packaging."

fat16copy: Allow to copy files to an existing directory.

This patch re-uses existing directories when copying directories,
allowing to copy files to a directory previously created.

Bug: 34811400
Test: fat16copy.py fat.bin a/b/foo; fat16copy.py fat.bin c/d/foo

(cherry-picked from 0af24a27a8d0dc7e1782f36a5a9963021ff8b3f6)

Change-Id: I283184da18f3c4951d6c84aafa4185074791acd2

Merge "Update CleanSpec.mk for the common testcase directories."

Merge "Add support for preopted app images"

Update CleanSpec.mk for the common testcase directories.

Fix an issue with incremental builds and the new common
testcase directories.

Bug: None
Test: Removed https://android-review.googlesource.com/#/c/234478/
      then ran `m cts-platform-version-check` then reapplied the
      change to verify the failure. Then applied this new change
      to verify the problem was resolved.

Change-Id: I77529021e852af35571fb99537d87100010b47af

Add support for preopted app images

Enable iff WITH_DEXPREOPT_APP_IMAGE is true.

Test: Clean + build with the flag on and off. Tested Sailfish
booting.

Bug: 34927277

(cherry picked from commit e8ab2a71d7bff1047c3d0497feba6bdcc02b8277)

Change-Id: I63f98a85c69d99746b11bb6e85c23cf5e48910cf

Merge "Output *TS testcases to their own subdir in the common testcase folders."

Support Soong native coverage

Soong (or any other prebuilt) can provide a .gcnodir file using
LOCAL_PREBUILT_COVERAGE_ARCHIVE. Just like today, if one isn't provided
for static libraries, an empty one will be created for Make modules to
use.

Also fixes a problem I noticed including whole static libs having
coverage into static libs. It was trying to write to /WHOLE since
PRIVATE_INTERMEDIATES_DIR was not set.

Bug: 32749731
Test: See build/soong change for soong-side tests
Test: Enabled coverage of a soong static lib, then include it in a make
static lib with LOCAL_WHOLE_STATIC_LIBRARIES and ensure that the any
shared libraries using the make static lib get the embedded coverage
information.
Change-Id: I32762f099c9757074fec922ee4822f819c9ceaf5

Merge "Remove mk file parser variables that are no longer required by system/bt"

Merge "Sort clear_vars.mk again"

Sort clear_vars.mk again

Test: none
Change-Id: Id776e81977695c4b488f44b155f2c098d5f8b61a

Remove mk file parser variables that are no longer required by system/bt

* Removed BLUETOOTH_HCI_USE_MCT
* Removed BOARD_BLUETOOTH_BDROID_HCILP_INCLUDED

Bug: 35203653, 34951912
Test: Code compilation, no user visible effects
Change-Id: Ic575d3a0fac541da3e70543d0f144ea4792bc058

Merge "Fix "Nothing to be done" messages when launching soong_ui"

Fix "Nothing to be done" messages when launching soong_ui

When specifying multiple arguments to make, we were getting messages
about nothing to be done:

m -j USE_SOONG_UI=true checkbuild tests cts blueprint_tools
...
make: Nothing to be done for `tests'.
make: Nothing to be done for `cts'.
make: Nothing to be done for `blueprint_tools'.

Test: m -j USE_SOONG_UI=true blueprint_tools libc adb
Change-Id: Id448bf0e09736fcd92ba058aec7ce556f6f778a3

Disble CFI for ARM32 processes.

This CL disables CFI for 32-bit ARM processes, which is broken due to
a compiler error in the most recent version of clang.

Bug: 35157333
Test: ENABLE_CFI=true m -j40 does not enable CFI for 32-bit processes
Change-Id: I6a6e1d14c8365da1056b4cbccad11ef2f15c70b2

Output *TS testcases to their own subdir in the common testcase folders.

Instead of dumping all *TS testcases into the common testcase
directory directly, place them all in their own subdirectory.

Bug: 35153177
Test: `make cts -j` and checked the contents of both:
out/host/linux-x86/testcases/ &
out/host/linux-x86/cts/android-cts/testcases/

Change-Id: I22a768dba58d40be1e61885534fb813e7ff08c99

Merge "Output *TS testcases to common testcase folders."

Merge "Add RenderScript Toolchain information to Soong."

Merge "releasetools: Support generating downgrade A/B OTAs."

Output *TS testcases to common testcase folders.

Aggregates the different *TS testcases into two common folders,
one under each the host and product hierarchies.

Bug: 34715822
Test: `make cts -j` and checked the contents of both:
out/host/linux-x86/testcases/ &
out/host/linux-x86/cts/android-cts/testcases/

Change-Id: I7d5201998ddcb011ecc687c272a0d0a36da248b7

releasetools: Support generating downgrade A/B OTAs.

We already support generating downgrade OTAs for non-A/B devices (with
mandatory data wipe), but we have missed the --downgrade flag in A/B OTA
path.

This CL factors out the function that writes the downgrade metadata, and
fixes the path for generating A/B OTAs.

Bug: 35094540
Test: Generate incrementals with --downgrade for A/B and non-A/B OTAs.
Change-Id: I30b9bf83e69e8aba3be666507681b555db6ab743

Merge "releasetools: Pack the offset/length for metadata."

Merge "Add USE_SOONG_UI to trigger soong_ui in Make mode"

Add USE_SOONG_UI to trigger soong_ui in Make mode

Test: USE_SOONG_UI=true m -j
Change-Id: Ic7a0b4e0881757b0324722d7d316fb8d13dabd9b

Merge "Create static archives in temporary file"

Merge "Add c_std and cpp_std properties."

Merge "Revert "Remove legacy-test from TARGET_DEFAULT_JAVA_LIBRARIES""

Revert "Remove legacy-test from TARGET_DEFAULT_JAVA_LIBRARIES"

This reverts commit 01fc887a07ed8d07f6e7cd75a812b90956543338.

A number of build breakages occurred which were not caused by presubmit checks.

Change-Id: I1351dba594c8a8b210416f75a67c68554d5b9372

Merge "Remove legacy-test from TARGET_DEFAULT_JAVA_LIBRARIES"

Merge "releasetools: Fix the bug when generating care_map.txt."

releasetools: Fix the bug when generating care_map.txt.

Commit f1a13180dba9c0145ac7bec8371340ae303c81c1 intended to remove the
verity blocks from care_map.txt, but it added new code without removing
the old one. This leads to a malformed care_map.txt and causes
update_verifier failure.

Bug: 34391662
Test: 'm -j dist' gives a TF.zip with 4-line META/care_map.txt (as
      opposed to a 6-line file).

Change-Id: I7ff1aa525795c4b049af54c1755b0f0ea84f7e0e

releasetools: Pack the offset/length for metadata.

For streaming OTAs, we will also need the info in the metadata entry
(META-INF/com/android/metadata). Compute and pack its offset/length
values into 'ota-streaming-property-files'.

Bug: 34986195
Test: Create an OTA package and check the offset/length values.

Change-Id: Id150700f2bc9bff02467cda9fe8927c8a374412a

Add c_std and cpp_std properties.

Test: Checked showcommands ouput for each setting.
Bug: None
Change-Id: Ibc25f3977eaaecfd1fb99a27c9e15acae0ec4106

Merge "Disable CFI on Mips and add -march to linkflags on ARM."

Create static archives in temporary file

Creating static archives is often a multi-command process due to
adding whole static libraires or hitting command line length limits.
If one of the intermediate commands fails, the output file may
already exist.  Unlike make, ninja has no option to delete output
files on failed builds, instead assuming all build commands will
produce their output file atomically
(https://github.com/ninja-build/ninja/issues/1135).

Change the static library rules to generate to a temporary file
that is then atomically moved into place as the output file.

Test: m -j checkbuild tests cts
Change-Id: I4faf269f0c8e313c738154870a5aa0b4774a72bc

Merge "Remove verity blocks from care map"

Add RenderScript Toolchain information to Soong.

Test: make -j64 from top of the tree.

Change-Id: I230f8d1265a71344bba8883ee9bdf2de98fcd41f

Merge "Switch to clang-3688880."

Disable CFI on Mips and add -march to linkflags on ARM.

Mips toolchain does not have ld.gold.
ARM change is a workaround for LLVM r290384.

Bug: 33678192
Test: make ENABLE_CFI=1
Change-Id: I77a127e0b472d5da10bf45a2983527a714339cb8

Merge changes Idd6b24a1,I4323acdb

* changes:
  Remove -host
  Add libext2_misc to otatools package

Remove -host

Bug: 34689537
Change-Id: Idd6b24a14caa863857303633ba15bd3b41a3a624
(cherry picked from commit b24470e9890fbbba5a56a88bba353191cb00700b)

Add libext2_misc to otatools package

Bug: 34689537
Change-Id: I4323acdbadb1c4c8faa6dbb17a79bce72c063407
(cherry picked from commit 3d1ce3a6400852eb9f3e687e6286f31b301fc766)

Merge "releasetools: Prepend 'ota-' to 'streaming-property-files'."

releasetools: Prepend 'ota-' to 'streaming-property-files'.

'streaming-property-files' is a property related to the OTA package
itself. Prepend 'ota-' to make it consistent with others like
'ota-type' and 'ota-required-cache'.

Bug: 34852392
Test: Generate an A/B OTA package and check METADATA entry.
Change-Id: Ia681e6e19ff509e6da0d8718933b42aac997e1cf

Switch to clang-3688880.

Bug: http://b/31532493
Test: Manual validation.
Change-Id: I7c415a1008c368a4b65bac2b5cd3ded3f4bb9de0

Merge "Fix symbol zip creation with OUT_DIR outside source"