aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "bp2build: Add support for cc_binary."Liz Kammer2021-10-204-9/+607
|\
| * bp2build: Add support for cc_binary.Liz Kammer2021-10-194-9/+607
| | | | | | | | | | | | Bug: 197920036 Test: build/bazel/ci/bp2build.sh Change-Id: I2c4200967653af15a330ab8cbaf796b70d43f32d
* | Merge "Remove artApex parameter from ApexBundleFactory"Paul Duffin2021-10-201-10/+1
|\ \
| * | Remove artApex parameter from ApexBundleFactoryPaul Duffin2021-10-201-10/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since coverage support was added to bootclasspath_fragment in https://r.android.com/1684807 and bootclasspath_fragments were integrated into the ART APEX it is no longer necessary to treat ART APEXes differently to other APEXes. Change https://r.android.com/1684807 added coverage support, follow up changes integrated Bug: 177892522 Test: m droid m EMMA_INSTRUMENT=true EMMA_INSTRUMENT_FRAMEWORK=true Change-Id: I4800a003c7459c4686607e16f4c5bde0be105f16
* | | Merge "Add OS to configuration key in mixed builds"Christopher Parsons2021-10-198-61/+79
|\ \ \
| * | | Add OS to configuration key in mixed buildsChris Parsons2021-10-198-61/+79
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This also removes the special-case filegroup from mixed builds buildroot; no special handling is required. Since we're currently hardcoding linux_x86_64 as our host platform, it should be fine to continue assumping that hardcoded host for now. Test: USE_BAZEL_ANALYSIS=1 m adbd Change-Id: I35509f4eb33ba7a243fab4c34b35958f6f2fceab
* | | | Merge "bp2build include generated hdrs in all src splits"Liz Kammer2021-10-192-22/+75
|\ \ \ \ | |/ / / |/| | / | | |/ | |/|
| * | bp2build include generated hdrs in all src splitsLiz Kammer2021-10-192-22/+75
| | | | | | | | | | | | | | | | | | | | | | | | | | | Currently generated headers are only included in the srcs split; however, if c or assembly sources depend on the headers, the dependendency is lacking and can cause non-deterministic failures. Test: build/bazel/ci/bp2build.sh Change-Id: Ic354c88300a5fb1b85241f68889e904c33a81488
* | | Merge "Add x86_host vars to config.bzl"Christopher Parsons2021-10-197-69/+194
|\ \ \
| * | | Add x86_host vars to config.bzlChris Parsons2021-10-197-69/+194
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | Test: USE_BAZEL_ANALYSIS=1 m adbd Test: Manually verified config.bzl contains various x86_host flags after bp2build. Test: Unit tests Change-Id: Ie9201ea2be4cd1c6659bea088a797cedbae37403
* | | Merge "Include compos-module-sdk in the SDK build."Treehugger Robot2021-10-191-0/+1
|\ \ \
| * | | Include compos-module-sdk in the SDK build.Victor Hsieh2021-10-151-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | Test: build/soong/scripts/build-mainline-modules.sh Bug: 203231943 Change-Id: Icbd4b15b2434c8a0a7983d52732b5462f3a8b978
* | | | Allow for the installation of binaries in xbinTrevor Radcliffe2021-10-191-0/+9
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | su needs to be installed in xbin, and these changes modify Soong to allow for this. Fixes: 202405540 Test: Confirmed that binary was dropped into xbin Change-Id: Ib664dc1255cd4d9049ac212f79cc53d5038cfcc7
* | | Merge "Reduce modules exposed to Make in Mac builds"Dan Willemsen2021-10-196-9/+72
|\ \ \
| * | | Reduce modules exposed to Make in Mac buildsDan Willemsen2021-10-186-9/+72
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Device builds are no longer supported on Mac, but we do support building various host tools, including the SDK build-tools and platform-tools packages. These have dependencies on [java] device modules, so we don't completely disable device modules, only hide them from Make (which makes them more difficult to trigger from the command line). Also fix the mac build of multiproduct_kati, so that `m blueprint_tools` works on Mac. Bug: 187222815 Test: `m`, `m dist`, etc on Mac Change-Id: I92f16605d5cd173d431cbcb79081234d45cc6e2e
* | | Merge "Fix wildcard ('%') handling in the filter pattern."Alexander Smundak2021-10-192-5/+16
|\ \ \
| * | | Fix wildcard ('%') handling in the filter pattern.Sasha Smundak2021-10-062-5/+16
| | | | | | | | | | | | | | | | | | | | | | | | Fixes: 200094780 Test: internal Change-Id: I5f6a46679cdfa965ad98b9c0c22ef396a13a0bf6
* | | | Merge changes Ifa4594ea,Ia756f309Colin Cross2021-10-182-2/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | * changes: Reduce commands per shard for java protos Up per-product memory usage to 30 GB
| * | | | Reduce commands per shard for java protosColin Cross2021-10-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `m platformprotos` with a long OUT_DIR can trigger E2BIG errors in posixspawn, reduce the commands per shard. Test: OUT_DIR=$PWD/really-long-out-directory m platformprotos Change-Id: Ifa4594eac7ba0e1091df0ef75e4dfc3ff366939b
| * | | | Up per-product memory usage to 30 GBColin Cross2021-10-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reduce multiproduct_kati parallelism by upping required memory per product to 30 GB. Bug: 201671935 Test: builds Change-Id: Ia756f309459d28e22a49f190a5227a139b356a1f
* | | | | Merge "Add tests for defaults used in conditions_default"Liz Kammer2021-10-181-5/+33
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Add tests for defaults used in conditions_defaultLiz Kammer2021-10-181-5/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 203123704 Test: go test soong tests Change-Id: Ia12c51d01108ad1f311d738b182d4bb94c500810
* | | | | Merge "Use hardlinks for Bazel's symlink actions"Christopher Parsons2021-10-182-3/+4
|\ \ \ \ \
| * | | | | Use hardlinks for Bazel's symlink actionsChris Parsons2021-10-142-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This prevents failures resulting from using `cp -d` on symlinks. Test: USE_BAZEL_ANAYSIS=1 m adbd Change-Id: Ic1756ffe407df81b0e99f99c50bee6af4ecf7796
* | | | | | Denylist genrules that create the same file in the same package.Jingwen Chen2021-10-181-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Test: bp2build presubmits Bug: 194642721 Bug: 203369847 Change-Id: Id0faff5fcf4136fac5236980dd60c5d8e54d9547
* | | | | | Fix mac/allowmissingdeps buildsDan Willemsen2021-10-171-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This WalkDeps loop was expecting the dependencies to be fully filled out, which isn't necessarily true on AllowMissingDependencies builds, especially on Mac when the modules may exist, but not be enabled. This was triggered when make_erofs was added to an apex, but wasn't enabled on Mac. It shouldn't be a problem to skip the disabled dependencies, since we'll have already marked the current module as missing dependencies, which means it won't actually get built. Test: `m nothing` on a mac Change-Id: Icd6d597117be4cde5bff041be3fd47361c54cad9
* | | | | | Switch to the new build-tools/platform-tools implementationDan Willemsen2021-10-151-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 187222815 Change-Id: I8ccf92e3185ddb831e8b3f426d397164aef9b7fb
* | | | | | Merge "rust: Package shared libraries with fuzzer zips"Ivan Lozano2021-10-155-83/+115
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | |
| * | | | | rust: Package shared libraries with fuzzer zipsIvan Lozano2021-10-145-83/+115
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rust fuzzers were not packaging up their CC shared dependencies. This would lead to fuzzers using the shared libraries included on system, which may not be sanitized, leading to incorrect behavior. This refactors the relevant code from CC and calls it from the Rust build logic. Bug: 202282599 Test: output rust fuzzer zip file includes shared dependencies. Change-Id: I92196eb0141733797a67eae24f8e9aedea94c3bc
* | | | | | Merge "Add EROFS support for APEX build system"Mohammad Islam2021-10-152-4/+11
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | |
| * | | | | Add EROFS support for APEX build systemHuang Jianan2021-08-022-4/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 195274797 Test: mmm system/apex/apexer/testdata Change-Id: I21dbe8b1bfe02d267c710f3c0dd4abaa737cffe0 Signed-off-by: Huang Jianan <huangjianan@oppo.com>
* | | | | | Merge "Update version_script to be handled as linkopt"Liz Kammer2021-10-154-60/+39
|\ \ \ \ \ \
| * | | | | | Update version_script to be handled as linkoptLiz Kammer2021-10-134-60/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows us to handle this consistently between cc_binary and cc_library* types. Test: build/bazel/ci/bp2build.sh Change-Id: I996f42bbe591215217c3d561662e775925b871ff
* | | | | | | Merge "Make all the PrebuiltEtc modules defaultable."Martin Stjernholm2021-10-151-0/+8
|\ \ \ \ \ \ \
| * | | | | | | Make all the PrebuiltEtc modules defaultable.Martin Stjernholm2021-10-141-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Test: m nothing Change-Id: I69d73ebf2f0761787c6e71241344a43d3b9d49f8
* | | | | | | | Merge "Extract the logic of setting Uncompress_dex to a common place."Jiakai Zhang2021-10-151-15/+14
|\ \ \ \ \ \ \ \
| * | | | | | | | Extract the logic of setting Uncompress_dex to a common place.Jiakai Zhang2021-10-111-15/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 194150908 Test: m nothing Change-Id: Ia33965067586cfd48183e6d17f5d17626eef3c5a
* | | | | | | | | Merge "Add android_sdk_repo_host to build platform-tools&build-tools"Treehugger Robot2021-10-1515-45/+586
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / |/| | | | | | | |
| * | | | | | | | Add android_sdk_repo_host to build platform-tools&build-toolsDan Willemsen2021-10-1315-45/+586
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The Android SDK has been built with Make up until now, monolithically, then split up into several sdk-repo zip files for different purposes. The Mac and Windows SDKs really only need to contain the platform-tools and build-tools pieces, but due to this monolithic sdk zip, we had to build the whole SDK first. This adds an `android_sdk_repo_host` module that can build these platform-tools and build-tools zips. Bug: 187222815 Change-Id: I55809e1d7447dd65e22461f921b2b8abb6d5f822
* | | | | | | | | Merge "Switch to clang-r433403b (13.0.3)"Stephen Hines2021-10-141-2/+2
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / |/| | | | | | | |
| * | | | | | | | Switch to clang-r433403b (13.0.3)Stephen Hines2021-10-141-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: http://b/202099787 Test: m Change-Id: Id923948c56b0308842c94846edd6528fd26dbf72
* | | | | | | | | Merge "Flag an error on empty rust module srcs"Seth Moore2021-10-142-0/+39
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Flag an error on empty rust module srcsSeth Moore2021-10-142-0/+39
| | |_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's an error if a rust module's srcs is empty. Prior to this change the output was a panic that complains loudly about a slice error and nothing else. This change doesn't stop the panic, but at least adds a bit of context so devs who make a simple mistake can more easily fix it. Test: SOONG_GEN_RUST_PROJECT=1 m nothing Change-Id: Id7d8465d533413c3000699661222a53a7c8678f3
* | | | | | | | | Merge "Add missing directories to bp2build."Liz Kammer2021-10-141-2/+10
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Add missing directories to bp2build.Liz Kammer2021-10-131-2/+10
| | |_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added directories with deps of adbd that were not converted and added them to be converted via bp2build. Test: bp2build.sh Test: mixed_droid.sh Change-Id: Iacb634b23e1fdc2767a0923882b71f482ebca53d
* | | | | | | | | Add Global ThinLTO option (2nd try)Yi Kong2021-10-141-36/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of making everything into ThinLTO variant by default (it works but many Soong tests don't like this, and got bit rot due to lack of active builder for this configuration), let the default option be ThinLTO and no LTO be a special variant. Test: m GLOBAL_THINLTO=true Test: m Bug: 195134194 Change-Id: I2fd98061ba55eba1fdfdd056fb2f8c2051fd2553
* | | | | | | | | Merge "Remove `use_apex_name_macro`"Jiyong Park2021-10-143-61/+0
|\ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / |/| | | | | | | |
| * | | | | | | | Remove `use_apex_name_macro`Jiyong Park2021-10-143-61/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The property is no longer used. Bug: 161926892 Test: m Change-Id: I98948152d6d09bde2eeba4875c7ca01531a24930
* | | | | | | | | Merge "Ignore obsolete lines when converting makefiles"Trevor Radcliffe2021-10-132-1/+23
|\ \ \ \ \ \ \ \ \ | |_|/ / / / / / / |/| | | | | | | |
| * | | | | | | | Ignore obsolete lines when converting makefilesTrevor Radcliffe2021-10-132-1/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Specifically this skips cases where LOCAL_MODULE_PATH is TARGET_OUT_OPTIONAL_EXECUTABLES or TARGET_OUT_DATA_APPS. Test: Added tests Test: Treehugger Change-Id: Ib64f22fc3936703c665d07dbb25d8ebf3b9f56c5