aboutsummaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
41 hoursMerge "Revert "Change test_module_config from copying files to symlink ..."" ↵HEADmastermainPriyanka Advani
into main
41 hoursRevert "Change test_module_config from copying files to symlink ..."Priyanka Advani
Revert submission 3060229-ron-tmc-symlinks Reason for revert: Droidmonitor created revert due to b/344045516 Reverted changes: /q/submissionid:3060229-ron-tmc-symlinks Change-Id: If5045366677163560cdae95c0ab74256b4b49b9a
41 hoursMerge "Change test_module_config from copying files to symlink files." into mainRonald Braunstein
44 hoursMerge changes from topic "list_releases" into mainJoe Onorato
* changes: Add list_releases command Add standalone list_products and list_variants commands.
46 hoursMerge "Update exportable to handle documentation issues being errors" into mainTreehugger Robot
48 hoursUpdate exportable to handle documentation issues being errorsPaul Duffin
Follow-up to https://r.android.com/3106478 to have exportable handle documentation issues being errors in the same way as everything does. Bug: 343157584 Test: ./gradlew Change-Id: Iba490e46e2ff202e2ef78f49389319ee8549fe3f
48 hoursMerge "Add `prior_stages:` to release config" into mainTreehugger Robot
2 daysMerge "Limit release config inheritance from aliases" into mainLaMont Jones
2 daysChange test_module_config from copying files to symlink files.Ronald Braunstein
Now the derived modules will have symlink's to base's testcase dir rather than copyfile files from base's intermediates dir. I also removed storing the "base" module as dependency and only use data from the provider in GenerateAndroidBuildActions and AndroidMkEntries. I did have to add two fields to the provider for this. To ensure the symlinks always resolve to a physical file, I also added goals such that building `derived-target` will also build `build-target` Create new Make variable: LOCAL_SOONG_INSTALLED_COMPATIBILITY_SUPPORT_FILES for tasks/general-tests.mk,device-tests.mk to read. This allows "support" files that are installed with soong rules rather than make rules to still end up in the zips. Sample dir structure: % tree -ls testcases aosp_cf_x86_64_phone[4:31:54]/0 [ 4096] testcases ├── [ 4096] FrameworksServicesTests │   ├── [ 4096] data │   │   └── [ 851] broken_shortcut.xml │   ├── [ 2800] FrameworksServicesTests.config │   ├── [ 12695] JobTestApp.apk │   ├── [ 8599] MediaButtonReceiverHolderTestHelperApp.apk │   ├── [ 16791] SimpleServiceTestApp1.apk │   ├── [ 16791] SimpleServiceTestApp2.apk │   ├── [ 16791] SimpleServiceTestApp3.apk │   ├── [ 1017763] SuspendTestApp.apk │   └── [ 4096] x86_64 │   └── [ 79827767] FrameworksServicesTests.apk └── [ 4096] FrameworksServicesTests_contentprotection ├── [ 4096] data │   └── [ 54] broken_shortcut.xml -> ../../FrameworksServicesTests/data/broken_shortcut.xml ├── [ 3005] FrameworksServicesTests_contentprotection.config ├── [ 41] JobTestApp.apk -> ../FrameworksServicesTests/JobTestApp.apk ├── [ 69] MediaButtonReceiverHolderTestHelperApp.apk -> ../FrameworksServicesTests/MediaButtonReceiverHolderTestHelperApp.apk ├── [ 52] SimpleServiceTestApp1.apk -> ../FrameworksServicesTests/SimpleServiceTestApp1.apk ├── [ 52] SimpleServiceTestApp2.apk -> ../FrameworksServicesTests/SimpleServiceTestApp2.apk ├── [ 52] SimpleServiceTestApp3.apk -> ../FrameworksServicesTests/SimpleServiceTestApp3.apk ├── [ 45] SuspendTestApp.apk -> ../FrameworksServicesTests/SuspendTestApp.apk ├── [ 36] test_module_config.manifest └── [ 4096] x86_64 ├── [ 64] FrameworksServicesTests.apk -> ../../FrameworksServicesTests/x86_64/FrameworksServicesTests.apk └── [ 36] UNUSED-FrameworksServicesTests.apk Test: m clean && atest FrameworksServicesTests_contentprotection Test: m clean && atest CtsDevicePolicyManagerTestCases_ParentProfileApiDisabled Bug: b/332320956 Change-Id: I8466f253fa559bc74cef4533edf263650e96bbfb
2 daysMerge "Add RELEASE_APEX_CONTRIBUTIONS_TELEMETRY_TVP build flag" into mainAlyssa Ketpreechasawat
3 daysLimit release config inheritance from aliasesLaMont Jones
If the release config has a name matching build prefixes, it may not inherit from an alias. Bug: 340208722 Bug: 328495189 Test: manual Change-Id: Idb7b1fa372db980c5732b700663553b7a9bf4a36
3 daysAdd `prior_stages:` to release configLaMont Jones
Include those paths in the inheritance graph. Bug: 348495189 Test: manual Merged-In: I993af3a34ab7dd9a3346c6ffccb17e7abff23545 Change-Id: I993af3a34ab7dd9a3346c6ffccb17e7abff23545
3 daysMerge "Make dex_preopt.enable_prof_rewrite=false an error for optimized ↵Treehugger Robot
apps" into main
3 daysMerge "Create inheritance graph" into mainTreehugger Robot
3 daysMerge "Better default value for RELEASE_PLATFORM_VERSION" into mainTreehugger Robot
3 daysCreate inheritance graphLaMont Jones
release-config now creates `inhertance_graph-${TARGET_PRODUCT}.dot` showing the inheritance graph for all release configs present for ${TARGET_PRODUCT}. Bug: 328495189 Test: manual, TH Change-Id: I79242eaa848e0374b33f376e44d8938e9d398e21
3 daysAdd list_releases commandJoe Onorato
Bug: 340648588 Test: list_releases Change-Id: Ibce4704c64620b74c24952f16c2be902d8483f02
3 daysAdd standalone list_products and list_variants commands.Joe Onorato
get_releases coming soon Test: build/make/tools/envsetup/run_envsetup_tests Bug: 340648588 Change-Id: I32e7b868d04656e39f277bb37661eadab9751d7c
3 daysBetter default value for RELEASE_PLATFORM_VERSIONLaMont Jones
If the release config name is a build prefix and different from the inherited value, set RELEASE_PLATFORM_VERSION based on the release config name. Bug: 348495189 Test: manual Change-Id: I95d715150cba9b57e343a8b8364d36f38dcc18a3
3 daysMake dex_preopt.enable_prof_rewrite=false an error for optimized appsSpandan Das
If profile guided dexpreopt _and_ optimization/obfuscation is true, then apps must use the rewritten profile created by r8/d8 Bug: 335418838 Test: m nothing --no-skip-soong-tests Change-Id: Iac45b614f43e52ac40ad1d13df0338bcafa788c3
3 daysAdd RELEASE_APEX_CONTRIBUTIONS_TELEMETRY_TVP build flagAlyssa Ketpreechasawat
Bug: 342624660 Test: check com.google.mainline.go.telemetry_tplus in out/soong/prebuilt_info.json (go target) Change-Id: I201b58c08772669b54c396b4c4dbeccbb4b5147c
3 daysHandle missing apex_contributionsSpandan Das
Some minimal branches have v/g_s/build/release (where apex_contibutions have been set to mainline prebuilts), but not v/g/b (where the apex_contributions for prebuilts have been defined). These minimal branches are unsuitable for building a product that consume mainline prebuilts, but they would still like to do aosp product builds. aosp products should not use the mainline prebuilts anyways, but this has been implemented as - always create the dependency edge to the selected apex contributions - do not visit the dependency edge subsequently if IgnoreApexContributions is set set to true To support aosp product builds in minimal branches, this CL updates the implementation to skip creating the dependency edge when IgnoreApexContributions is set to true Test: go test ./android Change-Id: Iaa0971760e64f9b7a03542f179231ce2268b6616
4 daysMerge "Override modules should always override source modules" into mainSpandan Das
5 daysMerge "Required deps don't track native bridge architectures" into mainJiyong Park
5 daysMerge "Use r8/d8 optimized profile for dexpreopt" into mainTreehugger Robot
5 daysUse r8/d8 optimized profile for dexpreoptSpandan Das
Currently, dexpreopt supports profile guided optimization. This does not work well with r8/d8 optimization, since the checked-in profile will not match the dex signatures after r8/d8 has optimized the dex code. This CL introduces a new property `dex_preopt.enable_profile_rewrting`. If set, the checked-in profile will passed as `input` to r8 via `--art-profile <input> <output>`. The <output> from the previous command will be used as the profile for dexpreopt. Test: m nothing --no-skip-soong-tests Test: m CredentialManager with https://ag.corp.google.com/27448930 and obfuscation turned on Test: nm -U symbol.odex # contains obfuscated methods Bug: 335418838 Change-Id: I53beed9ed76f013262f1c503de0f2b74997c2a7f
5 daysMerge "Revert "Enable -Wfortify-source"" into mainLiana Kazanova
5 daysMerge "Build build_flags.json in soong" into mainTreehugger Robot
5 daysOverride modules should always override source modulesSpandan Das
Since overrides are implemented as variants of the source module, the override module should not be replaced with prebuilts of the source module even when the prebuilt is preferred. Test: go test ./apex Change-Id: I26e97f700276e7beaf6d1bd61b164f11d57a5e09
5 daysRevert "Enable -Wfortify-source"Liana Kazanova
This reverts commit 5735ef6777f11e9057dee1552315a72780817488. Reason for revert: DroidMonitor: Potential culprit for b/343287447 - verifying through ABTD before revert submission. This is part of the standard investigation process, and does not mean your CL will be reverted. Change-Id: I1aeeddc251db760e2ab01c734733d351514ec5b0
5 daysMerge "Update build to handle documentation issues being errors" into mainTreehugger Robot
5 daysMerge "Enable -Wfortify-source" into mainAditya Kumar
5 daysall_flags.pb: Properly handle intermediate filesLaMont Jones
Bug: 343203557 Test: manual, TH Change-Id: I9a4ef39b0c2a641ccd996187ce5f7629520dffc0
5 daysUpdate build to handle documentation issues being errorsPaul Duffin
Previously, documentation issues had `lint` severity (which was between `warning` and `hide`) by default. However, a recent change in Metalava switched documentation issues to `error`. That broke some `droidstubs` modules that either did not specify `api_lint` or were previously allow listed to not use `--lints-as-errors`. This change adds `--error-when-new-category Documentation` to the Metalava arguments on any use of `droidstubs` that did not previously specify `--lints-as-errors` to prevent the build breakage while still warning developers of the problem and preventing new cases being added. Bug: 343157584 Test: ./gradlew Change-Id: Id80bc05e2dade580b2f3ac6416aeeed6b64ba22a
5 daysRequired deps don't track native bridge architecturesJiyong Park
Bug: 342945184 Test: build aosp_cf_x86_64_only_phone and check if out/target/product/vsoc_x86_64_only/system/lib64/arm64 directory doesn't exist. Change-Id: I1638610ac686b369b23390768af1564b3cf38d9b
7 daysAllow `required` property be conditional for release_aidl_use_unfrozenyangbill
Bug: 342336921 Test: cd build/soong/android ; go test -run TestProductVariables Test: add product_variables.release_aidl_use_unfrozen.required to the phony module. Change-Id: I9b934a8fb6245b9b6fd0f2db1e67d5e2876490b2
7 daysMerge "Remove VndkUseCoreVariant related code from build" into mainKiyoung Kim
9 daysget_build_var and get_abs_build_var are now scriptsJoe Onorato
Bug: 340648588 Test: run the scripts. lunch. Change-Id: Idc3395afe0dbec88abe7e8f46994eb2c5a22ee98
9 daysMore envsetup.sh cleanup -- move these to standalone scriptsJoe Onorato
Bug: 340648588 Test: Run them Change-Id: I11d086bcaa2f9eb52f7c556d94666321010b75fa
9 daysbuild/soong side of move the module-info.json commands into standalone scripts.Joe Onorato
Bug: 340648588 Test: run them Change-Id: I5c072ee8481630327be9dccbbca501293a7b34cd
9 daysMerge "Convert some properties to Configurable properties" into mainTreehugger Robot
9 daysMerge "Bump go.mod file to go 1.22" into mainTreehugger Robot
9 daysLet Release_aidl_use_unfrozen add vintf_fragmentsDevin Moore
The product_config variable can be used to add different vintf_fragments. Ignore-AOSP-First: this is required for vFRC finalization and needs to be submitted ASAP. This can be cherry-picked to aosp after finalization. Test: launch_cvd && atest hal_implementation_test Test: lunch `next` && atest vts_treble_vintf_framework_test Bug: 279809333 (cherry picked from https://googleplex-android-review.googlesource.com/q/commit:b93c1a74cf89caef950c63d64b93f2ccfefcbce8) Merged-In: Ied17ad4c4564a5d97f93001734fba7cc6648bbc2 Change-Id: Ied17ad4c4564a5d97f93001734fba7cc6648bbc2
9 daysEnable -Wfortify-sourceAditya Kumar
Workaround for the post-submit failures in b/341339765 has been merged. Test: woodchipper and presubmit Fixes: 315245071 Change-Id: I840f545cc0892cfdf9c896fd45761ba5308ff1ca
10 daysRemove VndkUseCoreVariant related code from buildKiyoung Kim
VndkUseCoreVariant feature is deprecated along with VNDK. This change removes related code from Soong. Bug: 330100430 Test: AOSP CF build succeeded Change-Id: Ie182c9e4dc9cf0a4fe9d5fddf5b36754ddb53d18
10 daysMerge "Propagate DirectlyInAnyApex to transitive dependencies" into mainJihoon Kang
10 daysMerge "Move sanitizer.libraries.txt logic into Soong" into mainKiyoung Kim
10 daysBump go.mod file to go 1.22Cole Faust
When building with blueprint, go 1.22 will be used, but if you used go build or go test manually, the version in the go.mod will be used. Go 1.22 has a subtle change that we want to make sure we're always using: https://go.dev/blog/loopvar-preview Bug: 330574836 Test: Presubmits Change-Id: Idf46b77c4f817345c1f9aca039d8735bb101967f
10 daysMerge "Fix post command of clang_verify." into mainTreehugger Robot
10 daysPropagate DirectlyInAnyApex to transitive dependenciesJihoon Kang
`UpdateDirectlyInAnyApex` is used to propagate the ApexProperties `DirectlyInAnyApex` and `InAnyApex` to the direct dependencies of the direct dependencies of an apex bundle. In other words, this will be propagated only to two-levels max dependency from the apex bundle. However, the implementation library of the sdk library can have longer dependency chain from the apex bundle than two levels: e.g. apex -> bcpf -> sdk_lib -> sdk_lib impl lib Therefore, even if the implementation library of the sdk library is registered as a dependency using the "CopyDirectlyInAnyApexTag" dependency tag, the ApexProperties would not be propagated to the implementation library. In order to resolve this issue and recognize the implementation library to be directly in apex and allow instrumentation for the implementation library, this change proposes propagating `DirectlyInAnyApex` and `InAnyApex` to transitive dependencies on top of direct dependencies. Test: DIST_DIR=dist_dir TARGET_BUILD_VARIANT=userdebug PRODUCT=mainline_modules_x86_64 COVERAGE_MODULES="uwb" ./vendor/google/build/build_unbundled_coverage_mainline_module.sh && \ unzip -l out/target/product/module_x86_64/jacoco-report-classes-all.jar and ensure that framework-uwb is included Bug: 341170242 Change-Id: I27d7a74f6e5bc3e0a044d13c619f4897b6b2eb57