Wrong default targetSdkVersion value in APK build

[brody4hire]

Bug Report

Problem

What is expected to happen?

If I would build with cordova-android from master branch of GitHub, with no special targetSdkVersion value explicitly defined, the resulting APK should have correct default value of targetSdkVersion.

What does actually happen?

If I would add cordova-android platform from GitHub to new Cordova project as follows:

cordova platform add https://github.com/apache/cordova-android#master

then do cordova build android and then check the target SDK version as follows:

aapt list -a platforms/android/app/build/outputs/apk/debug/app-debug.apk | grep -i sdkversion

I get the following result:

    A: android:compileSdkVersion(0x01010572)=(type 0x10)0x1c
    A: android:compileSdkVersionCodename(0x01010573)="9" (Raw: "9")
      A: android:minSdkVersion(0x0101020c)=(type 0x10)0x13
      A: android:targetSdkVersion(0x01010270)="__APILEVEL__" (Raw: "__APILEVEL__")

Note the nonsense android:targetSdkVersion value.

Information

see above

This issue is a result of merged PR #664, which was raised by me.

Command or Code

see above

Environment, Platform, Device

mac OS

Version information

Cordova CLI version: 8.1.2 ([email protected])

Cordova platforms: cordova-android @ bb45f4f from master branch

Other frameworks: none

Operating System: mac OS (10.13.6)

Checklist

• I searched for existing GitHub issues
• I updated all Cordova tooling to most recent version
• I included all the necessary information above

[brody4hire]

I can think of the following alternative solutions:

• revert PR Remove uses-sdk from AndroidManifest.xml #664 (this would be a temporary solution, see I have a question about development with android stuido 3.2 and higher #629)
• fix the Gradle scripts to use defaultTargetSdkVersion from root (top-level) build.gradle file (this would be a partial solution to Gradle default values ignored #658)
• fix the Gradle scripts to use the target SDK value specified in project.properties (this would also be a temporary solution, see Get rid of project.properties files #660)