This reverts commit 3bf31401fe.
Reason for revert: Build cop revert because this change is breaking Blueline builds.
Change-Id: I54f87f21eec0e93873ffedec6b0bbe154584ba4b
Otherwise, it causes the CTS-on-GSI failure on non-emulator devices:
CtsPermission2TestCases android.permission2.cts.PrivappPermissionsTest#testPrivappPermissionsEnforcement
Because non-emulator devices lack the permission file in their /vendor.
/system/priv-app/SdkSetup/SdkSetup.apk
/vendor/etc/permissions/privapp-permissions-goldfish.xml
Bug: 110517983
Test: boot a GSI and checks that SdkSetup.apk is moved to /vendor
Change-Id: I3086f99c477eeca38f42041e1809a5aa3b8bfa7a
This only includes a few tests to start with.
Bug: 62492960
Test: Ran new unit tests.
Test: Ran with all different options and verified it produces the same
Test: output as the previous script.
Change-Id: Iad29a5f04f49986139c92030a3259cae512859af
This commit adds `vndk-lib-extra-list-current.txt`. This file contains
the LLNDK/VNDK/SP-HAL shared libraries (and dependencies) that cannot be
specified in Android.bp.
Bug: 80521281
Test: make vts
Change-Id: If7b414589786dab424a21773bfd33f10b415d899
Merged-In: If7b414589786dab424a21773bfd33f10b415d899
(cherry picked from commit 7e177685d8)
This commit checks whether the path maps to a git project specified in
`manifest.xml`. If it does not map to a git project, emit an error and
skip it. If we don't do so, an exception will be raised because the
`sorted()` function cannot compare `None` object with `str` object.
Test: Add a project to external and it does not stop the analyzer.
Change-Id: I386ab7c1f76d5630a2c5f43186f38a791a3bb7b9
In Android P, the version field did not get incremented properly in
all places. So if a file with a v1.0 version is found, do a further check
to verify if the num field is really valid.
Test: Verified that a file with v1.0 gets checked and the valid num field
Test: is set correctly.
Test: Verified that a file with v1.1 automatically gets num field set to
Test: valid.
Change-Id: I7f828294c833e334cad76a866bbaa1d78ac02cfb
Merged-In: I7f828294c833e334cad76a866bbaa1d78ac02cfb
(cherry picked from commit b558fe4d53)
Add 'compile_dex' flag to API stub rules. Soong will generate build
targets which compile the JARs into dex files. Those will be compared
against the boot class path dex files to determine the public API surface.
Bug: 79409988
Test: m android_stubs_current, check javalib.jar contains classes.dex
Change-Id: Ie7238572ece0678133ba08203108a923ea2f56ba
- Convert method declarations to thunk symbols if thunk information
exists.
- Add a test case and reference dumps for virtual inheritance.
Bug: 80671888
Test: ./tests/test.py
Change-Id: I55ecbd7c8b0f5b0f534d81e6c6967f99ffd1d42e
Switching from remote to extended-remote allows the program being
debugged to be restarted like a normal non-remote program.
Test: Used gdbclient debugging ICU things for a while, restarted often
Bug: None
Change-Id: Ia295749b9aa86727479d760d7c743f98a06fa6d4
Starting in VNDK snapshot v28 (P), prebuilts built for 32-bit
binder interface are isolated in separate 'binder32' subdirectory
under prebuilts/vndk/v{VER}/{ARCH}/. All other prebuilts are
assumed to use 64-bit binder interface.
To differentiate prebuilts per binder bitness, the following
changes are made:
1) a new 'binder32bit' property added to the Android.bp files
autogenerated for the vndk_prebuilt_shared soong module
2) new phony modules, vndk_v{ver}_{arch}_binder32, defined for
32-bit binder prebuilts,
3) 'binder32' suffix added to the prebuilt versioned name,
e.g. libfoo.vndk.{ver}.{arch}.binder32.vendor
Test: python gen_buildfiles.py 28 -vv
Test: python gen_buildfiles.py 27 -vv
Bug: 78279738
Change-Id: I283fe25a34d3381d13097ede34c428c83dffe11e
This commit adds a missing `__init__.py` to `ninja/tests`, so that
unit test discovery can find unit tests without any arguments.
Test: sourcedr/ninja/tests/run_tests.sh
Change-Id: I6077d014f33ccb176bf2138e3a0035c871c8eead
`string.decode('mutf-8')` converts `bytes` into `str` in Python 3. On
the other hand, `string.decode('mutf-8')` converts `str` to `unicode` in
Python 2. However, finding a `str` object in a set of `unicode`
objects may lead to false negatives.
This commit fixes the problem by encoding strings into `utf-8` after
decoding from `mutf-8`.
Bug: 110450167
Test: ./tests/run.py
Change-Id: Ifa6e97fffb5f364d0488cc98b6c59fbf4fb7869f
This commit adds a manifest split analysis tool. This tool classifies
the Android.bp modules into 3 categories (i.e. system-only modules,
vendor-only modules, and both-side module) and list the git repositories
that define modules in two or more categories.
Test: analyze_manifest_split.py \
-m ${ANDROID_BUILD_TOP}/.repo/manifest.xml \
-b ${ANDROID_BUILD_TOP}/Android.bp
Change-Id: I74c4acc3cac8fc2795cbc03806e71685e1910818
The previous CL silently ignores all dex strings that cannot be decoded
with Modified UTF-8 decoder. However, bad unicode encodings are
usually a good indicator that the dex file format is changed. Thus,
this commit separates invalid surrogates as special cases and only
ignore those dex strings. Exceptions for other invalid encodings are
propagated after this CL.
Bug: 110450167
Test: ./tests/run.py
Change-Id: I61fd754eab76f0b9eb3c58ac7f48321258e46514
This commit lifts the decoding logic from DexFileReader. After this
commit, DexFileReader simply returns modified UTF-8 so that the user can
decide whether to decode it or not.
Bug: 110450167
Test: ./tests/run.py
Change-Id: I71bdc5e20553a17e9eab3eac597648e397e99730
Previously the --local option assumed the local dir to be DIST_DIR.
Since in the actual use case, the branch where the VNDK snapshot is
built is different from the branch where this VNDK snapshot is
uploaded to, allow the user to specify the local dir as a command
line parameter.
Test: python update.py --local=<path to local dir> 28 -vv
Bug: 78279738
Change-Id: I98a86296076994c951b98457ef47c384d4cc940c
Revisions in a *-release branch include merge CLs that merge
*-dev to *-release. These revisions will not be in the source
of a GPL git project, so in these cases need to extract the parent
revision of the merge commit.
Test: python check_gpl_license.py --branch pi-release --build
<bid> 27 -vv
Bug: 78279738
Change-Id: I2950b56605b9af39c3ab012ae7a1ad407942d419
This commit add an access check in _enumerate_path() so that soft links
that cannot be resolved on host can be ignored.
Bug: 80459974
Test: vndk_definition_tool.py check-dep --check-apk \
--system ${ANDROID_PRODUCT_OUT}/system \
--vendor ${ANDROID_PRODUCT_OUT}/vendor
Change-Id: I0b0b793784b005139a62c5a48a5bd8f48cdd55c3
