- Feb 28, 2024
-
-
Jihoon Kang authored
With the effort to support non-"everything" stubs generation from api signature files, https://r.android.com/2977092 enforces all java_api_library modules to specify the `stubs_type` property. In order to accustom to this enforcement, this change specifies the `stubs_type` property as "everything" to all bp-defined "everything" stubs generating java_api_library modules. Test: m nothing --no-skip-soong-tests Bug: 319162970 Change-Id: I0a21420b4aa9eccd1eeb2022f3c9c952f607a480
-
- Feb 15, 2024
-
-
Jihoon Kang authored
Currently in AconfigFlags.bp, two list of aconfig modules are being maintained: the list of java_aconfig_library modules and the list of aconfig_declarations modules. When a new aconfig module is being added, a developer must modify the two lists, and there is no guarantee that the two lists are kept in sync. In order to resolve such issue, this change replaces the two lists with the newly introduced aconfig_declarations_group module. Test: m Bug: 320492079 Merged-In: Ib32c06de4f0279a17e5c65314e984265592d004c Change-Id: Ib32c06de4f0279a17e5c65314e984265592d004c
-
Jihoon Kang authored
Currently in AconfigFlags.bp, two list of aconfig modules are being maintained: the list of java_aconfig_library modules and the list of aconfig_declarations modules. When a new aconfig module is being added, a developer must modify the two lists, and there is no guarantee that the two lists are kept in sync. In order to resolve such issue, this change replaces the two lists with the newly introduced aconfig_declarations_group module. Test: m Bug: 320492079 Change-Id: Ib32c06de4f0279a17e5c65314e984265592d004c
-
Jihoon Kang authored
Currently in AconfigFlags.bp, two list of aconfig modules are being maintained: the list of java_aconfig_library modules and the list of aconfig_declarations modules. When a new aconfig module is being added, a developer must modify the two lists, and there is no guarantee that the two lists are kept in sync. In order to resolve such issue, this change replaces the two lists with the newly introduced aconfig_declarations_group module. Test: m Bug: 320492079 Merged-In: Ib32c06de4f0279a17e5c65314e984265592d004c Change-Id: Ib32c06de4f0279a17e5c65314e984265592d004c
-
- Feb 14, 2024
-
-
Jihoon Kang authored
This change sets is_stubs_module property to true for combined modules genreated in `combined_apis` and non-updatable stub modules, which are not generated via java_sdk_library but are maintained separately. Test: m nothing --no-skip-soong-tests Bug: 310504781 Merged-In: I1293208e575a76db2cc4dc6c8564369804bb4ac5 Change-Id: I1293208e575a76db2cc4dc6c8564369804bb4ac5
-
- Feb 13, 2024
-
-
Peter Kalauskas authored
Bug: 236188940 Flag: NONE Test: m private_api_stubs dist Change-Id: I7bfa72cd1db254a9a6d26c07a0219ce8671829f0
-
- Feb 12, 2024
-
-
Jihoon Kang authored
This change sets is_stubs_module property to true for combined modules genreated in `combined_apis` and non-updatable stub modules, which are not generated via java_sdk_library but are maintained separately. Test: m nothing --no-skip-soong-tests Bug: 310504781 Change-Id: I1293208e575a76db2cc4dc6c8564369804bb4ac5
-
Spandan Das authored
This ensures that we do not run into circular depencies when UNBUNDLED_BUILD_TARGET_SDK_WITH_API_FINGERPRINT=true. When this variable is true, apps compile with a min_sdk_version equivalent to the sha of the apis (i.e. the java_genrule `api_fingerprint`). If the api stubs are allowed to depend on android apps, we will run into circular deps Implementation details 1. Modify the deps mutator of combined_apis to add a dependency on the stub java modules 2. Do a graph walk in GenerateAndroidBuildActions and ensure that no child is an android_app. This will be determined by casting to AndroidLibraryDependency. Re (2): this does a graph walk via ctx.WalkDeps, but should not be computationally intensive because (a) We do this for a single module (i.e. combined_apis) (b) The transitive closure is shallow since it starts from stubs Test: presubmits Test: locally, added an app to libs of `service-sdksandbox` and verified that the error is raised Bug: 315016205 Change-Id: Iaf35f03171d13d75e75de6e0e744fcf34e3294fd
-
- Feb 11, 2024
-
-
Nishant Panwar authored
SDK library for pdf related bootclasspath classes that were part of the non-updatable API before V, and were moved to the module in V. Other pdf related bootclasspath classes in mediaprovider should go to framework-pdf. Bug: b/324667310 Test: presubmit and m Change-Id: I3f7469ac72052e7529e998179fe57db1a5b39c82
-
- Feb 07, 2024
-
-
Jihoon Kang authored
Currently, ther merged txt filegroups are dist'ed as api/*-current.txt and api/*-removed.txt files during droid dist build. This change adds additional merged txt filegroup modules that depend on the non-updatable exportable txt files instead of the checked in non-updatable txt files (under frameworks/base/core). These added modules will be dist'ed in droid build, instead of the currently existing merged txt filegroups. Test: m nothing && inspect ninja output Bug: 324254328 Change-Id: Ib0b47ba032a2917d0f2df6ad585d2102d6940c50
-
- Feb 06, 2024
-
-
Jihoon Kang authored
frameworks-base-api-current.srcjar is used to generate the stubs srcjar file in dist directory during the sdk builds. Given that the sdk build artifacts should depend on the exportable stubs instead of the everything stubs, the src of this module should also be replaced with the exportable stubs from the everything stubs. Given that the only usage of this module is the stubs srcjar generation during the sdk builds, the source of this module can be safely replaced with the exportable stubs instead of creating a new module. Test: patch in git_main, lunch aosp_arm-ap31-eng && m sdk dist and inspect ninja query to verify that android-stubs-src.jar does not depend on the everything stubs Bug: 323261972 Change-Id: If37aa6ef862c69619768f91f2360b37459fd3cdb
-
- Feb 02, 2024
-
-
Jihoon Kang authored
This change modifies the sdk-annotations-*.zip definitions so that the modules depend on the exportable annotations.zip files instead of the everything annotations.zip files. This change also modifies the definition of the android_*_stubs_current_with_test_libs modules definitions to depend on the exportable stubs. Likewise, the droiddoc modules now depend on the exportable stubs. Test: m docs dist && inspect ninja file to verify that everything stubs are not exported Test: m sdk dist && inspect api-versions.xml file and verify that the disabled apis are not included Bug: 323261972 Change-Id: I015921f0d006bac29492be22b83043b39a8fe8eb
-
- Feb 01, 2024
-
-
Yisroel Forta authored
Test: m, presubmit Bug: 293957254 Change-Id: I50d024c167094dc559070f529a91af6e9cb49872
-
- Jan 18, 2024
-
-
Harshit Mahajan authored
ed0743da Bug:b/289203818 Test: m nothing Change-Id: Id156c0fd4b7b783c5b9f1488914f5650e30ffed5 Merged-In: If6789fee9a908231babd7624280b40515d377dfe
-
- Jan 17, 2024
-
-
Harshit Mahajan authored
Adding the module libraries to combined_apis for them to be added to Android SDK. Also adding bootclasspath-fragment to platform-bootclasspath. Bug: b/289203818 Change-Id: If6789fee9a908231babd7624280b40515d377dfe Test: m
-
- Jan 16, 2024
-
-
Jihoon Kang authored
This change pass the aconfig_declaration modules defined in AconfigFlags.bp to the *-api-stubs-docs-non-updatable droidstubs module, so that the "exportable" stubs (i.e. the stubs only containing the flagged apis that are enabled by the release configuration) can be generated. Test: m api-stubs-docs-non-updatable && m checkapi Bug: 317290555 Merged-In: I944d336e474b9ed61f4554fc9325a8df3a9cf40b Change-Id: I944d336e474b9ed61f4554fc9325a8df3a9cf40b
-
Jihoon Kang authored
This change pass the aconfig_declaration modules defined in AconfigFlags.bp to the *-api-stubs-docs-non-updatable droidstubs module, so that the "exportable" stubs (i.e. the stubs only containing the flagged apis that are enabled by the release configuration) can be generated. Test: m api-stubs-docs-non-updatable && m checkapi Bug: 317290555 Change-Id: I944d336e474b9ed61f4554fc9325a8df3a9cf40b
-
- Jan 12, 2024
-
-
Roshan Pius authored
Now that all the @hide usage of NFC API's from rest of platform is cleaned out, we can now switch to the module defaults and continue the separation of NFC API classes. Bug: 303286040 Test: Compiles Merged-In: I8bd5e38b1482600594c41660d14537cf15b48318 Change-Id: I8bd5e38b1482600594c41660d14537cf15b48318
-
- Jan 11, 2024
-
-
Mårten Kongstad authored
When building SDKs that include some, but not all @FlaggedApis, a few Javadoc @see/@link references will point to objects that are inaccessible, which will trigger Lint 101 errors. The correct fix is to update the Javadoc. However, to unblock the SDK build for now, temporarily hide Lint 101 errors. Bug: 319541666 Test: m sdk dist Change-Id: Icbd1d99d5fcd6cee37bf588ef45443e7e946f742
-
Jihoon Kang authored
The two major roles of the full api surface stubs (i.e. android_*_stubs_current) are: - Appended to classpath when a module sets sdk_version - Input to generating hiddenapi flags and determine the accessibility of the api at runtime With Trunk Stable, all flagged apis that are not enabled should not be accessible at runtime. In other words, the currently existing full api surface stubs that include both non flagged and flagged apis should not be used to generate the hiddenapi flags. To resolve this problem, this change introduces full api surface stubs of the "exportable" stubs (i.e. non flagged apis + flagged apis enabled by release configuration). The new modules are named similarly to the currently existing full api surface stubs, but appends the "_exportable" prefix to differentiate with the currently existing modules. These modules are solely used for hiddenapi flags generation, and the currently existing modules continue to serve the first aforementioned role. The new modules will toggle between the from-text stubs module and the from-source stubs module when the exportable stubs can be generated from the from-text stubs generating `java_api_library` module. For now, these modules are always built from source. Test: m android_stubs_current_exportable Bug: 317426356 Change-Id: I1f6f738ae9c5ebf151a1554aa1e357ef0092c89c
-
- Jan 10, 2024
-
-
Harshit Mahajan authored
Allow specifying default combined api properties. This would be currently used to control the enablement of new mainline modules Bug:b/289203818 Test: verified locally Change-Id: I9415140453ce04efd5461cb8684c70e987810ecc
-
Roshan Pius authored
Now that all the @hide usage of NFC API's from rest of platform is cleaned out, we can now switch to the module defaults and continue the separation of NFC API classes. Bug: 303286040 Test: Compiles Change-Id: I8bd5e38b1482600594c41660d14537cf15b48318
-
- Jan 08, 2024
-
-
Cole Faust authored
This android.bp file was moved, and the comment needs to be updated to reflect that. Test: Presubmits Change-Id: I225690505ccbf0f2234e65cbc668ade9bfbd5e17
-
- Jan 05, 2024
-
-
Paul Duffin authored
Copies the system version of the `api-versions.xml` file to a file `api-versions_system.xml` in the top level dist directory. Bug: 190665366 Test: m sdk dist # Check to make sure that the file exists. Change-Id: Ie139f81a819eb5f389363ffdefa7bc42d36804e0
-
- Jan 02, 2024
-
-
Nelson Li authored
Since Soong has been able to correctly add dependencies to .PHONY targets since aosp/2884366, it is now possible to directly use `phony_deps` to add the genrules used by `checkapi` to its dependencies and convert them to Android.bp. Bug: 309730110 Test: m checkapi Change-Id: Ideb9f647415423a5864d221b85442b995ac67906
-
- Dec 19, 2023
-
-
Paul Duffin authored
The `--api-lint-ignore-prefix` option adds a package prefix within which API lint checks will be ignored. However, the options are not actually necessary. That is because they are only used in the `android-non-updatable-stubs-defaults` and the non updatable code does not include any source classes from those packages. While modules that do use `android-non-updatable-stubs-defaults` do include classes from those packages on the classpath they are already excluded from having API lint checks performed on them as they do not contribute to the API surface being generated by those modules. Bug: 311690008 Test: m checkapi Change-Id: Ife750c586edc2b5af92c0b5e9211b23345083f69
-
- Dec 14, 2023
-
-
Yiming Pan authored
Bug: 312784289 Test: mmm -j frameworks/base/api/coverage/tools/:extract-flagged-apis extract-flagged-apis <input api.text file> <output pb file> Change-Id: Ibc2e1af9c3346543365ce42058b6e5913161da9f
-
- Dec 08, 2023
-
-
Yiming Pan authored
* Parameter list should be a part of the API signature. * Package, class, and method names are stored separately so they don't need to be parsed when being read. Bug: 315412454 Test: mmm -j frameworks/base/api/coverage/tools/:extract-flagged-apis extract-flagged-apis <input api.text file> <output pb file> Change-Id: I769cc10831f1c50c71e71cbaadec355f21258baf
-
Colin Cross authored
Bug: 315353489 Test: m blueprint_tests Change-Id: I50d938017203ad12198d8a6373c9aef80a8535c6
-
- Nov 20, 2023
-
-
Yiming Pan authored
Flag to API mapping files will be built and uploaded to `droid` target. Test: m flag-api-mapping-PublicApi m flag-api-mapping-SystemApi m flag-api-mapping-ModuleLibApi m flag-api-mapping-SystemServerApi Bug; 311059624 Change-Id: Ibb872e2c1f7772af59427bcec3ce6f6551a8dd17
-
Yiming Pan authored
Test: mmm -j frameworks/base/api/coverage/tools/:extract-flagged-apis extract-flagged-apis <input api.text file> <output pb file> Bug: 311059624 Change-Id: I7db4296e4ee78268b886b83cd822d8ebe151c4a5
-
- Nov 17, 2023
-
-
Roshan Pius authored
Bug: 303286040 Test: Device boots up after flashing Merged-In: Iad08614f0aa3b7bea12a1ad52debd0ee13302a8e Change-Id: Iad08614f0aa3b7bea12a1ad52debd0ee13302a8e
-
Roshan Pius authored
Bug: 303286040 Test: Device boots up after flashing Change-Id: Iad08614f0aa3b7bea12a1ad52debd0ee13302a8e
-
- Nov 16, 2023
-
-
Yiming Pan authored
This CL creates a binary that extracts flagged APIs from an API signature file, and writes the mapping of flags to APIs to a pb file. Bug: 311059624 Test: mmm -j frameworks/base/api/coverage/tools/:extract-flagged-apis extract-flagged-apis <input api.text file> <output pb file> Change-Id: I9b6f45bd126cd6a5b2ed1e9f83bdaa12dd580480
-
- Nov 15, 2023
-
-
Harshit Mahajan authored
Revert submission 2789412-cherrypicker-L65800000963076834:N61100001405784943 Reason for revert: Leading to merge conflict Bug: b/311082270 Reverted changes: /q/submissionid:2789412-cherrypicker-L65800000963076834:N61100001405784943 Change-Id: Ic80afd7a8e09cf2e0cf6e453cbe004db0c9fcbd7 Merged-In: I5c65d85d50b214926f5473908694070b4e896201
-
- Nov 14, 2023
-
-
Anton Hansson authored
The issues have been fixed. Bug: 303477132 Test: m docs Change-Id: Ic1e5cf00f1973a554af81192aa52d6839c77859e
-
- Nov 13, 2023
-
-
Elis Elliott authored
Fixes: 305195721 Test: delete relevant lines in javadoc-lint-baseline then run m docs. Change-Id: I4be8b3720bf9e9373fa173750ae070718a4fc834
-
- Nov 08, 2023
-
-
Alyssa Ketpreechasawat authored
Revert submission 2775627-revert-2759049-framework-pdf-UAOVGTHHXO Reason for revert: submit together with the fix to allow this new jar to coexist w/ mainline prebuilts Reverted changes: /q/submissionid:2775627-revert-2759049-framework-pdf-UAOVGTHHXO Change-Id: Idf009cf17e142a04f4ca73396c6559c14504fc6a
-
- Nov 06, 2023
-
-
Anton Hansson authored
The non-updatable part of the API has a few deps on module APIs. Extract the list of libs to satisfy those deps into a variable so that it can be shared between the stubs_default and the java_library. Test: m Change-Id: Id416d294150131b477fc9c73a2d5811cfbf7012f
-
- Nov 02, 2023
-
-
Anton Hansson authored
These checks have been disabled since the initial integration of metalava in the build system. Stop disabling them, but baseline existing issues. For the checks that are disabled for good reason, leave a comment that explains why. Bug: 306806877 Test: m checkapi Change-Id: Ica5c28f798b4aee0ae3722a0bd176bcc621d994e
-