Upgrade to target SDK version 34

[jere-mie]

Is your feature request related to a problem? Please describe.
Currently, bubblewrap targets SDK version 33, which is no longer supported for apps to be published on the Play Store.

Describe the solution you'd like
Target SDK version 34

Describe alternatives you've considered
Manually specifying a specific SDK

Additional context
This issue for PWABuilder is what prompted me to create this: pwa-builder/PWABuilder#4766

[jere-mie]

It appears that changing export const BUILD_TOOLS_VERSION = '33.0.2'; to some version above 34 in packages/core/src/lib/androidSdk/AndroidSdkTools.ts should be sufficient, though I haven't been able to verify

[andreban]

CC @ibrahimkarahan

[andreban]

It appears that changing export const BUILD_TOOLS_VERSION = '33.0.2'; to some version above 34 in packages/core/src/lib/androidSdk/AndroidSdkTools.ts should be sufficient, though I haven't been able to verify

Most likely you'd want to update targetSdkVersion and compileSdkVersion on build.gradle. Build tools, as you mentioned may be needed to.

The thing to be aware of is that Android introduces changes to OS between different target SDKs - once upgraded, there's a chance the generated application won't build anymore and require changes to fix it, or that it may behave differently at runtime.

[jere-mie]

The process of bumping versions is most likely the same as this PR: #764

CCing @PEConn since they were involved with the bump to v33

[siaeweb]

Also facing the same issue while creating my first PWA app. Upgrade to target SDK version 34.

[Aviv1000]

same here

[charlypa]

[jere-mie]

it would be nice if we could add a cli flag or environment variable or something of that nature that allows us to specify the default target - and would prevent this from being a blocker for people in the future when Google decides change the requirements again.

Maybe that's an idea for a separate issue though

[jgw96]

@andreban are we just looking for someone to make the changes you mentioned and put them up in a PR?

[andreban]

@andreban are we just looking for someone to make the changes you mentioned and put them up in a PR?

Looping @ibrahimkarahan for thoughts here.

[ibrahimkarahan]

A few people are out of office due to holidays in the US. We can take a look at this as soon as they’re back. PRs are also welcome.

…

On Tue, Jul 9, 2024 at 19:41 André Cipriani Bandarra < ***@***.***> wrote: @andreban <https://github.com/andreban> are we just looking for someone to make the changes you mentioned and put them up in a PR? Looping @ibrahimkarahan <https://github.com/ibrahimkarahan> for thoughts here. — Reply to this email directly, view it on GitHub <#860 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AANNGTBO3MJLB3KEFKGUAETZLQODBAVCNFSM6AAAAABKMLTJPGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJYGMYDENBVGE> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[A1kmm]

I didn't see this issue until now, but I did make a PR: #862. It seems to work to produce SDK version 35 APKs - but took quite a number of consequential updates (it isn't as simple as just bumping the version, as quite a few dependencies are old (including JDK), and Bubblewrap doesn't even build cleanly within the current NPM dependencies right now).

My PR doesn't upgrade SDK tools though.

If anyone wants to test it out while it is being reviewed:

git clone https://github.com/GoogleChromeLabs/bubblewrap
cd bubblewrap
git fetch origin pull/862/head:sdk35
git checkout sdk35
npm install && npm run build

That should give you a working script in ./packages/cli/bin/bubblewrap.js, which you can symlink as bubblewrap from somewhere in your path if you want (or just run it by absolute path). You might need to delete the jdkPath key from the JSON in ~/.bubblewrap/config.json if you have previously installed an older JDK with Bubblewrap (use the doctor subcommand to check if unsure). Since my changes are quite significant due to the number of things updated, more testing by other people would be appreciated.

[lukastribus]

The PR was merged. But a bubblewrap release is probably necessary for other projects to be able to pick up the change.

[viral32111]

When can we expect a release?

[Mahera72]

***@***.*** في الثلاثاء، ٢٣ يوليو ٢٠٢٤، ٩:١٨ ص viral32111 ***@***.***> كتب:

…

When can we expect a release? — Reply to this email directly, view it on GitHub <#860 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BI64KGDFMNSEMO3AVCZSFEDZNXYVFAVCNFSM6AAAAABKMLTJPGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBUGM2DAMJYHE> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[kaktus42]

For me it worked to change

• compileSdkVersion 33 to compileSdkVersion 34 and
• targetSdkVersion 33 to targetSdkVersion 34

in the file app/build.grade and then run `bubblewrap build.

Whenever you run bubblewrap update, it will overwrite app/build.grade and you have to make the changes again. (At least until bubblewrap gets an update.)

The comment by @andreban and comment by @jere-mie put me on the right track.

[Cristian021195]

The kaktus42 way worked for me, just one more thing, if you are using bubblewrap for twa and make any change in twa-manifest.json, when you wanna build it again, will set compileSdkVersion 33 again, so keep in mind.

[dhalldmg]

I followed @kaktus42 steps also (thank you for sharing). However, I'm confused because this warning is not going away.
The detail when clicking on "Go to Policy status" says that "App must target Android 14 (API level 34) or higher" Is there still something that needs done?

[maiconcarraro]

@dhalldmg the warning is saying something about policies it can be related to other changes, recently I got a similar warning to check some specific features (comply or not), recommend to check.

[dhalldmg]

@dhalldmg the warning is saying something about policies it can be related to other changes, recently I got a similar warning to check some specific features (comply or not), recommend to check.

The policy status / issue detail is "App must target Android 14 (API level 34) or higher"

[ibrahimkarahan]

Hi everyone, thanks for your patience! We just pushed version 1.22.0 into npm which bumps up targetSdkVersion to 35 and Play Billing Library to v6. You can update your Bubblewrap version to regenerate the Android project and upload to the Play Store.

[cmizzi]

Hi everyone, thanks for your patience! We just pushed version 1.22.0 into npm which bumps up targetSdkVersion to 35 and Play Billing Library to v6. You can update your Bubblewrap version to regenerate the Android project and upload to the Play Store.

The tag has been pushed as well as the Docker image. But the Docker image is still running the version 1.21.0.

docker run -ti --rm ghcr.io/googlechromelabs/bubblewrap:1.22.0 --version

,-----.        ,--.  ,--.  ,--.
|  |) /_,--.,--|  |-.|  |-.|  |,---.,--.   ,--,--.--.,--,--.,---.
|  .-.  |  ||  | .-. | .-. |  | .-. |  |.'.|  |  .--' ,-.  | .-. |
|  '--' '  ''  | `-' | `-' |  \   --|   .'.   |  |  \ '-'  | '-' '
`------' `----' `---' `---'`--'`----'--'   '--`--'   `--`--|  |-'
                                                           `--'
version 1.21.0

Probably related:

• Dockerfile needs to upgrade JDK after v1.22.0 #877
• JDK version error on build after updating to CLI v1.22.0 #873