Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 0ba61ee3 authored by Paul Duffin's avatar Paul Duffin
Browse files

Disable compatibility check for framework-doc*-stubs 

Previously, the `framework-doc-stubs` and `framework-doc-system-stubs`
used the `check_api.last_released` to provide a previously released API
to which flagged APIs could be reverted but hid any compatibility
issues to avoid having to baseline any incompatibilities in multiple
places.

This change disables the compatibility check altogether instead of just
hiding any issues found.

Flag: EXEMPT build fix to improve flag support
Bug: 425309457
Test: TH
Change-Id: I7f94957ad971a38193241d81c95eb22db4a6988c
parent 0e6d7abc

api/ApiDocs.bp

+8 −14
Original line number Diff line number Diff line
@@ -130,10 +130,6 @@ droidstubs { 
droidstubs {

    name: "framework-doc-stubs",

    defaults: ["android-non-updatable-doc-stubs-defaults"],

    flags: [

        // Ignore any compatibility errors, see check_api.last_released below for more information.

        "--hide-category Compatibility",

    ],

    srcs: [":all-modules-public-stubs-source-exportable"],

    api_levels_module: "api_versions_public",

    aidl: {
@@ -143,12 +139,12 @@ droidstubs { 
        ],

    },



    // Pass the previously released API to support reverting flagged APIs. Without this, reverting

    // a flagged API will cause it to be removed, even if it had previously been released. This

    // has the side effect of causing compatibility issues to be reported but they are already

    // checked elsewhere so they will be ignored, see `--hide-category Compatibility` above.

    // Use the compatibility check properties to supply the previously released API to which

    // flagged APIs can be reverted. However, the compatibility check is disabled as it is

    // not required.

    check_api: {

        last_released: {

            enabled: false,

            api_file: ":android.api.combined.public.latest",

            removed_api_file: ":android-removed.api.combined.public.latest",

        },
@@ -160,17 +156,15 @@ droidstubs { 
    defaults: ["framework-doc-stubs-sources-default"],

    flags: [

        "--show-annotation android.annotation.SystemApi\\(client=android.annotation.SystemApi.Client.PRIVILEGED_APPS\\)",

        // Ignore any compatibility errors, see check_api.last_released below for more information.

        "--hide-category Compatibility",

    ],

    api_levels_module: "api_versions_system",



    // Pass the previously released API to support reverting flagged APIs. Without this, reverting

    // a flagged API will cause it to be removed, even if it had previously been released. This

    // has the side effect of causing compatibility issues to be reported but they are already

    // checked elsewhere so they will be ignored, see `--hide-category Compatibility` above.

    // Use the compatibility check properties to supply the previously released API to which

    // flagged APIs can be reverted. However, the compatibility check is disabled as it is

    // not required.

    check_api: {

        last_released: {

            enabled: false,

            api_file: ":android.api.combined.system.latest",

            removed_api_file: ":android-removed.api.combined.system.latest",

        },